[ https://issues.apache.org/jira/browse/HDFS-9484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15034803#comment-15034803 ]
Mingliang Liu commented on HDFS-9484: ------------------------------------- Failing tests are not related and can not re-produce locally. > NNThroughputBenchmark$BlockReportStats should not send empty block reports > -------------------------------------------------------------------------- > > Key: HDFS-9484 > URL: https://issues.apache.org/jira/browse/HDFS-9484 > Project: Hadoop HDFS > Issue Type: Bug > Components: test > Reporter: Mingliang Liu > Assignee: Mingliang Liu > Attachments: HDFS-9484.000.patch > > > There are two potential bugs that make the > {{NNThroughputBenchmark$BlockReportStats}} send empty block reports. > # In {{NNThroughputBenchmark$BlockReportStats#formBlockReport()}}, the > {{blockReportList}} is always {{BlockListAsLongs.EMPTY}}. We should construct > the block report list by encoding generated {{blocks}} in test. > # {{TinyDatanode#blocks}} is an empty ArrayList with initial capacity. In > {{TinyDatanode#addBlock()}} first statement, the {{if(nrBlocks == > blocks.size()) {}} will always be true. We should either fill the blocks with > dummy report in {{TinyDatanode()}} constructor, or use initial capacity > instead of {{blocks.size()}} in the above _if_ statement (we should replace > {{ArrayList#set}} with {{ArrayList#add}} as well). -- This message was sent by Atlassian JIRA (v6.3.4#6332)