Skip to content

Conversation

@DaveCTurner
Copy link
Contributor

There is no point in timing out a join attempt any more. Timing out and
retrying with the same master is pointless, and an in-flight join
attempt to one master no longer blocks attempts to join other masters.
This commit removes this unnecessary setting.

Relates #60872 in which this setting was deprecated.

There is no point in timing out a join attempt any more. Timing out and retrying with the same master is pointless, and an in-flight join attempt to one master no longer blocks attempts to join other masters. This commit removes this unnecessary setting. Relates elastic#60872 in which this setting was deprecated.
@DaveCTurner DaveCTurner added >enhancement >breaking :Distributed Coordination/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. v8.0.0 labels Aug 10, 2020
@DaveCTurner DaveCTurner requested a review from ywelsch August 10, 2020 08:34
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (:Distributed/Cluster Coordination)

@elasticmachine elasticmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label Aug 10, 2020
Copy link
Contributor

@ywelsch ywelsch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

DaveCTurner added a commit that referenced this pull request Aug 10, 2020
There is no point in timing out a join attempt any more once a cluster is entirely in 7.x. Timing out and retrying with the same master is pointless, and an in-flight join attempt to one master no longer blocks attempts to join other masters. This commit deprecates this unnecessary setting and removes its effect from the joining process. Relates #60873 which removes this setting in master.
@DaveCTurner DaveCTurner merged commit 19eb922 into elastic:master Aug 10, 2020
@DaveCTurner DaveCTurner deleted the 2020-08-10-remove-join-timeout branch August 10, 2020 12:57
masseyke added a commit that referenced this pull request Sep 2, 2021
The cluster.join.timeout setting has been removed in 8.0. This commit adds a deprecation info check for that setting. Relates #42404 #60873
jrodewig added a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872. However, we didn't add a related item to the 7.10 deprecation docs. This adds the missing item. Relates to #60873.
elasticsearchmachine pushed a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872. However, we didn't add a related item to the 7.10 deprecation docs. This adds the missing item. Relates to #60873.
elasticsearchmachine pushed a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872. However, we didn't add a related item to the 7.10 deprecation docs. This adds the missing item. Relates to #60873.
elasticsearchmachine pushed a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872. However, we didn't add a related item to the 7.10 deprecation docs. This adds the missing item. Relates to #60873.
jrodewig added a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872. However, we didn't add a related item to the 7.10 deprecation docs. This adds the missing item. Relates to #60873.
jrodewig added a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872. However, we didn't add a related item to the 7.10 deprecation docs. This adds the missing item. Relates to #60873.
jrodewig added a commit that referenced this pull request Sep 16, 2021
We deprecated the `cluster.join.timeout` setting in 7.10 with PR #60872. However, we didn't add a related item to the 7.10 deprecation docs. This adds the missing item. Relates to #60873.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>breaking :Distributed Coordination/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. >enhancement Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. v8.0.0-alpha1

4 participants