[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16921147#comment-16921147 ] Hadoop QA commented on HADOOP-15876: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 0s{color} | {color:blue} Docker mode activated. {color} | | {color:red}-1{color} | {color:red} patch {color} | {color:red} 0m 9s{color} | {color:red} https://github.com/apache/hadoop/pull/434 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. {color} | \\ \\ || Subsystem || Report/Notes || | GITHUB PR | https://github.com/apache/hadoop/pull/434 | | JIRA Issue | HADOOP-15876 | | Console output | https://builds.apache.org/job/hadoop-multibranch/job/PR-434/9/console | | versions | git=2.17.1 | | Powered by | Apache Yetus 0.10.0 http://yetus.apache.org | This message was automatically generated. > 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 > Components: fs/azure >Affects Versions: 3.2.0 >Reporter: Ted Yu >Assignee: Da Zhou >Priority: Minor > Fix For: 3.2.1 > > 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 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 (v8.3.2#803003) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=1691#comment-1691 ] Hadoop QA commented on HADOOP-15876: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 0s{color} | {color:blue} Docker mode activated. {color} | | {color:red}-1{color} | {color:red} patch {color} | {color:red} 0m 10s{color} | {color:red} https://github.com/apache/hadoop/pull/434 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. {color} | \\ \\ || Subsystem || Report/Notes || | GITHUB PR | https://github.com/apache/hadoop/pull/434 | | JIRA Issue | HADOOP-15876 | | Console output | https://builds.apache.org/job/hadoop-multibranch/job/PR-434/8/console | | versions | git=2.17.1 | | Powered by | Apache Yetus 0.10.0 http://yetus.apache.org | This message was automatically generated. > 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 > Components: fs/azure >Affects Versions: 3.2.0 >Reporter: Ted Yu >Assignee: Da Zhou >Priority: Minor > Fix For: 3.2.1 > > 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 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 (v8.3.2#803003) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16913425#comment-16913425 ] Hadoop QA commented on HADOOP-15876: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 0s{color} | {color:blue} Docker mode activated. {color} | | {color:red}-1{color} | {color:red} patch {color} | {color:red} 0m 12s{color} | {color:red} https://github.com/apache/hadoop/pull/434 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. {color} | \\ \\ || Subsystem || Report/Notes || | GITHUB PR | https://github.com/apache/hadoop/pull/434 | | JIRA Issue | HADOOP-15876 | | Console output | https://builds.apache.org/job/hadoop-multibranch/job/PR-434/7/console | | versions | git=2.7.4 | | Powered by | Apache Yetus 0.10.0 http://yetus.apache.org | This message was automatically generated. > 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 > Components: fs/azure >Affects Versions: 3.2.0 >Reporter: Ted Yu >Assignee: Da Zhou >Priority: Minor > Fix For: 3.2.1 > > 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 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 (v8.3.2#803003) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16911500#comment-16911500 ] Hadoop QA commented on HADOOP-15876: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 0s{color} | {color:blue} Docker mode activated. {color} | | {color:red}-1{color} | {color:red} patch {color} | {color:red} 0m 10s{color} | {color:red} https://github.com/apache/hadoop/pull/434 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. {color} | \\ \\ || Subsystem || Report/Notes || | GITHUB PR | https://github.com/apache/hadoop/pull/434 | | JIRA Issue | HADOOP-15876 | | Console output | https://builds.apache.org/job/hadoop-multibranch/job/PR-434/6/console | | versions | git=2.17.1 | | Powered by | Apache Yetus 0.10.0 http://yetus.apache.org | This message was automatically generated. > 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 > Components: fs/azure >Affects Versions: 3.2.0 >Reporter: Ted Yu >Assignee: Da Zhou >Priority: Minor > Fix For: 3.2.1 > > 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 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 (v8.3.2#803003) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16909327#comment-16909327 ] Hadoop QA commented on HADOOP-15876: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 0s{color} | {color:blue} Docker mode activated. {color} | | {color:red}-1{color} | {color:red} patch {color} | {color:red} 0m 12s{color} | {color:red} https://github.com/apache/hadoop/pull/434 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. {color} | \\ \\ || Subsystem || Report/Notes || | GITHUB PR | https://github.com/apache/hadoop/pull/434 | | JIRA Issue | HADOOP-15876 | | Console output | https://builds.apache.org/job/hadoop-multibranch/job/PR-434/5/console | | versions | git=2.7.4 | | Powered by | Apache Yetus 0.10.0 http://yetus.apache.org | This message was automatically generated. > 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 > Components: fs/azure >Affects Versions: 3.2.0 >Reporter: Ted Yu >Assignee: Da Zhou >Priority: Minor > Fix For: 3.2.1 > > 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 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.14#76016) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16902679#comment-16902679 ] Hadoop QA commented on HADOOP-15876: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 0s{color} | {color:blue} Docker mode activated. {color} | | {color:red}-1{color} | {color:red} patch {color} | {color:red} 0m 12s{color} | {color:red} https://github.com/apache/hadoop/pull/434 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. {color} | \\ \\ || Subsystem || Report/Notes || | GITHUB PR | https://github.com/apache/hadoop/pull/434 | | JIRA Issue | HADOOP-15876 | | Console output | https://builds.apache.org/job/hadoop-multibranch/job/PR-434/4/console | | versions | git=2.7.4 | | Powered by | Apache Yetus 0.10.0 http://yetus.apache.org | This message was automatically generated. > 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 > Components: fs/azure >Affects Versions: 3.2.0 >Reporter: Ted Yu >Assignee: Da Zhou >Priority: Minor > Fix For: 3.2.1 > > 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 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.14#76016) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16898506#comment-16898506 ] Hadoop QA commented on HADOOP-15876: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 0s{color} | {color:blue} Docker mode activated. {color} | | {color:red}-1{color} | {color:red} patch {color} | {color:red} 0m 9s{color} | {color:red} https://github.com/apache/hadoop/pull/434 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. {color} | \\ \\ || Subsystem || Report/Notes || | GITHUB PR | https://github.com/apache/hadoop/pull/434 | | JIRA Issue | HADOOP-15876 | | Console output | https://builds.apache.org/job/hadoop-multibranch/job/PR-434/3/console | | versions | git=2.17.1 | | Powered by | Apache Yetus 0.10.0 http://yetus.apache.org | This message was automatically generated. > 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 > Components: fs/azure >Affects Versions: 3.2.0 >Reporter: Ted Yu >Assignee: Da Zhou >Priority: Minor > Fix For: 3.2.1 > > 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 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.14#76016) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16893936#comment-16893936 ] Hadoop QA commented on HADOOP-15876: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 0s{color} | {color:blue} Docker mode activated. {color} | | {color:red}-1{color} | {color:red} patch {color} | {color:red} 0m 12s{color} | {color:red} https://github.com/apache/hadoop/pull/434 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. {color} | \\ \\ || Subsystem || Report/Notes || | GITHUB PR | https://github.com/apache/hadoop/pull/434 | | JIRA Issue | HADOOP-15876 | | Console output | https://builds.apache.org/job/hadoop-multibranch/job/PR-434/2/console | | versions | git=2.7.4 | | Powered by | Apache Yetus 0.10.0 http://yetus.apache.org | This message was automatically generated. > 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 > Components: fs/azure >Affects Versions: 3.2.0 >Reporter: Ted Yu >Assignee: Da Zhou >Priority: Minor > Fix For: 3.2.1 > > 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 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.14#76016) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16888961#comment-16888961 ] Hadoop QA commented on HADOOP-15876: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 0s{color} | {color:blue} Docker mode activated. {color} | | {color:red}-1{color} | {color:red} patch {color} | {color:red} 0m 13s{color} | {color:red} https://github.com/apache/hadoop/pull/434 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. {color} | \\ \\ || Subsystem || Report/Notes || | GITHUB PR | https://github.com/apache/hadoop/pull/434 | | JIRA Issue | HADOOP-15876 | | Console output | https://builds.apache.org/job/hadoop-multibranch/job/PR-434/1/console | | versions | git=2.7.4 | | Powered by | Apache Yetus 0.10.0 http://yetus.apache.org | This message was automatically generated. > 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 > Components: fs/azure >Affects Versions: 3.2.0 >Reporter: Ted Yu >Assignee: Da Zhou >Priority: Minor > Fix For: 3.2.1 > > 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 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.14#76016) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16685808#comment-16685808 ] Hudson commented on HADOOP-15876: - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #15423 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/15423/]) HADOOP-15876. Use keySet().removeAll() to remove multiple keys from Map (stevel: rev a13be203b7877ba56ef63aac4a2e65d4e1a4adbc) * (edit) hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystemStore.java > 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 > Components: fs/azure >Affects Versions: 3.2.0 >Reporter: Ted Yu >Assignee: Da Zhou >Priority: Minor > Fix For: 3.2.1 > > 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 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-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16680373#comment-16680373 ] Hadoop QA commented on HADOOP-15876: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 15s{color} | {color:blue} Docker mode activated. {color} | || || || || {color:brown} Prechecks {color} || | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:red}-1{color} | {color:red} test4tests {color} | {color:red} 0m 0s{color} | {color:red} The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. {color} | || || || || {color:brown} trunk Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 22m 3s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 27s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 20s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 31s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 12m 33s{color} | {color:green} branch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 0m 40s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 21s{color} | {color:green} trunk passed {color} | || || || || {color:brown} Patch Compile Tests {color} || | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 0m 28s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 0m 22s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 0m 22s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 15s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 0m 26s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} shadedclient {color} | {color:green} 13m 25s{color} | {color:green} patch has no errors when building and testing our client artifacts. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 0m 43s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 0m 18s{color} | {color:green} the patch passed {color} | || || || || {color:brown} Other Tests {color} || | {color:green}+1{color} | {color:green} unit {color} | {color:green} 1m 10s{color} | {color:green} hadoop-azure in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 26s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 55m 2s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hadoop:8f97d6f | | JIRA Issue | HADOOP-15876 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12947302/HADOOP-15876-001.patch | | Optional Tests | dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle | | uname | Linux ed83e1129926 4.4.0-134-generic #160~14.04.1-Ubuntu SMP Fri Aug 17 11:07:07 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 31614bc | | maven | version: Apache Maven 3.3.9 | | Default Java | 1.8.0_181 | | findbugs | v3.1.0-RC1 | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/15495/testReport/ | | Max. process+thread count | 330 (vs. ulimit of 1) | | modules | C: hadoop-tools/hadoop-azure U: hadoop-tools/hadoop-azure | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/15495/console | | Powered by | Apache Yetus 0.8.0 http://yetus.apache.org | This message was automatically generated. > Use keySet().removeAll() to remove
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16679073#comment-16679073 ] Da Zhou commented on HADOOP-15876: -- Attaching patch on behalf of [~dedunumax]. > 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 > 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 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-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16679076#comment-16679076 ] Da Zhou commented on HADOOP-15876: -- Some tests failed but they are not related to this patch, the fix is in HADOOP-15872. I will re run the tests once HADOOP-15872 is resolved. > 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 > 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 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-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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 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-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16677632#comment-16677632 ] Dedunu Dhananjaya commented on HADOOP-15876: Attaching the patch file > 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 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-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16677619#comment-16677619 ] Dedunu Dhananjaya commented on HADOOP-15876: 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 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-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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=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 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-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-15876) Use keySet().removeAll() to remove multiple keys from Map in AzureBlobFileSystemStore
[ https://issues.apache.org/jira/browse/HADOOP-15876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16664640#comment-16664640 ] ASF GitHub Bot commented on HADOOP-15876: - GitHub user dedunumax opened a pull request: https://github.com/apache/hadoop/pull/434 HADOOP-15876: Refactoring AzureBlobFileSystemStore.java You can merge this pull request into a Git repository by running: $ git pull https://github.com/dedunumax/hadoop HADOOP-15876 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/hadoop/pull/434.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #434 commit 498e981ec9ce4ab292f3b0118b7af46ab279d749 Author: Dedunu Dhananjaya Date: 2018-10-26T04:25:57Z HADOOP-15876: Refactoring AzureBlobFileSystemStore.java > 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 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-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org