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-2316: comment does not match code logic #223
Conversation
@@ -724,6 +724,8 @@ synchronized public boolean tryToCommit(Proposal p, long zxid, SocketAddress fol | |||
// concurrent reconfigs are allowed, this can happen. | |||
if (outstandingProposals.containsKey(zxid - 1)) return false; | |||
|
|||
// in order to be committed, a proposal must be accepted by a quorum | |||
// |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This blank line can be removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hanm I think the blank line may be fine, it provides separation from the idea on the next line so it doesn't read as a single sentence.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would just use a period to do the separation.
Fixing ZOOKEEPER-2316. Author: Umesh Panchaksharaiah <upangit@users.noreply.github.com> Author: Umesh Panchaksharaiah <upanchak@yahoo.com> Reviewers: Abraham Fine <afine@apache.org>, Michael Han <hanm@apache.org> Closes #223 from upangit/ZOOKEEPER-2316 and squashes the following commits: 7669f19 [Umesh Panchaksharaiah] ZOOKEEPER-2316: comment does not match code logic 7705e24 [Umesh Panchaksharaiah] ZOOKEEPER-2316: comment does not match code logic (cherry picked from commit f7281f3) Signed-off-by: Michael Han <hanm@apache.org>
Fixing ZOOKEEPER-2316. Author: Umesh Panchaksharaiah <upangit@users.noreply.github.com> Author: Umesh Panchaksharaiah <upanchak@yahoo.com> Reviewers: Abraham Fine <afine@apache.org>, Michael Han <hanm@apache.org> Closes apache#223 from upangit/ZOOKEEPER-2316 and squashes the following commits: 7669f19 [Umesh Panchaksharaiah] ZOOKEEPER-2316: comment does not match code logic 7705e24 [Umesh Panchaksharaiah] ZOOKEEPER-2316: comment does not match code logic
Fixing ZOOKEEPER-2316. Author: Umesh Panchaksharaiah <upangit@users.noreply.github.com> Author: Umesh Panchaksharaiah <upanchak@yahoo.com> Reviewers: Abraham Fine <afine@apache.org>, Michael Han <hanm@apache.org> Closes apache#223 from upangit/ZOOKEEPER-2316 and squashes the following commits: 7669f19 [Umesh Panchaksharaiah] ZOOKEEPER-2316: comment does not match code logic 7705e24 [Umesh Panchaksharaiah] ZOOKEEPER-2316: comment does not match code logic
Fixing ZOOKEEPER-2316.