Naganarasimha G R commented on YARN-2934:

Hi [~jira.shegalov],
bq. Minor repetition is not big of a deal, IMO.
could please have a look at the latest patch if fine then will not rework, else 
will try to update as per your comment

bq.  Right now we are blindly grabbing file 0. It would however make much more 
sense to grab the most recent (highest mtime) non-empty file.
Are you suggesting for this change in this patch or just mentioning that its 
good to have ? If required can rework as it would be better to go for the 
latest modified file !

> 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, YARN-2934.v2.002.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

Reply via email to