Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Design rationale for DHT #304

Merged
merged 2 commits into from
Jul 24, 2024

Conversation

chirag-parmar
Copy link
Contributor

Description

This PR adds a section (under protocol/design-rationale.md) which addresses the use of DHTs within the Ethereum protocol. It also aims to answer, though not directly, the question raised here.

Copy link
Contributor

@taxmeifyoucan taxmeifyoucan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks for such a quick dive into this!

docs/wiki/protocol/design-rationale.md Outdated Show resolved Hide resolved
docs/wiki/protocol/design-rationale.md Outdated Show resolved Hide resolved
docs/wiki/protocol/design-rationale.md Outdated Show resolved Hide resolved
docs/wiki/protocol/design-rationale.md Outdated Show resolved Hide resolved
@taxmeifyoucan taxmeifyoucan merged commit fdf8443 into eth-protocol-fellows:main Jul 24, 2024
1 of 2 checks passed
taxmeifyoucan added a commit that referenced this pull request Jul 24, 2024
* Add study group info (#298)

* ✨ feat: Prehistory of Ethereum (#226)

* ✨ feat: p2p computing

* ✨ feat: Overview: Bitcoin

* ✨ feat: Overview: Ethereum world comptuer

* ✨ feat: Overview: Applications

* 🥢 nit:

* nit

Co-authored-by: Mário Havel <[email protected]>

* ✨ feat: Revised intro

* ✨ feat: Internet

* ✨ feat: Map of ARPANET

* ✨ feat: Unix

* ✨ feat: fin

* 🥢 nit: intro

* ✨ feat: Documentary GNU/Linux, Crypto Anarchy

---------

Co-authored-by: rahul <[email protected]>
Co-authored-by: Mário Havel <[email protected]>

* Add files via upload (#302)

* Create content for scourge under the staking economics track (#300)

* change title lowercase to uppercase

* create content for scourge under the staking economics track

* remove extra space in reference

* add words to wordlist

* reordered the numbering and removed unsed content

* MEV-track and staking economics track in the same table

removed the added tablea header: MEV-track and staking economics track can be in the same table

* reordered the staking economics table content

---------

Co-authored-by: DanGoron <[email protected]>

* Design rationale for DHT (#304)

* design rationale for DHT

* Apply suggestions from code review

Co-authored-by: Mário Havel <[email protected]>

---------

Co-authored-by: Mário Havel <[email protected]>

---------

Co-authored-by: rahul <[email protected]>
Co-authored-by: rahul <[email protected]>
Co-authored-by: Rory Arredondo <[email protected]>
Co-authored-by: Glory Agatevure <[email protected]>
Co-authored-by: DanGoron <[email protected]>
Co-authored-by: Chirag Parmar <[email protected]>
@shyam-patel-kira
Copy link
Member

The spell check is still broken because of the words never included in wordlist

@chirag-parmar chirag-parmar deleted the dht-rationale branch July 26, 2024 06:03
@chirag-parmar
Copy link
Contributor Author

chirag-parmar commented Jul 26, 2024

@shyam-patel-kira thanks for pointing it out. I did the typo check but forgot to add to the list. Should I create a new PR?

taxmeifyoucan added a commit that referenced this pull request Aug 27, 2024
* Add study group info (#298)

* ✨ feat: Prehistory of Ethereum (#226)

* ✨ feat: p2p computing

* ✨ feat: Overview: Bitcoin

* ✨ feat: Overview: Ethereum world comptuer

* ✨ feat: Overview: Applications

* 🥢 nit:

* nit

Co-authored-by: Mário Havel <[email protected]>

* ✨ feat: Revised intro

* ✨ feat: Internet

* ✨ feat: Map of ARPANET

* ✨ feat: Unix

* ✨ feat: fin

* 🥢 nit: intro

* ✨ feat: Documentary GNU/Linux, Crypto Anarchy

---------

Co-authored-by: rahul <[email protected]>
Co-authored-by: Mário Havel <[email protected]>

* Add files via upload (#302)

* Create content for scourge under the staking economics track (#300)

* change title lowercase to uppercase

* create content for scourge under the staking economics track

* remove extra space in reference

* add words to wordlist

* reordered the numbering and removed unsed content

* MEV-track and staking economics track in the same table

removed the added tablea header: MEV-track and staking economics track can be in the same table

* reordered the staking economics table content

---------

Co-authored-by: DanGoron <[email protected]>

* Design rationale for DHT (#304)

* design rationale for DHT

* Apply suggestions from code review

Co-authored-by: Mário Havel <[email protected]>

---------

Co-authored-by: Mário Havel <[email protected]>

* Meeting #5 notes (#307)

* ✨ feat: Meeting #2 notes

* 🥢 nit: link

* 🥢 nit:

* update notes 2

* Notes for wiki contributors meeting #3

* Formatting

* Spell check

* ✨ feat: Meeting 4 notes by Rory

* ✨ feat: Meeting notes 5, closes #299

---------

Co-authored-by: rahul <[email protected]>
Co-authored-by: Mário Havel <[email protected]>
Co-authored-by: Siddharth Vaderaa <[email protected]>

* Consensus Layer Wiki Page (#246)

* CL init, Update overview of CL

* Update ordering

* Add validators section to overview

* add iamges; added Beacon chain explainer; checkpoints and finality; slots and epochs

* Added validator life cylce

* fix typo; update wordlist

* Update state of validators

* Improve flow of the page;added simpler explanations; added some links

* add introduction; minor fixes

* Minor typos ffix

* add introduction; added byzantine generals problem

* revamp overview structure

* fix dark background in svg

* complete overview of CL; added cl-architecture structure

* Added Blocktree and fork-choice rules

* fix some typos; update wordlist

* add reorgs and reversion

* Add liveness and safey comparision

* Add some more details on consensus protocol

* Add architecture and blobs

* stf; control flow

* fix a broken link; added gasper file

* Use consistent naming for PoW and PoS

* Complete cl-architecture

* Update structure of cl-networking

* fix typos; added words to wordlist

* remove whitespace

* address some nits

* Omit some redudant content

* remove redundant content; fix broken links

* Update proposer and validator set wording

Co-authored-by: Mário Havel <[email protected]>

* Address some more nits

* Add resources; omit whitespace

* fix typos

* Update wordlist

* nit: grammar

Co-authored-by: rahul <[email protected]>

* nit: space

Co-authored-by: rahul <[email protected]>

* nit: word

Co-authored-by: rahul <[email protected]>

* nit: word

Co-authored-by: rahul <[email protected]>

* clean up

Co-authored-by: rahul <[email protected]>

* nit: spell

Co-authored-by: rahul <[email protected]>

* nit: content captilization

* Update wordlist

* nit: diagram name

Co-authored-by: rahul <[email protected]>

* nit: word

Co-authored-by: rahul <[email protected]>

* nit: clean up

Co-authored-by: rahul <[email protected]>

* nit: clean up

Co-authored-by: rahul <[email protected]>

* nit: title

Co-authored-by: rahul <[email protected]>

* nit: wording

Co-authored-by: rahul <[email protected]>

* nit: title

Co-authored-by: rahul <[email protected]>

* fix flow for the transition

* Update beacon-api.md

---------

Co-authored-by: Mário Havel <[email protected]>
Co-authored-by: rahul <[email protected]>

* add links to mirrors (#308)

closes #297

* Update wordlist.txt

* Modified EVM Resources section (#311)

* fixed type

* enhanced EVM wiki

* enhanced EVM wiki

* fixed titles

* fixed titles

---------

Co-authored-by: rahul <[email protected]>
Co-authored-by: rahul <[email protected]>
Co-authored-by: Rory Arredondo <[email protected]>
Co-authored-by: Glory Agatevure <[email protected]>
Co-authored-by: DanGoron <[email protected]>
Co-authored-by: Chirag Parmar <[email protected]>
Co-authored-by: Siddharth Vaderaa <[email protected]>
Co-authored-by: kira <[email protected]>
Co-authored-by: Zaryab <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants