[ 
https://issues.apache.org/jira/browse/HDFS-12976?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16514055#comment-16514055
 ] 

Erik Krogen edited comment on HDFS-12976 at 6/15/18 4:31 PM:
-------------------------------------------------------------

Hey [~csun], I don't see why the changes to 
{{ConfiguredFailoverProxyProvider#performFailover()}} are necessary. The intent 
as I understand it is that you increment the proxy index, then try it, and if 
the proxy is in the wrong state it will throw an exception and 
{{performFailover()}} will be called again. This process should work fine 
without the {{isObserverState}} check.


was (Author: xkrogen):
Hey [~csun], I don't see why the changes to 
{{ConfiguredFailoverProxyProvider#performFailover()}} are necessary. The intent 
is that you increment the proxy index, then try it, and if the proxy is in the 
wrong state it will throw an exception and {{performFailover()}} will be called 
again. This process should work fine without the {{isObserverState}} 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.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

Reply via email to