tomscut commented on pull request #3831: URL: https://github.com/apache/hadoop/pull/3831#issuecomment-1021321783
> @tomscut Thanks for the PR. It mostly looks good to me. > > * Could you fix the conflict? > * `dfs.blockreport.initialDelay` is only used when initializing DataNode. Do we need to make `dfs.blockreport.initialDelay` reconfiguable? Thanks @tasanuma for your comments. `dfs.blockreport.initialDelay` used in `BPServiceActor#register`. In some cases (the Namenode is stuck), the DN will heartbeat out and become a deadnode, but the process may be alive. When the exception is recovered, the datanode may need to register again, which triggers `BPServiceActor#register`. In this case, making `dfs.blockreport.initialDelay` reconfiguable is a little effect. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
