[ 
https://issues.apache.org/jira/browse/MAPREDUCE-2944?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Harsh J updated MAPREDUCE-2944:
-------------------------------

    Attachment: MAPREDUCE-2944.patch

+1 on most recent patch. I just made some superficial changes (uploaded). 
Committing.

Stuff I mainly changed was to remove newlines from exception messages. We 
should never do that as it messes the logs and stuff that would look for 
exceptions and stack traces. One long line of message is fine, lets always 
follow that ahead :)

That + some doc message fixes to look a bit more clear.
                
> Improve checking of input for Api displayTasks() 
> -------------------------------------------------
>
>                 Key: MAPREDUCE-2944
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-2944
>             Project: Hadoop Map/Reduce
>          Issue Type: Improvement
>          Components: client
>    Affects Versions: 0.24.0
>            Reporter: XieXianshan
>            Assignee: XieXianshan
>            Priority: Trivial
>             Fix For: 0.24.0
>
>         Attachments: MAPREDUCE-2944.patch, MAPREDUCE-2944.patch, 
> MAPREDUCE-2944_v0.2.patch, MAPREDUCE-2944_v0.3.patch
>
>
> The JobClient.displayTasks() will do nothing and won't throw any exception 
> either, 
> while user call it with invalid type/state of task.
> To be more friendly,it's better to remain user to check his parameters with 
> an exception. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to