Navigation Menu

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

When sending shard start/failed message due to a cluster state change, use the master indicated in the new state rather than current #6189

Closed
wants to merge 1 commit into from

Conversation

bleskes
Copy link
Contributor

@bleskes bleskes commented May 15, 2014

This commit also adds extra protection in other cases against a master node being de-elected and thus being null.

…, use the master indicated in the new state rather than current

This commit also adds extra protection in other cases against a master node being de-elected and thus being null.
@s1monw
Copy link
Contributor

s1monw commented May 15, 2014

LGTM

@bleskes bleskes closed this in 1f28cd0 May 15, 2014
bleskes added a commit that referenced this pull request May 15, 2014
…, use the master indicated in the new state rather than current

This commit also adds extra protection in other cases against a master node being de-elected and thus being null.

Closes #6189
@bleskes bleskes deleted the bug/shard_started_no_master branch May 15, 2014 16:49
@bleskes bleskes removed the review label May 15, 2014
@s1monw s1monw removed the review label May 15, 2014
@clintongormley clintongormley added :Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. and removed :Cluster labels Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed/Distributed A catch all label for anything in the Distributed Area. If you aren't sure, use this one. >enhancement v1.2.0 v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants