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

NodesFD: simplify concurrency control to fully rely on a single map #7889

Closed

Conversation

Projects
None yet
2 participants
@bleskes
Copy link
Member

bleskes commented Sep 26, 2014

The node fault detection class is used by the master node to ping the nodes in the cluster and verify they are alive. This PR simplifies the concurrency controls in the class + adds a test for a scenario that surfaced the problem.

bleskes added a commit to bleskes/elasticsearch that referenced this pull request Sep 26, 2014

NodesFD: simplify concurrency control to fully rely on a single map
The node fault detection class is used by the master node to ping the nodes in the cluster and verify they are alive. This PR simplifies the concurrency controls in the class + adds a test for a scenario that surfaced the problem.

Closes elastic#7889

bleskes added a commit that referenced this pull request Sep 26, 2014

NodesFD: simplify concurrency control to fully rely on a single map
The node fault detection class is used by the master node to ping the nodes in the cluster and verify they are alive. This PR simplifies the concurrency controls in the class + adds a test for a scenario that surfaced the problem.

Closes #7889

@bleskes bleskes closed this in 36c3e89 Sep 26, 2014

@bleskes bleskes added the resiliency label Feb 2, 2015

mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015

NodesFD: simplify concurrency control to fully rely on a single map
The node fault detection class is used by the master node to ping the nodes in the cluster and verify they are alive. This PR simplifies the concurrency controls in the class + adds a test for a scenario that surfaced the problem.

Closes elastic#7889
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.