[
https://issues.apache.org/jira/browse/HDFS-5349?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13794547#comment-13794547
]
Colin Patrick McCabe commented on HDFS-5349:
--------------------------------------------
bq. I think this is getting cleaned up in HDFS-5096, but In DatanodeManager,
it'll be nice to make forming the list of blockIds into a method.
Yeah, this is cleaned up in 5096
bq. In BPOfferService, might be more clear to have a separate switch for
BlockIdCommand.
Let's do that in the future to keep the diff down
thanks
> DNA_CACHE and DNA_UNCACHE should be by blockId only
> ----------------------------------------------------
>
> Key: HDFS-5349
> URL: https://issues.apache.org/jira/browse/HDFS-5349
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: datanode, namenode
> Affects Versions: HDFS-4949
> Reporter: Colin Patrick McCabe
> Assignee: Colin Patrick McCabe
> Fix For: HDFS-4949
>
> Attachments: HDFS-5349-caching.001.patch, HDFS-5349-caching.002.patch
>
>
> DNA_CACHE and DNA_UNCACHE should be by blockId only. We don't need length
> and genstamp to know what the NN asked us to cache.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
