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
HDDS-6894. EC: ReplicationManager - Add class to handle under-replication and form a command for a datanode #3542
Conversation
...s/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ECContainerReplicaCount.java
Outdated
Show resolved
Hide resolved
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.
thanks @umamaheswararao for the patch. i have few comments inline
...m/src/main/java/org/apache/hadoop/hdds/scm/container/replication/ECContainerHealthCheck.java
Outdated
Show resolved
Hide resolved
.../java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationCommandCreator.java
Outdated
Show resolved
Hide resolved
.../java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationCommandCreator.java
Outdated
Show resolved
Hide resolved
.../java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationCommandCreator.java
Outdated
Show resolved
Hide resolved
.../java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationCommandCreator.java
Outdated
Show resolved
Hide resolved
...m/src/main/java/org/apache/hadoop/hdds/scm/container/replication/ECContainerHealthCheck.java
Outdated
Show resolved
Hide resolved
.../java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationCommandCreator.java
Outdated
Show resolved
Hide resolved
.../java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationCommandCreator.java
Outdated
Show resolved
Hide resolved
.../java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationCommandCreator.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ReplicationCommandCreator.java
Outdated
Show resolved
Hide resolved
...m/src/test/java/org/apache/hadoop/hdds/scm/container/states/TestECContainerReplicaCount.java
Outdated
Show resolved
Hide resolved
…tion and form a command for a datanode
7cc7aed
to
b4d4ad5
Compare
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.
thanks @umamaheswararao for updating this patch . i have a few comment inline
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
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.
Thanks @umamaheswararao for working on this.
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...src/test/java/org/apache/hadoop/hdds/scm/container/states/TestECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...src/test/java/org/apache/hadoop/hdds/scm/container/states/TestECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
.../src/main/java/org/apache/hadoop/hdds/scm/container/replication/UnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...src/test/java/org/apache/hadoop/hdds/scm/container/states/TestECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
.../src/main/java/org/apache/hadoop/hdds/scm/container/replication/UnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
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.
Thanks @umamaheswararao for addressing my comments. LGTM, pending CI.
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...s/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ECContainerReplicaCount.java
Outdated
Show resolved
Hide resolved
…. Also used decommissiononlyIndexes instead of dueToDecom check
…ded. We should let it go and check decom command as well. They are not exclusive checks now.
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.
thanks @umamaheswararao for updating. i have few comment inline, PTAL!
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Outdated
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Show resolved
Hide resolved
...rc/main/java/org/apache/hadoop/hdds/scm/container/replication/ECUnderReplicationHandler.java
Show resolved
Hide resolved
LGTM +1! pending clean CI |
…rate API. Which should cover LegacyRM as well.
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.
LGTM
Thanks @sodonnel @adoroszlai @JacksonYao287 for the great reviews. |
HDDS-6894. EC: ReplicationManager - Add class to handle under-replication and form a command for a datanode (apache#3542) (cherry picked from commit 6537056) Change-Id: I8153852674fb54e838d71cc5dd4a452611528b2c
What changes were proposed in this pull request?
Added a class to handle the under replication processing and forming the reconstruction command.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6894
How was this patch tested?
Added the tests.