[
https://issues.apache.org/jira/browse/HDFS-2246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13109042#comment-13109042
]
Suresh Srinivas commented on HDFS-2246:
---------------------------------------
bq. Now BlockMetadaDataHeader becomes a public class. What's the "real"
visibility of that class?
So it can be used outside the package, in DFSClient.
bq. I don;t understand it very well, but why is generation stamp ignored when
reading from local ?
Not sure which code you are talking about here
bq. What is the correct behavior if metadata versions do not match ? (currently
you ignore checksums, but still read the data.)
It is just cut and paste from BlockSender. I think metadata version has not
changed in a while. At both these places, we should make changes with clearly
defined behavior in a separate issue.
Minor nit: soft lru cache is not specific to hdfs, and should be moved to
common.
Sounds good. I was doing that in my new patch.
> Shortcut a local client reads to a Datanodes files directly
> -----------------------------------------------------------
>
> Key: HDFS-2246
> URL: https://issues.apache.org/jira/browse/HDFS-2246
> Project: Hadoop HDFS
> Issue Type: Improvement
> Reporter: Sanjay Radia
> Attachments: 0001-HDFS-347.-Local-reads.patch, HDFS-2246.20s.1.patch,
> HDFS-2246.20s.patch, localReadShortcut20-security.2patch
>
>
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
