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

Hongyuan Li edited comment on MAPREDUCE-7186 at 2/21/19 9:17 AM:
-----------------------------------------------------------------

~[[email protected]] 

can this be assigned to me? i think the code should check that if the value of 
reducenums is valid.


was (Author: hongyuan li):
can this be assigned to me? i think the code should check that if the value of 
reducenums is valid.

> JobSubmitter#checkSpecs doesn not handle the situation when reduceNums is 
> less than zero.
> -----------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-7186
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-7186
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: client
>            Reporter: Hongyuan Li
>            Priority: Minor
>
> JobSubmitter#checkSpecs doesn not judge whether reduceNums is less than zero.
> {code:java}
> private void checkSpecs(Job job) throws ClassNotFoundException,
>       InterruptedException, IOException {
>       JobConf jConf = (JobConf)job.getConfiguration();
>       // Check the output specification
>       if (jConf.getNumReduceTasks() == 0 ?
>       jConf.getUseNewMapper() : jConf.getUseNewReducer()) {
>       org.apache.hadoop.mapreduce.OutputFormat<?, ?> output =
>       ReflectionUtils.newInstance(job.getOutputFormatClass(),
>       job.getConfiguration());
>       output.checkOutputSpecs(job);
>       } else {
>       jConf.getOutputFormat().checkOutputSpecs(jtFs, jConf);
>       }
>       }
>       {code}
> The code does handle the situation when reduceTaskNums is set to -1 or any 
> other value less than zero.
> When use hive, the "mapred.reduce.tasks" has a default value -1.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to