-
Notifications
You must be signed in to change notification settings - Fork 7
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
Fix pubsub broadcast loops #576
Comments
I still really want to do this but... I'm not sure how critical it is now:
That means this is only an issue when we have a long instance. But in that case, we kind of want to rebroadcast and, if we're worried about exponentially increasing traffic, we should probably just drop messages from old rounds instead. |
It makes sense, given that we have to be able to handle the parings from N participants either way. |
We can also fix this in a future network upgrade without too much difficulty.
|
We're deferring this because it's not a priority. Instead, we're focusing on #583. |
See libp2p/go-libp2p-pubsub#573.
Tasks:
The text was updated successfully, but these errors were encountered: