[HDFS-9734] Refactoring of checksum failure report related codes

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha1
• Hadoop Flags: Reviewed
• Assignee: Kai Zheng
• Reporter: Kai Zheng
• Created: 27/Jan/16 08:42
• Updated: 12/May/16 18:15
• Resolved: 25/Feb/16 17:56

Description

This was from discussion with jingzhao in HDFS-9646. There is some duplicate codes between client and datanode sides:

    private void addCorruptedBlock(ExtendedBlock blk, DatanodeInfo node,
        Map<ExtendedBlock, Set<DatanodeInfo>> corruptionMap) {
      Set<DatanodeInfo> dnSet = corruptionMap.get(blk);
      if (dnSet == null) {
        dnSet = new HashSet<>();
        corruptionMap.put(blk, dnSet);
      }
      if (!dnSet.contains(node)) {
        dnSet.add(node);
      }
    }

This would resolve the duplication and also simplify the codes some bit.