You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by "Zheng Shao (JIRA)" <ji...@apache.org> on 2009/06/07 10:20:07 UTC

[jira] Created: (HIVE-548) Code cleanup: refactor the 4 methods like SemanticAnalyzer.genGroupByPlanGroupByOperator

Code cleanup: refactor the 4 methods like SemanticAnalyzer.genGroupByPlanGroupByOperator
----------------------------------------------------------------------------------------

                 Key: HIVE-548
                 URL: https://issues.apache.org/jira/browse/HIVE-548
             Project: Hadoop Hive
          Issue Type: Improvement
            Reporter: Zheng Shao


We have 4 methods to generate the GroupByOperators in the SemanticAnalyzer:

{code}
  SemanticAnalyzer.genGroupByPlanGroupByOperator
  SemanticAnalyzer.genGroupByPlanMapGroupByOperator
  SemanticAnalyzer.genGroupByPlanGroupByOperator1
  SemanticAnalyzer.genGroupByPlanGroupByOperator2
{code}

While they are all different, a substantial portion of them are the same. We should factor out all the common functionality into shared functions.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.