[
https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16679119#comment-16679119
]
Thomas Marquardt commented on HADOOP-15876:
-------------------------------------------
+1, looks good to me.
> Use keySet().removeAll() to remove multiple keys from Map in
> AzureBlobFileSystemStore
> -------------------------------------------------------------------------------------
>
> Key: HADOOP-15876
> URL: https://issues.apache.org/jira/browse/HADOOP-15876
> Project: Hadoop Common
> Issue Type: Sub-task
> Reporter: Ted Yu
> Assignee: Da Zhou
> Priority: Minor
> Attachments: HADOOP-15876-001.patch
>
>
> Looking at
> hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystemStore.java
> , {{removeDefaultAcl}} in particular:
> {code}
> for (Map.Entry<String, String> defaultAclEntry :
> defaultAclEntries.entrySet()) {
> aclEntries.remove(defaultAclEntry.getKey());
> }
> {code}
> The above operation can be written this way:
> {code}
> aclEntries.keySet().removeAll(defaultAclEntries.keySet());
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
