Dan Hecht has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 13:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/5161/13/be/src/exprs/agg-fn-evaluator.cc
File
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 13:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5161/13/be/src/exprs/scalar-fn-call.h
File
Marcel Kornacker has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 13:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5161/13/be/src/exprs/scalar-fn-call.h
File
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 13: Code-Review+1
carry +1
--
To view, visit http://gerrit.cloudera.org:8080/5161
To unsubscribe,
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 12:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/5161/12/be/src/exprs/agg-fn-evaluator.cc
File
Hello Michael Ho,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5161
to look at the new patch set (#13).
Change subject: IMPALA-1430: enable codegen for native UDAs
..
IMPALA-1430:
Michael Ho has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 12: Code-Review+1
(2 comments)
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 11:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5161/11/be/src/exprs/agg-fn-evaluator.cc
File
Tim Armstrong has uploaded a new patch set (#12).
Change subject: IMPALA-1430: enable codegen for native UDAs
..
IMPALA-1430: enable codegen for native UDAs
This uses the existing infrastructure for codegening builtin UDAs and
Michael Ho has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 11:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5161/11/be/src/exprs/agg-fn-evaluator.cc
File
Michael Ho has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 11:
(1 comment)
Can you please address the question in agg-fn-evaluator.cc ? I can +1
afterwards.
Tim Armstrong has uploaded a new patch set (#11).
Change subject: IMPALA-1430: enable codegen for native UDAs
..
IMPALA-1430: enable codegen for native UDAs
This uses the existing infrastructure for codegening builtin UDAs and
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 10:
(7 comments)
http://gerrit.cloudera.org:8080/#/c/5161/10/be/src/exprs/agg-fn-evaluator.cc
File
Michael Ho has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 10:
(7 comments)
Looking good. Most comments about comments.
Tim Armstrong has uploaded a new patch set (#9).
Change subject: IMPALA-1430: enable codegen for native UDAs
..
IMPALA-1430: enable codegen for native UDAs
This uses the existing infrastructure for codegening builtin UDAs and
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 6:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5161/6/be/src/exprs/agg-fn-evaluator.cc
File
Tim Armstrong has uploaded a new patch set (#8).
Change subject: IMPALA-1430: enable codegen for native UDAs
..
IMPALA-1430: enable codegen for native UDAs
This uses the existing infrastructure for codegening builtin UDAs and
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 6:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/5161/6/be/src/exprs/agg-fn-evaluator.cc
File
Michael Ho has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 6:
(4 comments)
http://gerrit.cloudera.org:8080/#/c/5161/6/be/src/exprs/agg-fn-evaluator.cc
File
Tim Armstrong has uploaded a new patch set (#7).
Change subject: IMPALA-1430: enable codegen for native UDAs
..
IMPALA-1430: enable codegen for native UDAs
This uses the existing infrastructure for codegening builtin UDAs and
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-1430: enable codegen for native UDAs
..
Patch Set 6:
(7 comments)
http://gerrit.cloudera.org:8080/#/c/5161/6/be/src/exprs/agg-fn-evaluator.cc
File
Tim Armstrong has uploaded a new patch set (#3).
Change subject: IMPALA-1430: enable codegen for native UDAs
..
IMPALA-1430: enable codegen for native UDAs
This uses the existing infrastructure for codegening builtin UDAs and
Tim Armstrong has uploaded a new patch set (#2).
Change subject: IMPALA-1430: enable codegen for native UDAs
..
IMPALA-1430: enable codegen for native UDAs
This uses the existing infrastructure for codegening builtin UDAs and
Tim Armstrong has uploaded a new change for review.
http://gerrit.cloudera.org:8080/5161
Change subject: IMPALA-1430: enable codegen for native UDAs
..
IMPALA-1430: enable codegen for native UDAs
This uses the existing
24 matches
Mail list logo