[ https://issues.apache.org/jira/browse/HDFS-10965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15563889#comment-15563889 ]
Mingliang Liu commented on HDFS-10965: -------------------------------------- {code} 384 /* init reused vars */ 385 List<String> outs = null; 386 int ret; 387 388 /** 389 * test normal run 390 */ {code} No reuse found. Make them final. {code} 400 assertEquals( 401 "three lines per Datanode: the 1st line is rack info, 2nd node info," 402 + " 3rd empty line.", 403 12, outs.size()); {code} "There should be three lines per Datanode: the 1st line ...." 12 -> 3 * numDn Otherwise +1 pending on Jenkins. > Add unit test for HDFS command 'dfsadmin -printTopology' > -------------------------------------------------------- > > Key: HDFS-10965 > URL: https://issues.apache.org/jira/browse/HDFS-10965 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: fs, shell, test > Reporter: Xiaobing Zhou > Assignee: Xiaobing Zhou > Attachments: HDFS-10965.000.patch, HDFS-10965.001.patch, > HDFS-10965.002.patch, HDFS-10965.003.patch, HDFS-10965.004.patch > > > DFSAdmin#printTopology should also be tested. This proposes adding it in > TestDFSAdmin. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org