hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-9484) NNThroughputBenchmark$BlockReportStats should not send empty block reports
Date Thu, 03 Dec 2015 23:27:11 GMT

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

Hudson commented on HDFS-9484:
------------------------------

FAILURE: Integrated in Hadoop-trunk-Commit #8916 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/8916/])
HDFS-9484. NNThroughputBenchmark$BlockReportStats should not send empty (shv: rev 924a33d02d82adef11a56bf97ff4100c567d9e5d)
* hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/NNThroughputBenchmark.java
* hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt


> 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
>             Fix For: 2.8.0
>
>         Attachments: HDFS-9484.000.patch, HDFS-9484.001.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)

Mime
View raw message