[ 
https://issues.apache.org/jira/browse/HDFS-6345?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Wang updated HDFS-6345:
------------------------------

    Attachment: hdfs-6345-2.patch

Thanks for reviewing Colin, new patch adds better comments. The if check in 
CacheManager is a bit tricky, since it's used to break out of the loop either 
if the directive was not found (since it should be the first item in the 
tailMap), or if we found it and are on the second item. Made the comment say 
that.

I think the test failure is also a flake, saw it on another JIRA and works okay 
locally.

> DFS.listCacheDirectives() should allow filtering based on cache directive ID
> ----------------------------------------------------------------------------
>
>                 Key: HDFS-6345
>                 URL: https://issues.apache.org/jira/browse/HDFS-6345
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: caching
>    Affects Versions: 2.4.0
>            Reporter: Lenni Kuff
>            Assignee: Andrew Wang
>         Attachments: hdfs-6345-1.patch, hdfs-6345-2.patch
>
>
> DFS.listCacheDirectives() should allow filtering based on cache directive ID. 
> Currently it throws an exception.
> For example:
> {code}
> long directiveId = <some id value>;
> CacheDirectiveInfo filter = new CacheDirectiveInfo.Builder()  
>     .setId(directiveId)
>     .build();
> RemoteIterator<CacheDirectiveEntry> itr = dfs.listCacheDirectives(filter);
> {code}



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to