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

Make ChunkStateWitness a Tier1 network message #10668

Closed
jancionear opened this issue Feb 27, 2024 · 4 comments
Closed

Make ChunkStateWitness a Tier1 network message #10668

jancionear opened this issue Feb 27, 2024 · 4 comments
Labels
A-stateless-validation Area: stateless validation

Comments

@jancionear
Copy link
Contributor

In stateless validation chunk producer sends out a ChunkStateWItness message to every chunk validator responsible for validating the corresponding chunk. Currently it's a Tier2 RoutedMessage, but I think it would be better to make it a Tier1 message.

AFAIU Tier1 is meant to exchange information between validators in a quick and stable manner. There's a direct Tier1 connection between every pair of validators, so the connections are much more reliable and efficient than routed Tier2 connections.
ChunkStateWitness messages are sent only between chunk validators and chunk producers, and additionally they're big and time-sensitive. I think they'd be a good fit for Tier1.

Having direct connections could also allow us to ban peers which send invalid ChunkStateWItness messages, which was disabled in #10667

cc @saketh-are

@walnut-the-cat
Copy link
Contributor

how much useful this can be at this moment, considering the low participation rate of tier 1?

@jancionear
Copy link
Contributor Author

jancionear commented Mar 13, 2024

how much useful this can be at this moment, considering the low participation rate of tier 1?

Is the participation rate of tier 1 network low? I though that all active validators participate in the tier1 network o-0

@walnut-the-cat
Copy link
Contributor

That's the hope, but I don't think it's happening right now.
cc. @saketh-are

@jancionear
Copy link
Contributor Author

It appears that this has been implemented in #11021

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-stateless-validation Area: stateless validation
Projects
None yet
Development

No branches or pull requests

2 participants