-
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-16314. Support to make dfs.namenode.block-placement-policy.exclude-slow-nodes.enabled reconfigurable #3651
Conversation
…de-slow-nodes.enabled reconfigurable
🎊 +1 overall
This message was automatically generated. |
} else { | ||
enable = Boolean.parseBoolean(newVal); | ||
throw new IllegalArgumentException("Unexpected property " + | ||
property + "in reconfReplicationParameters"); |
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.
There is a space missing and the method name needs to be changed to reconfigureSlowNodesParameters
. BTW, please add a space in line 2237
. Thanks.
The other changes look good to me.
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.
@tomscut Thank you for your reply.
Fixed and updated PR.
Sorry, code branch HDFS-16314 was deleted by mistake, resubmit a PR #3664 |
💔 -1 overall
This message was automatically generated. |
Description of PR
Consider that make dfs.namenode.block-placement-policy.exclude-slow-nodes.enabled reconfigurable and rapid rollback in case this feature HDFS-16076 unexpected things happen in production environment
Details: HDFS-16314