[GitHub] [hadoop] tomscut commented on pull request #3912: HDFS-13923. Remove no need TODO comment for ObserverReadProxyProvider

[GitBox <gi...@apache.org>]

tomscut commented on pull request #3912:
URL: https://github.com/apache/hadoop/pull/3912#issuecomment-1019787146


   > Based on discussion in [HDFS-13923](https://issues.apache.org/jira/browse/HDFS-13923), we don't think need to Add a configuration to turn on/off observer reads.
   > 
   > So I suggest removing the `TODO comment` that are not needed.
   
   Hi @ayushtkn , I am sorry that I did not explain the reason in the PR description.
   
   Based on discussion in [HDFS-13923](https://issues.apache.org/jira/browse/HDFS-13923), we don't think need to Add a configuration to turn on/off observer reads. So I suggest removing the `TODO comment` that are not needed. 
   
   I don't know if it is appropriate to change the title of the original ISSUE, this PR is just a suggestion.
   
   
   


-- 
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: common-issues-unsubscribe@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org