[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-05-17 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16014617#comment-16014617 ] Wangda Tan commented on YARN-5949: -- Latest patch LGTM, will commit tomorrow if no opposite opinions.

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

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

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-05-15 Thread Jonathan Hung (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16011463#comment-16011463 ] Jonathan Hung commented on YARN-5949: - Thanks [~leftnoteasy]! Seems reasonable. Attached 005 to address

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-05-13 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16009593#comment-16009593 ] Wangda Tan commented on YARN-5949: -- Thanks [~jhung] for updating the patch, Few comments: 1) Do you think

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

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

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-05-10 Thread Jonathan Hung (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16005358#comment-16005358 ] Jonathan Hung commented on YARN-5949: - 004 fixes

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

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

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-05-09 Thread Jonathan Hung (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16003668#comment-16003668 ] Jonathan Hung commented on YARN-5949: - Attached 003 patch which pushes the client request -> key value

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-05-09 Thread Jonathan Hung (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16003557#comment-16003557 ] Jonathan Hung commented on YARN-5949: - Discussed this offline, will upload a patch addressing scheduler

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-05-02 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15994089#comment-15994089 ] Wangda Tan commented on YARN-5949: -- Thanks [~jhung], bq. This makes sense, should just be able to either

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

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

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-05-01 Thread Jonathan Hung (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15991882#comment-15991882 ] Jonathan Hung commented on YARN-5949: - Attached 002 patch which uses MutableConfScheduler instead of

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-04-27 Thread Jonathan Hung (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15987979#comment-15987979 ] Jonathan Hung commented on YARN-5949: - Thanks for the comments [~leftnoteasy], had a few

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-04-24 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15981951#comment-15981951 ] Wangda Tan commented on YARN-5949: -- Thanks [~jhung] for your patch, overall looks good to me as well, few

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-04-24 Thread Jonathan Hung (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15981639#comment-15981639 ] Jonathan Hung commented on YARN-5949: - Thanks [~xgong] for the review. Right now the default policy is

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-04-21 Thread Xuan Gong (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15978972#comment-15978972 ] Xuan Gong commented on YARN-5949: - Thanks for the patch. [~jhung] Overall looks good. In the same patch,

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-04-20 Thread Jonathan Hung (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15977622#comment-15977622 ] Jonathan Hung commented on YARN-5949: - Rebase branch and resubmit patch. > Add pluggable configuration

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

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

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-04-06 Thread Jonathan Hung (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15960085#comment-15960085 ] Jonathan Hung commented on YARN-5949: - Added 001 patch which creates a policy interface, a default

[jira] [Commented] (YARN-5949) Add pluggable configuration policy interface as a component of MutableCSConfigurationProvider

2017-04-06 Thread Jonathan Hung (JIRA)
[ https://issues.apache.org/jira/browse/YARN-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15960083#comment-15960083 ] Jonathan Hung commented on YARN-5949: - Added 001 patch which creates a policy interface, a default