[jira] [Commented] (YARN-7788) Refactor AllocationTagsManager to move management of temp tags to a separate class

2018-01-22 Thread Konstantinos Karanasos (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16335466#comment-16335466 ] Konstantinos Karanasos commented on YARN-7788: -- Committing this shortly to the branch. Thanks

[jira] [Commented] (YARN-7788) Refactor AllocationTagsManager to move management of temp tags to a separate class

2018-01-22 Thread genericqa (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16335321#comment-16335321 ] genericqa commented on YARN-7788: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-7788) Refactor AllocationTagsManager to move management of temp tags to a separate class

2018-01-22 Thread Konstantinos Karanasos (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16335232#comment-16335232 ] Konstantinos Karanasos commented on YARN-7788: -- Sounds good, will commit the patch tonight. >

[jira] [Commented] (YARN-7788) Refactor AllocationTagsManager to move management of temp tags to a separate class

2018-01-22 Thread genericqa (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16335227#comment-16335227 ] genericqa commented on YARN-7788: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-7788) Refactor AllocationTagsManager to move management of temp tags to a separate class

2018-01-22 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16335210#comment-16335210 ] Wangda Tan commented on YARN-7788: -- Thanks [~asuresh] , +1 to move Temp container functionalities out of

[jira] [Commented] (YARN-7788) Refactor AllocationTagsManager to move management of temp tags to a separate class

2018-01-22 Thread Arun Suresh (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16335207#comment-16335207 ] Arun Suresh commented on YARN-7788: --- Updating patch with [~kkaranasos]'s suggestion > Refactor

[jira] [Commented] (YARN-7788) Refactor AllocationTagsManager to move management of temp tags to a separate class

2018-01-22 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16335200#comment-16335200 ] Wangda Tan commented on YARN-7788: -- [~kkaranasos] , please give me a while to review the patch, will post

[jira] [Commented] (YARN-7788) Refactor AllocationTagsManager to move management of temp tags to a separate class

2018-01-22 Thread Konstantinos Karanasos (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16335188#comment-16335188 ] Konstantinos Karanasos commented on YARN-7788: -- Thanks for the patch, [~asuresh]. +1 for

[jira] [Commented] (YARN-7788) Refactor AllocationTagsManager to move management of temp tags to a separate class

2018-01-22 Thread Arun Suresh (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16335071#comment-16335071 ] Arun Suresh commented on YARN-7788: --- Removed the failed testcase - it was supposed to be in the YARN-7783

[jira] [Commented] (YARN-7788) Refactor AllocationTagsManager to move management of temp tags to a separate class

2018-01-22 Thread genericqa (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16335000#comment-16335000 ] genericqa commented on YARN-7788: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-7788) Refactor AllocationTagsManager to move management of temp tags to a separate class

2018-01-22 Thread Arun Suresh (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16334811#comment-16334811 ] Arun Suresh commented on YARN-7788: --- Uplodaing patch with the refactoring > Refactor