[ https://issues.apache.org/jira/browse/HDFS-12976?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16527146#comment-16527146 ]
Chao Sun commented on HDFS-12976: --------------------------------- My bad [~shv]: when merging your change to my local branch I forgot to remove my existing changes on {{ConfiguredFailoverProxyProvider}}. Agree with both you and [~xkrogen] that a separate method in {{ClientProtocol}} may be needed and it should be safe to do so, which I think can be tackled in a separate JIRA? Attaching a clean patch. One question though: why we need to add {{isObserver}} check in the {{StandbyState}} - seems {{context.allowStaleReads}} is enough (all tests passedĀ w/o the {{isObserver}} check). > Introduce ObserverReadProxyProvider > ----------------------------------- > > Key: HDFS-12976 > URL: https://issues.apache.org/jira/browse/HDFS-12976 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: hdfs-client > Reporter: Konstantin Shvachko > Assignee: Chao Sun > Priority: Major > Attachments: HDFS-12976-HDFS-12943.000.patch, > HDFS-12976-HDFS-12943.001.patch, HDFS-12976-HDFS-12943.002.patch, > HDFS-12976-HDFS-12943.003.patch, HDFS-12976-HDFS-12943.004.patch, > HDFS-12976-HDFS-12943.005.patch, HDFS-12976-HDFS-12943.006.patch, > HDFS-12976-HDFS-12943.007.patch, HDFS-12976.WIP.patch > > > {{StandbyReadProxyProvider}} should implement {{FailoverProxyProvider}} > interface and be able to submit read requests to ANN and SBN(s). -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org