[ 
https://issues.apache.org/jira/browse/HDFS-2367?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Aaron T. Myers updated HDFS-2367:
---------------------------------

    Attachment: HDFS-2367-HDFS-1623.patch

bq. Rather than add the new initialize API, can we just add another argument to 
the "conventional constructor" for the interface? ie we already require that 
the implementations have a constructor which takes the interface - let's just 
require they also take the URI? It seems reasonable to me.

Agreed. Done.

bq. Perhaps we should add a test case which sets up a federated cluster, each 
of the nameservices being HA? I think we have the pieces in place to do it, but 
would be good to verify.

Agreed. Can we break that out into a separate JIRA?
                
> HA: Enable the configuration of multiple HA cluster addresses
> -------------------------------------------------------------
>
>                 Key: HDFS-2367
>                 URL: https://issues.apache.org/jira/browse/HDFS-2367
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ha, hdfs client
>    Affects Versions: HA branch (HDFS-1623)
>            Reporter: Aaron T. Myers
>            Assignee: Aaron T. Myers
>             Fix For: HA branch (HDFS-1623)
>
>         Attachments: HDFS-2367-HDFS-1623.patch, HDFS-2367-HDFS-1623.patch
>
>
> Todd pointed out this issue with the current patch posted for HDFS-1623. It 
> seems like the solution would be to replace "dfs.ha.namenode.addresses" with 
> something like "dfs.ha.namenode.addresses.<logical-uri>", so as to be able to 
> support configuring the addresses of multiple HA clusters.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to