[
https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16677429#comment-16677429
]
Da Zhou commented on HADOOP-15876:
----------------------------------
Hi [~dedunumax], thanks for providing this fix.
Instead of creating a PR, could you attach the patch here? Also it will be
great if you can paste the tests results here.
For the cmds you need to run mvn tests, please check
[here|https://issues.apache.org/jira/browse/HADOOP-15740?focusedCommentId=16611070&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16611070].
> 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
> Priority: Minor
>
> 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]
