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

BELUGA BEHR commented on HDFS-14108:
------------------------------------

Failed tests are unrelated.  They are all related to the 
{{org.apache.hadoop.hdfs.web.TestWebHdfsTimeouts}} test suit and this seems to 
be a known issue.  I've seen it on several different runs now across various 
JIRAs.  Please consider accepting this patch into the project.

> BlockManager Data Structures
> ----------------------------
>
>                 Key: HDFS-14108
>                 URL: https://issues.apache.org/jira/browse/HDFS-14108
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: hdfs
>    Affects Versions: 3.2.0
>            Reporter: BELUGA BEHR
>            Assignee: BELUGA BEHR
>            Priority: Minor
>         Attachments: HDFS-14108.1.patch
>
>
> # Prefer {{ArrayList}} to {{LinkedList}} when simply adding/iterating
> # Prefer {{HashSet}} to {{TreeSet}} when no ordering is required
> # Other performance improvements
> # Check style fixes
> https://stackoverflow.com/questions/322715/when-to-use-linkedlist-over-arraylist-in-java
> {code:java}
>     final Set<Node> excludedNodes = new HashSet<>();
>     for(BlockReconstructionWork rw : reconWork){
>       // Do no bother wasting time clearing out the collection, let GC do 
> that work later
>       excludedNodes.clear();
>       // use {{addAll}} here
>       for (DatanodeDescriptor dn : rw.getContainingNodes()) {
>         excludedNodes.add(dn);
>       }
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to