Hive
  1. Hive
  2. HIVE-697

GroupByOperator's initialization calls GenericUDAF.newAggregation before GenericUDAF.init

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.4.0
    • Component/s: Query Processor
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      It should call GenericUDAF.init first, so GenericUDAF knows what to store in the AggregationBuffer.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        23m 11s 1 Zheng Shao 28/Jul/09 07:21
        Patch Available Patch Available Resolved Resolved
        12h 56m 1 Namit Jain 28/Jul/09 20:17
        Resolved Resolved Closed Closed
        871d 4h 49m 1 Carl Steinbach 17/Dec/11 00:07
        Carl Steinbach made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Namit Jain made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Hadoop Flags [Reviewed]
        Resolution Fixed [ 1 ]
        Hide
        Namit Jain added a comment -

        Committed. Thanks Zheng

        Show
        Namit Jain added a comment - Committed. Thanks Zheng
        Hide
        Namit Jain added a comment -

        +1

        looks good - will commit if the tests pass

        Show
        Namit Jain added a comment - +1 looks good - will commit if the tests pass
        Zheng Shao made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Fix Version/s 0.4.0 [ 12313714 ]
        Zheng Shao made changes -
        Field Original Value New Value
        Attachment HIVE-697.1.patch [ 12414717 ]
        Hide
        Zheng Shao added a comment -

        This patch moves the block of code up. It does not contain any additional tests but I think it's OK since it's such a simple change.

        Show
        Zheng Shao added a comment - This patch moves the block of code up. It does not contain any additional tests but I think it's OK since it's such a simple change.
        Zheng Shao created issue -

          People

          • Assignee:
            Zheng Shao
            Reporter:
            Zheng Shao
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development