[
https://issues.apache.org/jira/browse/HDFS-9022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14791677#comment-14791677
]
Mingliang Liu commented on HDFS-9022:
-------------------------------------
The only failing test {{TestReplaceDatanodeOnFailure}} fails occasionally as a
known bug, see [HDFS-6101]. Timeout tests can not reproduced in my local Mac.
New javac warning in TestMRCredentials.java:
{quote}getUri(InetSocketAddress) in NameNode has been deprecated{quote}
This is expected as we will file a new jira to replace the
{{NameNode.getUri()}} with {{DFSUtilClient.getNNUri()}}. See [comments above |
https://issues.apache.org/jira/browse/HDFS-9022?focusedCommentId=14791104&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14791104]
> Move NameNode.getAddress() and NameNode.getUri() to hadoop-hdfs-client
> ----------------------------------------------------------------------
>
> Key: HDFS-9022
> URL: https://issues.apache.org/jira/browse/HDFS-9022
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: hdfs-client, namenode
> Reporter: Mingliang Liu
> Assignee: Mingliang Liu
> Attachments: HDFS-9022.000.patch, HDFS-9022.001.patch,
> HDFS-9022.002.patch, HDFS-9022.003.patch, HDFS-9022.004.patch
>
>
> The static helper methods in NameNodes are used in {{hdfs-client}} module.
> For example, it's used by the {{DFSClient}} and {{NameNodeProxies}} classes
> which are being moved to {{hadoop-hdfs-client}} module. Meanwhile, we should
> keep the {{NameNode}} class itself in the {{hadoop-hdfs}} module.
> This jira tracks the effort of moving the following static helper methods out
> of {{NameNode}} and thus {{hadoop-hdfs}} module. A good place to put these
> methods is the {{DFSUtilClient}} class:
> {code}
> public static InetSocketAddress getAddress(String address);
> public static InetSocketAddress getAddress(Configuration conf);
> public static InetSocketAddress getAddress(URI filesystemURI);
> public static URI getUri(InetSocketAddress namenode);
> {code}
> Be cautious not to bring new checkstyle warnings.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
