Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-15763 Über-JIRA: abfs phase II: Hadoop 3.3 features & fixes
  3. HADOOP-15876

Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore

VotersStop watchingWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 3.2.0
    • 3.2.1
    • fs/azure
    • None

    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());
      

      Attachments

        1. HADOOP-15876-001.patch
          0.9 kB
          Da Zhou

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            DanielZhou Da Zhou
            yuzhihong@gmail.com Ted Yu
            Votes:
            0 Vote for this issue
            Watchers:
            8 Stop watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment