[HDFS-6627] Rename DataNode#checkWriteAccess to checkReadAccess.

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.5.0, 3.0.0-alpha1
• Fix Version/s: 2.6.0
• Component/s: datanode
• Target Version/s: 2.6.0
• Hadoop Flags: Reviewed
• Assignee: Liang Xie
• Reporter: Liang Xie
• Created: 04/Jul/14 04:00
• Updated: 12/May/16 18:15
• Resolved: 08/Jul/14 20:31

Description

Just read getReplicaVisibleLength() code and found it, DataNode.checkWriteAccess is only invoked by DataNode.getReplicaVisibleLength(), let's rename it to checkReadAccess to avoid confusing, since the real impl here is check AccessMode.READ:

        blockPoolTokenSecretManager.checkAccess(id, null, block,
            BlockTokenSecretManager.AccessMode.READ);

Issue Links

• relates to: HDFS-1813 Hdfs Federation: Authentication using BlockToken in RPC to datanode fails.