Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      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

        Activity

        There are no comments yet on this issue.

          People

          • Assignee:
            He Yongqiang
            Reporter:
            Namit Jain
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:

              Development