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

Hudson commented on HDFS-13848:
-------------------------------

SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #14827 (See 
[https://builds.apache.org/job/Hadoop-trunk-Commit/14827/])
HDFS-13848. Refactor NameNode failover proxy providers. Contributed by (shv: 
rev a4121c71c29d13866a605d9c0d013e5de9c147c3)
* (edit) 
hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/server/namenode/ha/AbstractNNFailoverProxyProvider.java
* (edit) 
hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/server/namenode/ha/ConfiguredFailoverProxyProvider.java
* (edit) 
hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/server/namenode/ha/IPFailoverProxyProvider.java
* (edit) 
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/retry/FailoverProxyProvider.java


> Refactor NameNode failover proxy providers
> ------------------------------------------
>
>                 Key: HDFS-13848
>                 URL: https://issues.apache.org/jira/browse/HDFS-13848
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ha, hdfs-client
>    Affects Versions: 2.7.5
>            Reporter: Konstantin Shvachko
>            Assignee: Konstantin Shvachko
>            Priority: Major
>             Fix For: 2.10.0, 3.2.0, 2.9.2, 3.0.4, 3.1.2
>
>         Attachments: HDFS-13848-003.patch, HDFS-13848-004.patch, 
> HDFS-13848-005.patch, HDFS-13848.002.patch, HDFS-13848.patch
>
>
> Looking at NN failover proxy providers in the context of HDFS-13782 I noticed 
> that {{ConfiguredFailoverProxyProvider}} and {{IPFailoverProxyProvider}} have 
> a lot of common logic. We can move this common logic into 
> {{AbstractNNFailoverProxyProvider}}, which simplifies things a lot.



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