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

Jason Lowe commented on MAPREDUCE-5870:
---------------------------------------

It would be nice to cleanup most of the checkstyle nits, as there are a lot of 
switch statements without default clauses that were added.  I realize 
semantically we're covered since the fall-through case returns some default 
value or throws, but we can trivially remove the warning by having an explicit 
default clause that explicitly breaks or move the fallthrough code into that 
default case.

Other than that latest patch looks good to me.


> Support for passing Job priority through Application Submission Context in 
> Mapreduce Side
> -----------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-5870
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-5870
>             Project: Hadoop Map/Reduce
>          Issue Type: Improvement
>          Components: client
>            Reporter: Sunil G
>            Assignee: Sunil G
>         Attachments: 0001-MAPREDUCE-5870.patch, 0002-MAPREDUCE-5870.patch, 
> 0003-MAPREDUCE-5870.patch, 0004-MAPREDUCE-5870.patch, 
> 0005-MAPREDUCE-5870.patch, 0006-MAPREDUCE-5870.patch, 
> 0007-MAPREDUCE-5870.patch, 0008-MAPREDUCE-5870.patch, 
> 0009-MAPREDUCE-5870.patch, 0010-MAPREDUCE-5870.patch, 
> 0011-MAPREDUCE-5870.patch, Yarn-2002.1.patch
>
>
> Job Priority support in MapReduce.
> 1. Job Priority can be set from client side as below (Configuration and api).
>                       - JobConf.getJobPriority() and 
> JobConf.setJobPriority(JobPriority priority) 
>                       - We can also use configuration 
> "mapreduce.job.priority".
>               Now this Job priority can be passed in Application Submission 
> context from Client side.
>               Here we can reuse the MRJobConfig.PRIORITY configuration. 
> 2. CLI changes to support {{--set-priority}}. Run time change of JobPriority 
> is also to be handled.
> 3. Change {{Job}} to have the support for {{setPriority}} and {{getPriority}} 
> (getter is handled with JobStatus)



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

Reply via email to