You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tez.apache.org by "Siddharth Seth (JIRA)" <ji...@apache.org> on 2013/08/13 06:31:49 UTC

[jira] [Resolved] (TEZ-334) There are redundant variables in MultiStageMRConfToTezTranslator.getStageConfs

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

Siddharth Seth resolved TEZ-334.
--------------------------------

       Resolution: Fixed
    Fix Version/s: 0.2.0

Committed to master.
                
> 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
>            Assignee: jiang jie
>            Priority: Minor
>             Fix For: 0.2.0
>
>         Attachments: TEZ-334.patch
>
>
> 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