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

Konstantin Shvachko commented on HDFS-13782:
--------------------------------------------

Thanks for the review, [~xkrogen].
* I target this issue as a minimal patch to bridge HDFS-13848 and enable IP 
failover. HDFS-13848 will break ORPP once pulled into the branch, so this will 
need to be committed along with the merge. So let's do other improvements in 
subsequent jiras, including HDFS-13779 and HDFS-13780.
* {{ObserverReadProxyProvider}} is not tied to CFPP, rather it uses CFPP by 
default. I did add {{ObserverReadProxyProviderWithIPFailover}}, but I think 
that most people use CFPP, so it seems natural to default ORPP to CFPP for 
failover, rather than creating an extra class.

> ObserverReadProxyProvider should work with IPFailoverProxyProvider
> ------------------------------------------------------------------
>
>                 Key: HDFS-13782
>                 URL: https://issues.apache.org/jira/browse/HDFS-13782
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: test
>            Reporter: Konstantin Shvachko
>            Assignee: Konstantin Shvachko
>            Priority: Major
>         Attachments: HDFS-13782-HDFS-12943.001.patch
>
>
> Currently {{ObserverReadProxyProvider}} is based on 
> {{ConfiguredFailoverProxyProvider}}. We should also be able perform SBN reads 
> in case of {{IPFailoverProxyProvider}}.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to