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

Consider making the gossiper component optional #19

Open
ralexstokes opened this issue Feb 2, 2024 · 0 comments
Open

Consider making the gossiper component optional #19

ralexstokes opened this issue Feb 2, 2024 · 0 comments

Comments

@ralexstokes
Copy link

The gossiper linked is always created, even if config.relays is empty. If this config item is empty, everything still works although it does bring in some additional overhead to try to gossip various data which simply bails early upon finding there is no one to gossip to.

It would be nice if this entire feature is optional (and dependent on config.relays.len() != 0). It will avoid the (minimal) resource overhead in the case where an operator is not using the feature and more importantly make the codebase easier to reason about if an operator does not need the geo-distribution feature.

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

No branches or pull requests

1 participant