Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: tez-branch
    • Component/s: tez
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Order by desc is broken. Tests such as Types_15, Types_17 fail.

      1. PIG-3758-1.patch
        0.9 kB
        Daniel Dai
      2. PIG-3758-2.patch
        3 kB
        Daniel Dai

        Activity

        Hide
        Daniel Dai added a comment -

        Patch committed to trunk. Thanks Rohini for review!

        Show
        Daniel Dai added a comment - Patch committed to trunk. Thanks Rohini for review!
        Hide
        Rohini Palaniswamy added a comment -

        +1

        Show
        Rohini Palaniswamy added a comment - +1
        Hide
        Daniel Dai added a comment -

        Good point. We shall consistently use the successor for the properties. Update the patch.

        Show
        Daniel Dai added a comment - Good point. We shall consistently use the successor for the properties. Update the patch.
        Hide
        Mark Wagner added a comment -

        Is there a purpose to storing the sort order in the predecessor operator, but the global sort property in the successor? Other than that this looks good.

        Show
        Mark Wagner added a comment - Is there a purpose to storing the sort order in the predecessor operator, but the global sort property in the successor? Other than that this looks good.

          People

          • Assignee:
            Daniel Dai
            Reporter:
            Daniel Dai
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development