Pig
  1. Pig
  2. PIG-1512

PlanPrinter does not print LOJoin operator in the new logical optimization framework

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 0.8.0
    • Component/s: None
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      PlanPrinter does not print LOJoin relational operator. As such, the LOJoin operator would not get printed when we do an explain.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        56s 1 Swati Jain 22/Jul/10 09:11
        In Progress In Progress Patch Available Patch Available
        8h 32m 1 Swati Jain 22/Jul/10 17:43
        Patch Available Patch Available Resolved Resolved
        36d 4h 53m 1 Daniel Dai 27/Aug/10 22:37
        Resolved Resolved Closed Closed
        112d 1h 7m 1 Olga Natkovich 17/Dec/10 22:44
        Olga Natkovich made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Daniel Dai made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Daniel Dai added a comment -

        This is already fixed in the latest code. Thanks Swati!

        Show
        Daniel Dai added a comment - This is already fixed in the latest code. Thanks Swati!
        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12450145/printJoin.patch
        against trunk revision 979503.

        +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 generated 407 release audit warnings (more than the trunk's current 405 warnings).

        -1 core tests. The patch failed core unit tests.

        -1 contrib tests. The patch failed contrib unit tests.

        Test results: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/380/testReport/
        Release audit warnings: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/380/artifact/trunk/patchprocess/releaseAuditDiffWarnings.txt
        Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/380/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
        Console output: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/380/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/12450145/printJoin.patch against trunk revision 979503. +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 generated 407 release audit warnings (more than the trunk's current 405 warnings). -1 core tests. The patch failed core unit tests. -1 contrib tests. The patch failed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/380/testReport/ Release audit warnings: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/380/artifact/trunk/patchprocess/releaseAuditDiffWarnings.txt Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/380/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: http://hudson.zones.apache.org/hudson/job/Pig-Patch-h7.grid.sp2.yahoo.net/380/console This message is automatically generated.
        Swati Jain made changes -
        Status In Progress [ 3 ] Patch Available [ 10002 ]
        Swati Jain made changes -
        Patch Info [Patch Available]
        Swati Jain made changes -
        Attachment printJoin.patch [ 12450142 ]
        Swati Jain made changes -
        Attachment printJoin.patch [ 12450143 ]
        Swati Jain made changes -
        Attachment printJoin.patch [ 12450145 ]
        Hide
        Swati Jain added a comment -

        Attach the right file, final upload.

        Show
        Swati Jain added a comment - Attach the right file, final upload.
        Swati Jain made changes -
        Attachment printJoin.patch [ 12450143 ]
        Hide
        Swati Jain added a comment -

        Fix tab character

        Show
        Swati Jain added a comment - Fix tab character
        Swati Jain made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Swati Jain made changes -
        Attachment printJoin.patch [ 12450142 ]
        Swati Jain made changes -
        Field Original Value New Value
        Assignee Swati Jain [ swati.j ]
        Swati Jain created issue -

          People

          • Assignee:
            Swati Jain
            Reporter:
            Swati Jain
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development