[ https://issues.apache.org/jira/browse/HADOOP-2512?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12555813#action_12555813 ]
rangadi edited comment on HADOOP-2512 at 1/3/08 9:48 PM: -------------------------------------------------------------- > Earlier I did append and here I am doing insert thus putting the exit code > message on the top. In your patch the exit code will be in the end, no? That is inside ' {{if (errMsg.length() == 0) { ..}} ', so {{append()}} and {{insert(0)}} are the same. I am missing something? I am not sure what the policy should be regd errmsg. I would prefer not to modify it at all and let the user get exit code from {{getExitCode()}}. Here, I am just fixing code to align with the intent in HADOOP-2344. was (Author: rangadi): > Earlier I did append and here I am doing insert thus putting the exit code message on the top. In your patch the exit code will be in the end, no? That is inside '{{if (errMsg.length() == 0) { .. }}', so {{append()}} and {{insert(0)}} are the same. I am missing something? I am not sure what the policy should be regd how errmsg. I would prefer not to modify it at all and let the user get exit code from {{getExitCode()}}. Here, I am just fixing code to align with the intent in HADOOP-2344. > error stream handling in Shell executor > ---------------------------------------- > > Key: HADOOP-2512 > URL: https://issues.apache.org/jira/browse/HADOOP-2512 > Project: Hadoop > Issue Type: Bug > Components: util > Affects Versions: 0.16.0 > Reporter: Raghu Angadi > Assignee: Raghu Angadi > Priority: Blocker > Fix For: 0.16.0 > > Attachments: HADOOP-2512.patch > > > Fix a couple of issues while handling error stream in Shell (added in > HADOOP-2344) : > # fix typo in {{System.getProperty("line.seperator")}}, currently it adds > "null" instead of "\n". > # completed is not set to {{true}} when a process exits with an error. > # In normal error case, it reads errMsg (to create IOException) before > waiting for errThread to complete, which results in in consistent error > message. I will attach a patch. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.