[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13720913#comment-13720913
]
Junping Du commented on YARN-873:
-
Hi [~bikassaha] and [~jeffgx619], TestNonExistentJob on
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13718819#comment-13718819
]
Bikas Saha commented on YARN-873:
-
Per Vinod's comment above we should make sure other API's
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13718831#comment-13718831
]
Xuan Gong commented on YARN-873:
bq.Per Vinod's comment above we should make sure other
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13718836#comment-13718836
]
Xuan Gong commented on YARN-873:
We can fix them together
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13718839#comment-13718839
]
Bikas Saha commented on YARN-873:
-
Its fine if its a different patch. Please make sure that
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13718867#comment-13718867
]
Bikas Saha commented on YARN-873:
-
+1. Committing to trunk, branch-2 and branch-2.1-beta.
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13714902#comment-13714902
]
Xuan Gong commented on YARN-873:
That works.
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13714910#comment-13714910
]
Hadoop QA commented on YARN-873:
{color:red}-1 overall{color}. Here are the results of
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13714212#comment-13714212
]
Xuan Gong commented on YARN-873:
Currently, exist code = 0: everything is correct.
exist
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13714224#comment-13714224
]
Hitesh Shah commented on YARN-873:
--
I dont think it is that trivial to set -1/-2/-3 ...
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13714236#comment-13714236
]
Bikas Saha commented on YARN-873:
-
Folks, why dont we simply not catch the exception. A
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13712197#comment-13712197
]
Devaraj K commented on YARN-873:
The latest patch overall looks good to me. These are things
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13712507#comment-13712507
]
Hitesh Shah commented on YARN-873:
--
[~xgong] Requiring to parse a string message to
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13712514#comment-13712514
]
Hadoop QA commented on YARN-873:
{color:green}+1 overall{color}. Here are the results of
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13712528#comment-13712528
]
Bikas Saha commented on YARN-873:
-
Having a return statement in a catch/finally block is not
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13712968#comment-13712968
]
Xuan Gong commented on YARN-873:
bq.Having a return statement in a catch/finally block is
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13712985#comment-13712985
]
Xuan Gong commented on YARN-873:
bq.Requiring to parse a string message to determine whether
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13712994#comment-13712994
]
Hitesh Shah commented on YARN-873:
--
[~xgong] When you say different errors, are you
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13711949#comment-13711949
]
Xuan Gong commented on YARN-873:
create the new Exception class called
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13707122#comment-13707122
]
Vinod Kumar Vavilapalli commented on YARN-873:
--
bq. Would be good to make the
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13705345#comment-13705345
]
Xuan Gong commented on YARN-873:
YARNClient.getApplicationReport(unknownAppId) and
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13702987#comment-13702987
]
Zhijie Shen commented on YARN-873:
--
bq. Null report can be unclear as to what happened.
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13703505#comment-13703505
]
Bikas Saha commented on YARN-873:
-
In that case, we can update the javadoc in the protocol
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13702938#comment-13702938
]
Xuan Gong commented on YARN-873:
[~bikassaha] any comments
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13701687#comment-13701687
]
Xuan Gong commented on YARN-873:
Instead of simplying throwing out the YarnException, how
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13700382#comment-13700382
]
Bikas Saha commented on YARN-873:
-
There is probably no concept of an error code in the
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13697513#comment-13697513
]
Xuan Gong commented on YARN-873:
Throwing Exceptions may not be a good option. If we throw
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13693593#comment-13693593
]
Omkar Vinit Joshi commented on YARN-873:
I think the client should get an error code
[
https://issues.apache.org/jira/browse/YARN-873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13692601#comment-13692601
]
Xuan Gong commented on YARN-873:
At commandLine, if we type yarn application -status
29 matches
Mail list logo