Details

    • Type: Improvement
    • Status: Open
    • Priority: 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

        Attachments

          Activity

            People

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

              Dates

              • Created:
                Updated: