[jira] [Commented] (YARN-10199) Simplify UserGroupMappingPlacementRule#getPlacementForUser

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

[jira] [Commented] (YARN-10199) Simplify UserGroupMappingPlacementRule#getPlacementForUser

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

[jira] [Commented] (YARN-10199) Simplify UserGroupMappingPlacementRule#getPlacementForUser

2020-04-29 Thread Andras Gyori (Jira)
[ https://issues.apache.org/jira/browse/YARN-10199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17095223#comment-17095223 ] Andras Gyori commented on YARN-10199: - Uploaded a new revision which is rebased on trunk, containing

[jira] [Commented] (YARN-10199) Simplify UserGroupMappingPlacementRule#getPlacementForUser

2020-04-23 Thread Peter Bacsko (Jira)
[ https://issues.apache.org/jira/browse/YARN-10199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17090622#comment-17090622 ] Peter Bacsko commented on YARN-10199: - [~gandras] in the meantime, YARN-10226 went in because there

[jira] [Commented] (YARN-10199) Simplify UserGroupMappingPlacementRule#getPlacementForUser

2020-03-30 Thread Andras Gyori (Jira)
[ https://issues.apache.org/jira/browse/YARN-10199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17070811#comment-17070811 ] Andras Gyori commented on YARN-10199: - Thank you [~pbacsko] for the feedback. I have applied you

[jira] [Commented] (YARN-10199) Simplify UserGroupMappingPlacementRule#getPlacementForUser

2020-03-27 Thread Peter Bacsko (Jira)
[ https://issues.apache.org/jira/browse/YARN-10199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17068568#comment-17068568 ] Peter Bacsko commented on YARN-10199: - [~gandras] some minor nits: 1. in {{getPlacementForUser()}}:

[jira] [Commented] (YARN-10199) Simplify UserGroupMappingPlacementRule#getPlacementForUser

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

[jira] [Commented] (YARN-10199) Simplify UserGroupMappingPlacementRule#getPlacementForUser

2020-03-26 Thread Andras Gyori (Jira)
[ https://issues.apache.org/jira/browse/YARN-10199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17067612#comment-17067612 ] Andras Gyori commented on YARN-10199: - The new version takes into account the changes that was

[jira] [Commented] (YARN-10199) Simplify UserGroupMappingPlacementRule#getPlacementForUser

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

[jira] [Commented] (YARN-10199) Simplify UserGroupMappingPlacementRule#getPlacementForUser

2020-03-17 Thread Andras Gyori (Jira)
[ https://issues.apache.org/jira/browse/YARN-10199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17061032#comment-17061032 ] Andras Gyori commented on YARN-10199: - Thank you  for the suggestion [~pbacsko]. It is indeed a very

[jira] [Commented] (YARN-10199) Simplify UserGroupMappingPlacementRule#getPlacementForUser

2020-03-17 Thread Peter Bacsko (Jira)
[ https://issues.apache.org/jira/browse/YARN-10199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17060876#comment-17060876 ] Peter Bacsko commented on YARN-10199: - [~gandras] I'm wondering whether it makes sense to create a