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

Konstantin Shvachko commented on HDFS-12976:
--------------------------------------------

Here is another variant v006, which actually passes the {{TestObserverNode}}.
[~csun] I don't think the changes in {{ConfiguredFailoverProxyProvider}} you 
restored in v005 are needed anymore. The whole point was to isolate that logic 
inside {{ObserverReadProxyProvider}}.

In my patch I used a very hacky approach to verify whether the node is an 
Observer. See comments in the patch. This is to make it work only. We will need 
to introduce a new ClientProtocol call, which checks or returns the state of 
the service and does not require administration privileges. I think admin 
status is not required for reading the state, similar to SafeMode, which allows 
GET for anybody.

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