[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-08-26 Thread Rohith Sharma K S (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15438922#comment-15438922 ] Rohith Sharma K S commented on YARN-3940: - +1LGTM > Application moveToQueue should check NodeLabel

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-08-25 Thread Naganarasimha G R (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15438377#comment-15438377 ] Naganarasimha G R commented on YARN-3940: - [~sunilg] & [~rohithsharma], if no other comments on the

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

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

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-08-24 Thread Sunil G (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15436235#comment-15436235 ] Sunil G commented on YARN-3940: --- That makes sense. Thanks [~bibinchundatt] > Application moveToQueue should

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-08-24 Thread Bibin A Chundatt (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15436231#comment-15436231 ] Bibin A Chundatt commented on YARN-3940: [~sunilg] Will add as a comment that all labels in life

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-08-24 Thread Sunil G (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15436200#comment-15436200 ] Sunil G commented on YARN-3940: --- Sorry for pitching in late. I thought of discussing a scenario here. I know

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-08-24 Thread Naganarasimha G R (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15435528#comment-15435528 ] Naganarasimha G R commented on YARN-3940: - Thanks for pointing it out [~rohithsharma], just cross

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-08-24 Thread Rohith Sharma K S (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15435472#comment-15435472 ] Rohith Sharma K S commented on YARN-3940: - Unexpectedly I focused on this JIRA. And got interest to

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-08-24 Thread Naganarasimha G R (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15435390#comment-15435390 ] Naganarasimha G R commented on YARN-3940: - Thanks for the patch [~bibinchundatt], +1 LGTM, Will

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

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

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-08-23 Thread Naganarasimha G R (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15433672#comment-15433672 ] Naganarasimha G R commented on YARN-3940: - Hi [~bibinchundatt], {code} boolean isValidrequest =

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

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

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-01-27 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15120731#comment-15120731 ] Wangda Tan commented on YARN-3940: -- I think you can take a look at

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-01-19 Thread Bibin A Chundatt (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15106437#comment-15106437 ] Bibin A Chundatt commented on YARN-3940: Hi [~leftnoteasy], Could you please review patch attached.

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

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

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

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

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-01-07 Thread Bibin A Chundatt (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15088779#comment-15088779 ] Bibin A Chundatt commented on YARN-3940: Hi [~Naganarasimha] Thank you for review .I have already

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-01-06 Thread Bibin A Chundatt (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15085917#comment-15085917 ] Bibin A Chundatt commented on YARN-3940: Uploading patch after fixing check style issue. Please do

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

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

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2016-01-06 Thread Sunil G (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15085972#comment-15085972 ] Sunil G commented on YARN-3940: --- Thanks [~bibinchundatt] Few comments: 1. {{ LeafQueue dest}} --> {{

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

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

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

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

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2015-12-29 Thread Bibin A Chundatt (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15073930#comment-15073930 ] Bibin A Chundatt commented on YARN-3940: Hi [~leftnoteasy] Sorry for delayed response. But if

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2015-08-10 Thread Bibin A Chundatt (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14680272#comment-14680272 ] Bibin A Chundatt commented on YARN-3940: [~leftnoteasy] Any thoughts??

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2015-08-10 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14680538#comment-14680538 ] Wangda Tan commented on YARN-3940: -- [~bibinchundatt], Rethink about this, I think to make

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2015-07-31 Thread Bibin A Chundatt (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14648939#comment-14648939 ] Bibin A Chundatt commented on YARN-3940: For first case mentioned above Will check

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2015-07-30 Thread Bibin A Chundatt (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14648686#comment-14648686 ] Bibin A Chundatt commented on YARN-3940: Hi [~leftnoteasy] Thank you for review

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2015-07-30 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14647955#comment-14647955 ] Wangda Tan commented on YARN-3940: -- [~bibinchundatt], I took a look at the patch, it

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2015-07-26 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14641914#comment-14641914 ] Hadoop QA commented on YARN-3940: - \\ \\ | (/) *{color:green}+1 overall{color}* | \\ \\ ||

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2015-07-25 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14641769#comment-14641769 ] Hadoop QA commented on YARN-3940: - \\ \\ | (x) *{color:red}-1 overall{color}* | \\ \\ ||

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2015-07-20 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14633847#comment-14633847 ] Wangda Tan commented on YARN-3940: -- I just converted this to sub-task of YARN-2492 for

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2015-07-20 Thread Bibin A Chundatt (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14633870#comment-14633870 ] Bibin A Chundatt commented on YARN-3940: [~leftnoteasy] IMO ,Since the destination

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2015-07-20 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14633876#comment-14633876 ] Wangda Tan commented on YARN-3940: -- [~bibinchundatt], I agree with you. One approach is in

[jira] [Commented] (YARN-3940) Application moveToQueue should check NodeLabel permission

2015-07-20 Thread Bibin A Chundatt (JIRA)
[ https://issues.apache.org/jira/browse/YARN-3940?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14633881#comment-14633881 ] Bibin A Chundatt commented on YARN-3940: [~leftnoteasy] Thnks for comment will