You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by "Namit Jain (Created) (JIRA)" <ji...@apache.org> on 2011/12/21 18:21:30 UTC

[jira] [Created] (HIVE-2668) Minor cleanup to HIVE-2642

Minor cleanup to HIVE-2642
--------------------------

                 Key: HIVE-2668
                 URL: https://issues.apache.org/jira/browse/HIVE-2668
             Project: Hive
          Issue Type: Improvement
            Reporter: Namit Jain
            Assignee: He Yongqiang


INLINE COMMENTS
ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMRProcContext.java:105 can you add some comments here ?

This is not really the top operators - this contains the list of intermediate tables also. This code is difficult to debug later on, so more comments would be helpful


Look at union22.q.out.

map-join followed by union, an extra stage is introduced.
We dont have to optimize this - just wanted to make sure it is intentional.



1 general comment about the new test union26.q -

Reduce the test output, I mean, you dont need to load all 500 rows for this test.
It makes the test output really difficult to review.


ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMRUnion1.java:245 remove this comment

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira