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

Wait for first applied index on Log.Join(). #2199

Merged
merged 1 commit into from
Apr 9, 2015
Merged

Wait for first applied index on Log.Join(). #2199

merged 1 commit into from
Apr 9, 2015

Conversation

benbjohnson
Copy link
Contributor

Overview

Previously the join method returned immediately after connecting. Now the join method will wait until the snapshot and/or log entry is replicated as well before returning.

Previously the join method returned immediately after connecting.
Now the join method will wait until the snapshot and/or log entry
is replicated as well before returning.
@otoolep
Copy link
Contributor

otoolep commented Apr 8, 2015

I understand what this does. What does it help?

@otoolep
Copy link
Contributor

otoolep commented Apr 8, 2015

Just makes sure that when Join returns, the node absolutely is part of the cluster?

@jwilder
Copy link
Contributor

jwilder commented Apr 8, 2015

@otoolep Yes. Should help to avoid some races when spinning up nodes in the tests. 👍

@otoolep
Copy link
Contributor

otoolep commented Apr 8, 2015

Great change.

toddboom added a commit that referenced this pull request Apr 9, 2015
Wait for first applied index on Log.Join().
@toddboom toddboom merged commit 2adafea into master Apr 9, 2015
@toddboom toddboom deleted the broker-wait branch May 4, 2015 21:58
mark-rushakoff pushed a commit that referenced this pull request Jan 11, 2019
Update range function example constant to include now() instead of now
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.

4 participants