Skip to content

[Improvement][Master/Worker]Change Master/Worker default zookeeper timeout setting#14949

Closed
AmriStrong wants to merge 3 commits intoapache:devfrom
AmriStrong:dev_master_zk_timeOut
Closed

[Improvement][Master/Worker]Change Master/Worker default zookeeper timeout setting#14949
AmriStrong wants to merge 3 commits intoapache:devfrom
AmriStrong:dev_master_zk_timeOut

Conversation

@AmriStrong
Copy link
Contributor

@AmriStrong AmriStrong commented Sep 21, 2023

solve to Local running Master project connection remote zookeeper always reported connection timeout。

This problem, for the new people just started, build the project, may take a lot of time to understand, debugging, and then the normal operation

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

…n timeout time use the curator default setting
@AmriStrong AmriStrong changed the title [Improvement][Master]Change Master default zookeeper timeout setting #14946 [Improvement][Master]Change Master default zookeeper timeout setting Sep 21, 2023
@ruanwenjun
Copy link
Member

Please update the master/ worker config and update the doc in this one PR.

@ruanwenjun ruanwenjun added improvement make more easy to user or prompt friendly minor Chore Chores about the project, like code clean, typo, etc. labels Sep 22, 2023
@ruanwenjun ruanwenjun changed the title [Improvement][Master]Change Master default zookeeper timeout setting [Improvement][Master/Worker]Change Master/Worker default zookeeper timeout setting Sep 22, 2023
@AmriStrong
Copy link
Contributor Author

Please update the master/ worker config and update the doc in this one PR.

OK,It has been adjusted and submitted, please review, thank you

@AmriStrong
Copy link
Contributor Author

@ruanwenjun Review and merge code

@AmriStrong AmriStrong closed this Oct 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend improvement make more easy to user or prompt friendly minor Chore Chores about the project, like code clean, typo, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants