[HADOOP-15876] Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore

• Type: Sub-task
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.2.0
• Fix Version/s: 3.2.1
• Component/s: fs/azure
• Assignee: Da Zhou
• Reporter: Ted Yu
• Created: 24/Oct/18 03:11
• Updated: 14/Apr/20 21:45
• Resolved: 13/Nov/18 21:49

Description

Looking at hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystemStore.java , removeDefaultAcl in particular:

    for (Map.Entry<String, String> defaultAclEntry : defaultAclEntries.entrySet()) {
      aclEntries.remove(defaultAclEntry.getKey());
    }

The above operation can be written this way:

    aclEntries.keySet().removeAll(defaultAclEntries.keySet());

Issue Links

• is blocked by: HADOOP-15872 ABFS: Update to target 2018-11-09 REST version for ADLS Gen 2
• links to: GitHub Pull Request #434
• links to: GitHub Pull Request #434