tasanuma commented on pull request #3704:
URL: https://github.com/apache/hadoop/pull/3704#issuecomment-988535550


   @symious Sorry for reviewing it very late. I looked through it, and it 
mostly looks good to me.
   
   I have one question. This feature is similar to ECN. The configuration of 
ECN is `dfs.pipeline.ecn`, which is for the DataNode configuration. 
`dfs.client.mark.slownode.as.badnode` seems the client configuration. If it 
works the same way as ECN, it would be better to make it on the DataNode 
configuration. What do you think?


-- 
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]

Reply via email to