[ https://issues.apache.org/jira/browse/HDFS-2246?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Suresh Srinivas updated HDFS-2246: ---------------------------------- Attachment: HDFS-2246.20s.patch Here is an updated patch with the following changes: # Addressed all Todd's comments # Made variables final where necessary # Changed "dfs.read.shortcircuit." to "dfs.client.read.shortcircuit" and moved it to DFSConfigKeys. # TestFileLocalRead.java #* Renamed the test to TestShortCircuitLocalRead.java #* Added class javadoc that describes the test correctly. #* Added call to checkFullFile() to trigger local reads and to verify local read works. #* Removed unused method checkFile # DFSClient.java #* Changed the local host check to more comprehensive check, which will handle multiple interfaces better. *Please review this part closely.* #* Made changes to unwrap InvalidToken from RemoteException when making datanode#getBlockPathInfo() call. # ClientDataNodeProtocol.java - reverted changing the versionID for newly added method. > 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.patch, > localReadShortcut20-security.2patch > > -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira