[ 
https://issues.apache.org/jira/browse/YARN-2934?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15063197#comment-15063197
 ] 

Naganarasimha G R commented on YARN-2934:
-----------------------------------------

[~jira.shegalov] thanks for the comments,

bq. good to see the patch lose 3kb, most of all there are no more changes to 
the common Configuration class.
:)

bq. Still don't see any value in this, please drop: ??if (listStatus.length > 
1) ....??
I placed it as i understood your comment ??Not sure how fancy we need to be 
with the case where multiple log files qualify for the pattern, but maybe at 
least mention to the user there are more files to look at.?? in this way. But 
felt its better to have a log than showing it in the diagnostic msg , thoughts ?

bq. We can FileUtil.stat2Paths or add a loop here to extract just the last path 
component.
Going by the propability, in most of the cases there will be only one error 
file matching the pattern, in that case  based on your earlier approach which 
you had suggested will repeat the name of the file twice, which i felt 
redundant hence removed it. Message would look something like :
{code}
Container exited with a non-zero exit code 127. Error file(s): [error.log]
Last 4096 bytes of error.log :
/bin/bash: /no/jvm/here/bin/java: No such file or directory
{code}
Please share your thoughts if required can add it  and for other comments i am 
working on it .



> Improve handling of container's stderr 
> ---------------------------------------
>
>                 Key: YARN-2934
>                 URL: https://issues.apache.org/jira/browse/YARN-2934
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Gera Shegalov
>            Assignee: Naganarasimha G R
>            Priority: Critical
>         Attachments: YARN-2934.v1.001.patch, YARN-2934.v1.002.patch, 
> YARN-2934.v1.003.patch, YARN-2934.v1.004.patch, YARN-2934.v1.005.patch, 
> YARN-2934.v1.006.patch, YARN-2934.v1.007.patch, YARN-2934.v1.008.patch, 
> YARN-2934.v2.001.patch
>
>
> Most YARN applications redirect stderr to some file. That's why when 
> container launch fails with {{ExitCodeException}} the message is empty.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to