Vincent Tran has posted comments on this change.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
Patch Set 3:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/7000/3/be/src/exprs/expr-test.cc
File be/src/exprs
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
Patch Set 3:
(2 comments)
Looks good aside from a couple of minor things.
http://gerrit.cloudera.org:8080/#/c/
Vincent Tran has uploaded a new patch set (#3).
Change subject: IMPALA-5030: Adds support for NVL2() function
..
IMPALA-5030: Adds support for NVL2() function
This change adds value tests to expr-test.cc to ensure
that NVL2() fu
Vincent Tran has uploaded a new patch set (#2).
Change subject: IMPALA-5030: Adds support for NVL2() function
..
IMPALA-5030: Adds support for NVL2() function
This change adds value and null tests to expr-test.cc for
IMPALA-5030
Vincent Tran has uploaded a new change for review.
http://gerrit.cloudera.org:8080/7000
Change subject: IMPALA-5030: Adds support for NVL2() function
..
IMPALA-5030: Adds support for NVL2() function
This change adds support t
Vincent Tran has posted comments on this change.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
Patch Set 5:
I can make that happen. Can I just push the patch again as normal?
--
To view, visit http://gerrit
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
Patch Set 5:
I'm late to the party but we need a test that confirms the expr gets written
correctly - e.g. in e
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
IMPALA-5030: Adds support for NVL2() function
This change adds support to the function NVL2(exp
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
Patch Set 4: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/6962
To unsubscribe, visit ht
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
Patch Set 4:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/637/
--
To view, vi
Alex Behm has posted comments on this change.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
Patch Set 4: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/6962
To unsubscribe, visit http://gerr
Vincent Tran has uploaded a new patch set (#4).
Change subject: IMPALA-5030: Adds support for NVL2() function
..
IMPALA-5030: Adds support for NVL2() function
This change adds support to the function NVL2(expr1, expr2, expr3).
T
Vincent Tran has posted comments on this change.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
Patch Set 3:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/6962/3/fe/src/main/java/org/apache/impala/analysis
Alex Behm has posted comments on this change.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
Patch Set 3:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6962/3/fe/src/main/java/org/apache/impala/analysis/Fun
Michael Ho has posted comments on this change.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
Patch Set 3:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/6962/3/fe/src/main/java/org/apache/impala/analysis/F
Vincent Tran has uploaded a new patch set (#3).
Change subject: IMPALA-5030: Adds support for NVL2() function
..
IMPALA-5030: Adds support for NVL2() function
This change adds support to the function NVL2(expr1, expr2, expr3).
T
Vincent Tran has posted comments on this change.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
Patch Set 2:
(16 comments)
http://gerrit.cloudera.org:8080/#/c/6962/1//COMMIT_MSG
Commit Message:
Line 17
> Cle
Alex Behm has posted comments on this change.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
Patch Set 2:
(7 comments)
http://gerrit.cloudera.org:8080/#/c/6962/2//COMMIT_MSG
Commit Message:
Line 11: expr3.
B
Vincent Tran has uploaded a new patch set (#2).
Change subject: IMPALA-5030: Adds support for NVL2() function
..
IMPALA-5030: Adds support for NVL2() function
This change adds support to the function NVL2(expr1, expr2, expr3).
T
Vincent Tran has abandoned this change.
Change subject: IMPALA-5030: Adds support for NVL2() function
..
Abandoned
--
To view, visit http://gerrit.cloudera.org:8080/6972
To unsubscribe, visit http://gerrit.cloudera.org:8080/se
Vincent Tran has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6972
Change subject: IMPALA-5030: Adds support for NVL2() function
..
IMPALA-5030: Adds support for NVL2() function
This change adds support t
21 matches
Mail list logo