-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
HDFS-16846. EC: Only EC blocks should be effected by max-streams-hard-limit configuration #5143
Conversation
💔 -1 overall
This message was automatically generated. |
...hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeManager.java
Outdated
Show resolved
Hide resolved
@lfxy Thanks for submitting the PR quickly. The change seems good to me. I left a minor comment. |
@lfxy Thanks for updating the PR. |
@tasanuma Yes, you are right, when decommission DNs, both ec blocks and replication blocks are belong to replication tasks. |
@lfxy I agreed. Let's go with the plan. |
🎊 +1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
5ce5d55
to
6b90a15
Compare
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
6b90a15
to
86140ac
Compare
💔 -1 overall
This message was automatically generated. |
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.
@lfxy Thanks for updating PR. I left some comments, mainly about the variables' names.
...hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeManager.java
Outdated
Show resolved
Hide resolved
...hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeManager.java
Outdated
Show resolved
Hide resolved
...oop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeDescriptor.java
Outdated
Show resolved
Hide resolved
...oop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeDescriptor.java
Outdated
Show resolved
Hide resolved
...oop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeDescriptor.java
Outdated
Show resolved
Hide resolved
...hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeManager.java
Outdated
Show resolved
Hide resolved
...op-hdfs/src/test/java/org/apache/hadoop/hdfs/server/blockmanagement/TestDatanodeManager.java
Outdated
Show resolved
Hide resolved
7ffcb83
to
c63e7d2
Compare
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
c63e7d2
to
4222b09
Compare
💔 -1 overall
This message was automatically generated. |
4222b09
to
6e569b1
Compare
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
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.
@lfxy Thanks for updating PR. The changes in the main code look pretty good to me. I left some comments about the test code.
...op-hdfs/src/test/java/org/apache/hadoop/hdfs/server/blockmanagement/TestDatanodeManager.java
Outdated
Show resolved
Hide resolved
...op-hdfs/src/test/java/org/apache/hadoop/hdfs/server/blockmanagement/TestDatanodeManager.java
Outdated
Show resolved
Hide resolved
...op-hdfs/src/test/java/org/apache/hadoop/hdfs/server/blockmanagement/TestDatanodeManager.java
Show resolved
Hide resolved
...op-hdfs/src/test/java/org/apache/hadoop/hdfs/server/blockmanagement/TestDatanodeManager.java
Show resolved
Hide resolved
💔 -1 overall
This message was automatically generated. |
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 for the update. LGTM.
Merged it. Thanks for your contribution and fixing this issue, @lfxy! |
…-limit configuration (apache#5143) Signed-off-by: Takanobu Asanuma <tasanuma@apache.org>
…y max-streams-hard-limit configuration (apache#5143) Signed-off-by: Takanobu Asanuma <tasanuma@apache.org>
In HDFS-16613, the dfs.namenode.replication.max-streams-hard-limit configuration will only affect decommissioning DataNode, but will not distinguish between replication blocks and EC blocks. Even if DataNodes have only replication files, they will always generate high network traffic. So this configuration should only effect EC blocks.