Details

      Description

      Extend the VectorGroupByOperator and the VectorUDAF aggregates to support group by.

      1. HIVE-4450-p1.patch.txt
        264 kB
        Remus Rusanu
      2. HIVE-4450-p1.patch.txt
        264 kB
        Remus Rusanu
      3. HIVE-4450-p1.patch.txt
        261 kB
        Remus Rusanu
      4. HIVE-4450-p1.patch.txt
        523 kB
        Remus Rusanu

        Issue Links

          Activity

          Hide
          Remus Rusanu added a comment -
          Show
          Remus Rusanu added a comment - Phabricator review https://reviews.facebook.net/D10731
          Hide
          Jitendra Nath Pandey added a comment -

          Please upload the patch on the jira as well.

          Show
          Jitendra Nath Pandey added a comment - Please upload the patch on the jira as well.
          Hide
          Eric Hanson added a comment -

          Finished code review. See my comments on Phabricator entry.

          Show
          Eric Hanson added a comment - Finished code review. See my comments on Phabricator entry.
          Hide
          Remus Rusanu added a comment -

          Updated patch, incorporates Phabricator review feed back

          Show
          Remus Rusanu added a comment - Updated patch, incorporates Phabricator review feed back
          Hide
          Remus Rusanu added a comment -

          This one is definetely ANSI

          Show
          Remus Rusanu added a comment - This one is definetely ANSI
          Hide
          Remus Rusanu added a comment -

          added missing file

          Show
          Remus Rusanu added a comment - added missing file
          Hide
          Remus Rusanu added a comment -

          This one I applied to second enlistment and confirmed it compiles

          Show
          Remus Rusanu added a comment - This one I applied to second enlistment and confirmed it compiles
          Hide
          Owen O'Malley added a comment -

          I just committed this to vectorization branch. Thanks, Remus!

          Show
          Owen O'Malley added a comment - I just committed this to vectorization branch. Thanks, Remus!

            People

            • Assignee:
              Remus Rusanu
              Reporter:
              Remus Rusanu
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development