[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-21 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. IMPALA-4821: Update AVG() for DECIMAL_V2 This change implements the DECIMAL_V2's behavior for AVG().

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-21 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6038 To unsubscribe, visit http://

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-21 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 5: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/291/ -- To view, visit h

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-21 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6038 To unsubscribe, visit http://gerrit.cl

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-21 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 5: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6038 To unsubscribe, visit http://gerrit.cl

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-21 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6038/4//COMMIT_MSG Commit Message: PS4, Line 25: types > D

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-21 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/6038/4//COMMIT_MSG Commit Message: PS4, Line 25: types >

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-21 Thread Michael Ho (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6038 to look at the new patch set (#5). Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. IMPALA-4821: Update AVG

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-21 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/6038/4//COMMIT_MSG Commit Message: PS4, Line 25: types va

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-19 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/6038/3//COMMIT_MSG Commit Message: Line 12: 2. precision

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-19 Thread Michael Ho (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6038 to look at the new patch set (#4). Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. IMPALA-4821: Update AVG

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-17 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/6038/3//COMMIT_MSG Commit Message: Line 12: 2. precision

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-17 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6038/2/testdata/workloads/functional-query/queries/QueryTe

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-17 Thread Michael Ho (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6038 to look at the new patch set (#3). Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. IMPALA-4821: Update AVG

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-17 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/6038/2/be/src/exprs/aggregate-functions-ir.cc File be/src

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-16 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/6038/2/fe/src/main/java/org/apache/impala/analysis/Functio

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-16 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 2: Code-Review+1 (2 comments) +2 for backend and tests http://gerrit.cloudera.org:8080/#/c/6038/2/be/src/expr

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-16 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/6038/1/be/src/exprs/aggregate-functions-ir.cc File be/src

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-16 Thread Michael Ho (Code Review)
Michael Ho has uploaded a new patch set (#2). Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. IMPALA-4821: Update AVG() for DECIMAL_V2 This change implements the DECIMAL_V2's behavior for AVG(). The major differences

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-16 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/6038/1/be/src/exprs/aggregate-functions-ir.cc File be/src/

[Impala-ASF-CR] IMPALA-4821: Update AVG() for DECIMAL V2

2017-02-16 Thread Michael Ho (Code Review)
Michael Ho has uploaded a new change for review. http://gerrit.cloudera.org:8080/6038 Change subject: IMPALA-4821: Update AVG() for DECIMAL_V2 .. IMPALA-4821: Update AVG() for DECIMAL_V2 This change implements the DECIMAL_V2'