[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16972581#comment-16972581 ] Manikandan R commented on YARN-9865: Thanks [~snemeth] and [~pbacsko]. Created YARN-9969  and 

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971491#comment-16971491 ] Hudson commented on YARN-9865: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #17627 (See

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16971482#comment-16971482 ] Szilard Nemeth commented on YARN-9865: -- Hi [~maniraj...@gmail.com]! Patch looks good, +1, committed

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16969938#comment-16969938 ] Szilard Nemeth commented on YARN-9865: -- Hi [~maniraj...@gmail.com]! Can you ask someone else please? 

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16969366#comment-16969366 ] Manikandan R commented on YARN-9865: [~snemeth] Can you check v5 patch and commit this? > Capacity

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16963133#comment-16963133 ] Hadoop QA commented on YARN-9865: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16963002#comment-16963002 ] Peter Bacsko commented on YARN-9865: Patch v4 looks good. I made a small refactor regarding the

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16962335#comment-16962335 ] Hadoop QA commented on YARN-9865: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16962185#comment-16962185 ] Manikandan R commented on YARN-9865: Makes sense. Attached .004.patch. > Capacity scheduler: add

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16962111#comment-16962111 ] Peter Bacsko commented on YARN-9865: Thanks for the update. I just realized two things: 1) The two

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16961438#comment-16961438 ] Hadoop QA commented on YARN-9865: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16961250#comment-16961250 ] Manikandan R commented on YARN-9865: Thanks [~pbacsko] for review. Attached .003.patch. > Capacity

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16959662#comment-16959662 ] Peter Bacsko commented on YARN-9865: [~maniraj...@gmail.com] I agree, this is negligible. Let's skip

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16956235#comment-16956235 ] Manikandan R commented on YARN-9865: Tried to fix the checksytle issue by removing the space after {,

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16956096#comment-16956096 ] Peter Bacsko commented on YARN-9865: [~maniraj...@gmail.com] could you please fix the last checkstyle

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16954540#comment-16954540 ] Hadoop QA commented on YARN-9865: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16953944#comment-16953944 ] Peter Bacsko commented on YARN-9865: [~maniraj...@gmail.com] I'll ask [~snemeth] to review YARN-9841

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16953010#comment-16953010 ] Manikandan R commented on YARN-9865: Dependency link has been fixed. It requires YARN-9841. Can you

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16952999#comment-16952999 ] Hadoop QA commented on YARN-9865: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16952990#comment-16952990 ] Manikandan R commented on YARN-9865: Attached .002.patch. > Capacity scheduler: add support for

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16948576#comment-16948576 ] Szilard Nemeth commented on YARN-9865: -- Hi [~maniraj...@gmail.com]! Patch001 seems to does not apply

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16940944#comment-16940944 ] Hadoop QA commented on YARN-9865: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-9865) Capacity scheduler: add support for combined %user + %secondary_group mapping

[ https://issues.apache.org/jira/browse/YARN-9865?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16940245#comment-16940245 ] Manikandan R commented on YARN-9865: Attached .001.patch. > Capacity scheduler: add support for