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

ZOOKEEPER-2788: The define of MAX_CONNECTION_ATTEMPTS in QuorumCnxManager.java seems useless, should it be removed? #266

Closed
wants to merge 1 commit into from

Conversation

afine
Copy link
Contributor

@afine afine commented May 30, 2017

No description provided.

…ager.java seems useless, should it be removed?
@asfgit asfgit closed this in e358a80 May 30, 2017
asfgit pushed a commit that referenced this pull request May 30, 2017
…ager.java.

Author: Abraham Fine <afine@apache.org>

Reviewers: Michael Han <hanm@apache.org>

Closes #266 from afine/ZOOKEEPER-2788

(cherry picked from commit e358a80)
Signed-off-by: Michael Han <hanm@apache.org>
@hanm
Copy link
Contributor

hanm commented May 30, 2017

lgtm. merged to 3.5 and master.
@afine Please open a separate PR targeting branch-3.4. Git is not smart enough to figure out the merge conflicts on branch-3.4.

@afine
Copy link
Contributor Author

afine commented May 31, 2017

@hanm see: #268

lvfangmin pushed a commit to lvfangmin/zookeeper that referenced this pull request Jun 17, 2018
…ager.java.

Author: Abraham Fine <afine@apache.org>

Reviewers: Michael Han <hanm@apache.org>

Closes apache#266 from afine/ZOOKEEPER-2788
RokLenarcic pushed a commit to RokLenarcic/zookeeper that referenced this pull request Sep 3, 2022
…ager.java.

Author: Abraham Fine <afine@apache.org>

Reviewers: Michael Han <hanm@apache.org>

Closes apache#266 from afine/ZOOKEEPER-2788
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants