[ 
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)

Reply via email to