[ https://issues.apache.org/jira/browse/TEZ-334?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
jiang jie updated TEZ-334:
--------------------------
Description:
MultiStageMRConfToTezTranslator.getStageConfs uses three local variables as follows
int totalStages = numIntermediateStages + (hasFinalReduceStage ? 2 : 1);
int numEdges = totalStages - 1;
int numStages = numEdges + 1;
The numStages is equal to the totalStages and only the numStages is used later, therefore
we can only use one local variable.
was:
MultiStageMRConfToTezTranslator.getStageConfs uses three local variables as follows
int totalStages = numIntermediateStages + (hasFinalReduceStage ? 2 : 1);
int numEdges = totalStages - 1;
int numStages = numEdges + 1;
The numStages is equal to the totalStages and only the numStages is used later, therefore
we can use only one local variable.
> There are redundant variables in MultiStageMRConfToTezTranslator.getStageConfs
> ------------------------------------------------------------------------------
>
> Key: TEZ-334
> URL: https://issues.apache.org/jira/browse/TEZ-334
> Project: Apache Tez
> Issue Type: Improvement
> Reporter: jiang jie
> Priority: Minor
>
> MultiStageMRConfToTezTranslator.getStageConfs uses three local variables as follows
> int totalStages = numIntermediateStages + (hasFinalReduceStage ? 2 : 1);
> int numEdges = totalStages - 1;
> int numStages = numEdges + 1;
> The numStages is equal to the totalStages and only the numStages is used later, therefore
we can only use one local variable.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
|