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

Removed link to misleading information about Thanos #68

Closed
wants to merge 1 commit into from

Conversation

bwplotka
Copy link

@bwplotka bwplotka commented Jun 17, 2019

Hi 👋

While this item was true for like first 4 months of Thanos existence (until ~Feb 2018 when we added other means for discovery), currently Thanos project totally removed gossip protocol in last release and disabled couple months ago, so that information is no longer true. (:

Cheers!

Hi 👋 

While this item was true for like first 5 months of Thanso existence (until Feb 2018), currently Thanos project totally removed gossip protocol in last release and disabled couple months ago, so that information is no longer true. (:
valyala added a commit that referenced this pull request Jun 17, 2019
… of a link to document about Gossip protocol removal from Thanos

Thanos removed non-working gossip protocol a few months ago - thanos-io/thanos#734 ,
so the link to the design document https://github.com/improbable-eng/thanos/blob/master/docs/proposals/approved/201809_gossip-removal.md
became unavailable. So use a link to Wikipedia article instead.

Closes #68
@valyala
Copy link
Collaborator

valyala commented Jun 17, 2019

@bwplotka , thanks for the pull request! I left the bullet line about fragile gossip protocol and used the link to Wikipedia instead. See 0241707 .

@valyala valyala closed this Jun 17, 2019
@bwplotka bwplotka deleted the patch-1 branch June 18, 2019 09:14
@bwplotka
Copy link
Author

Thanks! Quick question what's the difference between master and cluster branch? Which one is the main one?

master...cluster

@valyala
Copy link
Collaborator

valyala commented Jun 18, 2019

The master branch contains code for single-node version, while cluster branch contains code for cluster version. These branches share ~90% of code. There are plans for merging these branches into a single branch. We didn't it yet, since this may complicate contributions from users of a single-node version, while the majority of VictoriaMetrics users stick to single-node version due to its' simplicity and performance.

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

Successfully merging this pull request may close these issues.

2 participants