-
Notifications
You must be signed in to change notification settings - Fork 583
HDDS-13842. Exit safemode rule quickly at follower for idle SCM #9202
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
Conversation
|
Could you please check this @octachoron? thanks! |
szetszwo
left a comment
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.
@sumitagrawl , thanks for working on this! Just a minor comment inlined.
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/ha/SCMStateMachine.java
Show resolved
Hide resolved
szetszwo
left a comment
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 the change looks good.
|
This revert has not fixed the issue, closing the PR. |
c2768ef to
a171ab6
Compare
|
@szetszwo Updated the PR revert back to original state and remaining changes will be done as separate PR, please have a look |
szetszwo
left a comment
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 the change looks good.
a9deb88 to
9c36cc0
Compare
What changes were proposed in this pull request?
revert Ratis log metadata - ensure the termUpdate event happens for metadata changes also
HDDS-13281. Disable Ratis metadata write to Raft Log on OM & SCM
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-13842
How was this patch tested?