ssvlabs / ssv

Secret-Shared-Validator(SSV) for ethereum staking
https://ssv.network
GNU General Public License v3.0
179 stars 93 forks source link

Rethink getting duties for all participants in the network for message validation #1660

Open olegshmuelov opened 4 weeks ago

olegshmuelov commented 4 weeks ago

Currently, the duty scheduler contains logic specifically for message validation, such as retrieving all duties for all participants in the network for proposal and sync committee duties. This logic should be separated to keep the duty scheduler focused solely on scheduling tasks.

Proposed Enhancement:

y0sher commented 4 weeks ago

Great Idea!

I understand your need to separate the message validation handling logic from the regular scheduler. But wouldn't that mean we have some duplication in our calls to the beacon node?

What do you think about splitting the logic of the scheduler to fetcher and executor. then both the duty task executor and the message validator executor could consume the same data previously fetched by the fetcher? Actually it's pretty similar to how it works now with the attester/sync committee and committee consensus handler.

Unless you think the extra calls to bn and memory management is negligible..