[
https://issues.apache.org/jira/browse/HDFS-5349?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13793121#comment-13793121
]
Andrew Wang commented on HDFS-5349:
-----------------------------------
What's here looks good to me, +1. Do you want to also remove genstamp and
length from the {{CacheReportRequest}} proto while you're here? I don't think
it'll cause much merge pain with HDFS-5096, but we can do it later too.
> 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
> Attachments: HDFS-5349-caching.001.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)