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

Hudson commented on HDFS-13970:
-------------------------------

SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #15606 (See 
[https://builds.apache.org/job/Hadoop-trunk-Commit/15606/])
HDFS-13970. Use MultiMap for CacheManager Directives to simplify the (aajisaka: 
rev ca379e1c43fd733a34f3ece6172c96d74c890422)
* (edit) 
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/CacheManager.java
* (edit) 
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/protocol/CacheDirective.java


> Use MultiMap for CacheManager Directives to simplify the code
> -------------------------------------------------------------
>
>                 Key: HDFS-13970
>                 URL: https://issues.apache.org/jira/browse/HDFS-13970
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: caching, hdfs
>    Affects Versions: 3.2.0
>            Reporter: BELUGA BEHR
>            Assignee: BELUGA BEHR
>            Priority: Minor
>             Fix For: 3.3.0
>
>         Attachments: HDFS-13970.1.patch, HDFS-13970.2.patch
>
>
> # Use Guava Multimap to simplify code
>  ## Currently, code uses a mix of LinkedList and ArrayList - just pick one
>  ## Currently, {{directivesByPath}} structure is sorted but never used in a 
> sorted way, it only performs remove and add operations, no iteration - use a 
> {{Set}} instead of a {{List}} for values to support faster remove operation.  
> Use a {{HashSet}} instead of a {{TreeSet}} for keys since it doesn't appear 
> that order really matters.
>  # The {{CacheDirective}} class needs a better hashcode implementation since 
> it will be used in a Set.  Do not instantiate a {{HashBuilder}} object every 
> time {{hashcode}} is called. Ouch.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to