[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

2018-04-24 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16450814#comment-16450814 ] Hudson commented on YARN-6246: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #14057 (See

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

2017-05-31 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16032202#comment-16032202 ] Hudson commented on YARN-6246: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #11807 (See

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

2017-05-31 Thread Daniel Templeton (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16031478#comment-16031478 ] Daniel Templeton commented on YARN-6246: I agree. Seems like a good choice to me. > Identifying

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

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

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

2017-05-25 Thread Daniel Templeton (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16025578#comment-16025578 ] Daniel Templeton commented on YARN-6246: LGTM. Fix your javadoc error and the checkstyle issue,

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

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

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

2017-05-24 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16023954#comment-16023954 ] Karthik Kambatla commented on YARN-6246: Updated patch uses an AtomicLong for

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

2017-05-11 Thread Daniel Templeton (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16006676#comment-16006676 ] Daniel Templeton commented on YARN-6246: Thanks for taking a closer look. I agree that the impact

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

2017-05-10 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16005727#comment-16005727 ] Karthik Kambatla commented on YARN-6246: Carefully looked through the locking. dumpStateInternal

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

2017-03-29 Thread Daniel Templeton (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15947815#comment-15947815 ] Daniel Templeton commented on YARN-6246: Thanks, [~kasha]. Looking at the patch, I'm concerned

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

2017-03-28 Thread Miklos Szegedi (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15946416#comment-15946416 ] Miklos Szegedi commented on YARN-6246: -- +1 (Non-binding) Thank you [~kasha] for reporting the issue

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

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

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

2017-03-23 Thread Miklos Szegedi (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15939497#comment-15939497 ] Miklos Szegedi commented on YARN-6246: -- Thank you for the patch [~kasha]. shouldAttemptPreemption()

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

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

[jira] [Commented] (YARN-6246) Identifying starved apps does not need the scheduler writelock

2017-03-09 Thread Tao Jie (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15904237#comment-15904237 ] Tao Jie commented on YARN-6246: --- Thank you [~kasha] for working on this. It seems to me that checking