Skip to content

Wait for connect on remote settings update #48497

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

Merged
merged 1 commit into from
Oct 25, 2019

Conversation

Tim-Brooks
Copy link
Contributor

This is related to #47718. It introduces a 10 seconds wait for a
connection to complete when remote clsuter settings introduce a new
remote cluster connection.

This is related to elastic#47718. It introduces a 10 seconds wait for a
connection to complete when remote clsuter settings introduce a new
remote cluster connection.
@Tim-Brooks Tim-Brooks added >test Issues or PRs that are addressing/adding tests :Distributed Coordination/Network Http and internode communication implementations v7.5.0 v7.4.2 labels Oct 24, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (:Distributed/Network)

@Tim-Brooks Tim-Brooks requested a review from ywelsch October 24, 2019 23:55
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

@Tim-Brooks Tim-Brooks merged commit c0ecdf8 into elastic:7.5 Oct 25, 2019
Tim-Brooks added a commit that referenced this pull request Oct 25, 2019
This is related to #47718. It introduces a 10 seconds wait for a
connection to complete when remote clsuter settings introduce a new
remote cluster connection.
@Tim-Brooks Tim-Brooks deleted the fix_connect_race branch April 30, 2020 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Coordination/Network Http and internode communication implementations >test Issues or PRs that are addressing/adding tests v7.4.2 v7.5.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants