hyperledger / besu

An enterprise-grade Java-based, Apache 2.0 licensed Ethereum client https://wiki.hyperledger.org/display/besu
https://www.hyperledger.org/projects/besu
Apache License 2.0
1.43k stars 757 forks source link

[SPIKE] Design for how to add discv5 support 3 #4087

Open macfarla opened 1 year ago

macfarla commented 1 year ago

Note

Per Meredith, '... was chatting with Adrian Sutton and he mentioned that the teku discv5 implementation actually holds onto all live nodes in memory instead of just relying on the buckets data structure used within the discovery protocol. When peers are streamed from the discovery service, the bucket structure is searched as well as the list of active sessions.'

Timebox - 3 days

macfarla commented 1 year ago

there is a spec discv5.1 using the consensys/discovery library which would be v5.1 https://github.com/ConsenSys/discovery/pull/72

non-fungible-nelson commented 1 year ago

Spoke with the other client teams. It looks like in Geth it is optional for users at this time (the client can run v4 and v5 simultaneously). We are going to have a discussion with them post-Merge to determine when/how we should move all Els to v5. And I would definitely recommend consolidating around the Teku library if it makes sense.

Discord context here on Eth R&D.