[
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: [email protected]
For additional commands, e-mail: [email protected]
