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

["Dedunu Dhananjaya (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16677619#comment-16677619 ] 

Dedunu Dhananjaya edited comment on HADOOP-15876 at 11/7/18 3:46 AM:
---------------------------------------------------------------------

[~DanielZhou] I created a patch file but I can't attach it. It doesn't work it seems.


was (Author: dedunumax):
Sure, Will create a patch an attach.

> 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: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org