-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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-2307:ZooKeeper not starting because acceptedEpoch is less than the currentEpoch #1145
ZOOKEEPER-2307:ZooKeeper not starting because acceptedEpoch is less than the currentEpoch #1145
Conversation
…han the currentEpoch
|
|
||
Assert.assertEquals("acceptedEpoch must not have changed", 0, | ||
quorumPeer.getAcceptedEpoch()); | ||
} |
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.
we also need assert currentEpoch
here?
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.
if acceptedEpoch is 0 currentEpoch will be 0. But we can add assertion.
Added assertion for currentEpoch as well.
Refer to this link for build results (access rights to CI server needed): |
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.
+1 lgtm.
Merged to master. Thanks @arshadmohammad ! |
Hi, which version of zookeeper will have this fix? |
@rohitagl91 According to the jira: 3.6.0 |
Thanks @anmolnar.. and do we have expected release date of 3.6.0? |
RC1 is out. Feel free to vote for it. ;) |
…than the currentEpoch Update acceptedEpoch and currentEpoch in file first then in memory. Author: Mohammad Arshad <arshad@apache.org> Reviewers: andor@apache.org Closes apache#1145 from arshadmohammad/ZOOKEEPER-2307-epochUpdate and squashes the following commits: b05bc1f [Mohammad Arshad] review comment fix c8d620f [Mohammad Arshad] ZOOKEEPER-2307:ZooKeeper not starting because acceptedEpoch is less than the currentEpoch
…than the currentEpoch Update acceptedEpoch and currentEpoch in file first then in memory. Author: Mohammad Arshad <arshad@apache.org> Reviewers: andor@apache.org Closes apache#1145 from arshadmohammad/ZOOKEEPER-2307-epochUpdate and squashes the following commits: b05bc1f [Mohammad Arshad] review comment fix c8d620f [Mohammad Arshad] ZOOKEEPER-2307:ZooKeeper not starting because acceptedEpoch is less than the currentEpoch
…than the currentEpoch Update acceptedEpoch and currentEpoch in file first then in memory. Author: Mohammad Arshad <arshad@apache.org> Reviewers: andor@apache.org Closes apache#1145 from arshadmohammad/ZOOKEEPER-2307-epochUpdate and squashes the following commits: b05bc1f [Mohammad Arshad] review comment fix c8d620f [Mohammad Arshad] ZOOKEEPER-2307:ZooKeeper not starting because acceptedEpoch is less than the currentEpoch (cherry picked from commit 19d8567)
…than the currentEpoch Update acceptedEpoch and currentEpoch in file first then in memory. Author: Mohammad Arshad <arshadapache.org> Reviewers: andorapache.org Closes #1145 from arshadmohammad/ZOOKEEPER-2307-epochUpdate and squashes the following commits: b05bc1f [Mohammad Arshad] review comment fix c8d620f [Mohammad Arshad] ZOOKEEPER-2307:ZooKeeper not starting because acceptedEpoch is less than the currentEpoch (cherry picked from commit 19d8567) Author: Mohammad Arshad <arshad@apache.org> Reviewers: Andor Molnar <andor@apache.org>, Norbert Kalmar <nkalmar@apache.org> Closes #1613 from symat/ZOOKEEPER-4222
I think it wasn't fixed. I am running 3.6.1 got the same error "ERROR [main:QuorumPeer@1136] - Unable to load database on disk Is this resolved in 3.7 ? |
…than the currentEpoch Update acceptedEpoch and currentEpoch in file first then in memory. Author: Mohammad Arshad <arshad@apache.org> Reviewers: andor@apache.org Closes apache#1145 from arshadmohammad/ZOOKEEPER-2307-epochUpdate and squashes the following commits: b05bc1f [Mohammad Arshad] review comment fix c8d620f [Mohammad Arshad] ZOOKEEPER-2307:ZooKeeper not starting because acceptedEpoch is less than the currentEpoch
…than the currentEpoch Update acceptedEpoch and currentEpoch in file first then in memory. Author: Mohammad Arshad <arshad@apache.org> Reviewers: andor@apache.org Closes apache#1145 from arshadmohammad/ZOOKEEPER-2307-epochUpdate and squashes the following commits: b05bc1f [Mohammad Arshad] review comment fix c8d620f [Mohammad Arshad] ZOOKEEPER-2307:ZooKeeper not starting because acceptedEpoch is less than the currentEpoch
Update acceptedEpoch and currentEpoch in file first then in memory.