Pig
  1. Pig
  2. PIG-959

Merge Join fails when there is a blocking operator before it in query.

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 0.7.0
    • Fix Version/s: None
    • Component/s: impl
    • Labels:
      None

      Description

      If there is an order-by, distinct or any other blocking operator in query followed by Merge Join, pig fails to compile it.

      1. pig-959.patch
        12 kB
        Ashutosh Chauhan

        Issue Links

          Activity

          Hide
          Ashutosh Chauhan added a comment -

          Yeah.. resolved.

          Show
          Ashutosh Chauhan added a comment - Yeah.. resolved.
          Hide
          Dmitriy V. Ryaboy added a comment -

          Ashutosh, should we close this as won't fix?

          Show
          Dmitriy V. Ryaboy added a comment - Ashutosh, should we close this as won't fix?
          Hide
          Dmitriy V. Ryaboy added a comment -

          Note that a subset of this is implemented in PIG-2673

          Show
          Dmitriy V. Ryaboy added a comment - Note that a subset of this is implemented in PIG-2673
          Hide
          Olga Natkovich added a comment -

          We are not seeing any asks for this at this time

          Show
          Olga Natkovich added a comment - We are not seeing any asks for this at this time
          Hide
          Daniel Dai added a comment -

          +1

          Show
          Daniel Dai added a comment - +1
          Hide
          Ashutosh Chauhan added a comment -

          Patch includes two new tests. Not sure why hudson thought otherwise. Patch is ready for review.

          Show
          Ashutosh Chauhan added a comment - Patch includes two new tests. Not sure why hudson thought otherwise. Patch is ready for review.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12440943/pig-959.patch
          against trunk revision 930168.

          +1 @author. The patch does not contain any @author tags.

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no tests are needed for this patch.

          +1 javadoc. The javadoc tool did not generate any warning messages.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 findbugs. The patch does not introduce any new Findbugs warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          +1 core tests. The patch passed core unit tests.

          +1 contrib tests. The patch passed contrib unit tests.

          Test results: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/274/testReport/
          Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/274/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
          Console output: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/274/console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12440943/pig-959.patch against trunk revision 930168. +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no tests are needed for this patch. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. +1 core tests. The patch passed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/274/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/274/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/274/console This message is automatically generated.
          Hide
          Ashutosh Chauhan added a comment -

          Running through hudson.

          Show
          Ashutosh Chauhan added a comment - Running through hudson.
          Hide
          Ashutosh Chauhan added a comment -

          Attached patch which lifts the restriction that no blocking operator could be placed before merge join. With this work, data can be ordered and joined in same script.

          Show
          Ashutosh Chauhan added a comment - Attached patch which lifts the restriction that no blocking operator could be placed before merge join. With this work, data can be ordered and joined in same script.
          Hide
          Ashutosh Chauhan added a comment -

          This issue is blocked on PIG-858

          Show
          Ashutosh Chauhan added a comment - This issue is blocked on PIG-858

            People

            • Assignee:
              Ashutosh Chauhan
              Reporter:
              Ashutosh Chauhan
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development