-
Notifications
You must be signed in to change notification settings - Fork 9.1k
HDFS-16398. Reconfig block report parameters for datanode #3831
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
1b3dd91
to
f7eb594
Compare
🎊 +1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
Hi @tasanuma , I have solved the conflict. If you are free, please help to review this PR. Thanks a lot. |
@tomscut Thanks for the PR. It mostly looks 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.
How about keeping it as before?
DFS_BLOCKREPORT_INITIAL_DELAY_DEFAULT, TimeUnit.SECONDS) * 1000L; | |
DFS_BLOCKREPORT_INITIAL_DELAY_DEFAULT, TimeUnit.SECONDS, TimeUnit.MILLISECONDS); |
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 your review. I fix it.
Thanks @tasanuma for your comments.
|
🎊 +1 overall
This message was automatically generated. |
@tomscut Thanks for the explanation. It makes sense 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.
LGTM.
Thanks for your contribution, @tomscut. |
Thanks @tasanuma for your review and merging it. |
(cherry picked from commit c2ff390) Conflicts: hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/DataNode.java
Cherry-picked it into branch-3.3 with fixing a minor conflict. |
(cherry picked from commit c2ff390) Conflicts: hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/DataNode.java
JIRA: HDFS-16398.
Reconfig block report parameters for datanode. These configurations are included:
dfs.blockreport.intervalMsec
(see HDFS-16331)dfs.blockreport.initialDelay
,dfs.blockreport.split.threshold
.