Skip to content

ZOOKEEPER-4269: acceptedEpoch.tmp rename failure will cause server startup error#1668

Closed
arshadmohammad wants to merge 1 commit intoapache:branch-3.6from
arshadmohammad:ZOOKEEPER-4269-branch-3.6
Closed

ZOOKEEPER-4269: acceptedEpoch.tmp rename failure will cause server startup error#1668
arshadmohammad wants to merge 1 commit intoapache:branch-3.6from
arshadmohammad:ZOOKEEPER-4269-branch-3.6

Conversation

@arshadmohammad
Copy link
Copy Markdown
Contributor

Using accepted epoch from acceptedEpoch.tmp if it is available

@arshadmohammad arshadmohammad force-pushed the ZOOKEEPER-4269-branch-3.6 branch from aff8ebb to 72b39a3 Compare March 31, 2021 05:57
…artup error

Using accepted epoch from acceptedEpoch.tmp if it is available
Copy link
Copy Markdown
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

we should wait for @ztzg feedback on the related PR for master branch before merging this patch, that is the backport for it

@eolivelli eolivelli requested a review from ztzg March 31, 2021 09:47
Copy link
Copy Markdown
Contributor

@ztzg ztzg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, LGTM. (Still a bit queasy about these .tmp files, though :)

asfgit pushed a commit that referenced this pull request Mar 31, 2021
…artup error

Using accepted epoch from acceptedEpoch.tmp if it is available

Author: Mohammad Arshad <arshad@apache.org>

Reviewers: Enrico Olivelli <eolivelli@apache.org>,Damien Diederen <dd@crosstwine.com>

Closes #1668 from arshadmohammad/ZOOKEEPER-4269-branch-3.6
@arshadmohammad
Copy link
Copy Markdown
Contributor Author

Merged

asfgit pushed a commit that referenced this pull request May 17, 2022
…artup error

Using accepted epoch from acceptedEpoch.tmp if it is available

Author: Mohammad Arshad <arshad@apache.org>

Reviewers: Enrico Olivelli <eolivelli@apache.org>,Damien Diederen <dd@crosstwine.com>

Closes #1668 from arshadmohammad/ZOOKEEPER-4269-branch-3.6

(cherry picked from commit 54e563b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants