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

Colin Patrick McCabe commented on HDFS-5213:
--------------------------------------------

* fix exception translation error in cache directive add code path

* add exception to represent a directive with the given path already existing 
in a cache pool

* since PathBasedCacheEntry is intended to be a private (implementation) class,
return PathBasedCacheDirectiveWithId from all public APIs instead.

* get rid of unnecessary CacheManager#entriesByDirective.
PathBasedCacheDirective no longer needs to be Comparable.

* fix unit test failures

* fix bug where -listPaths didn't work without a -path argument
                
> separate PathBasedCacheEntry and PathBasedCacheDirectiveWithId
> --------------------------------------------------------------
>
>                 Key: HDFS-5213
>                 URL: https://issues.apache.org/jira/browse/HDFS-5213
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: datanode, namenode
>            Reporter: Colin Patrick McCabe
>            Assignee: Colin Patrick McCabe
>
> Since PathBasedCacheEntry is intended to be a private (implementation) class,
> return PathBasedCacheDirectiveWithId from all public APIs instead.  Some 
> other miscellaneous cleanups in the caching RPC stuff.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to