[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-24 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 11: (6 comments) Thank you Alex for the review. I submitted a change to address your comment

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/6032/11/fe/src/main/java/org/apache/impala

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-23 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/6032/11/fe/src/main/java/org/apache

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-23 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 11: (7 comments) http://gerrit.cloudera.org:8080/#/c/6032/11/be/src/exec/hdfs-parquet-scanner.

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 10: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6032 To unsubscri

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-23 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. IMPALA-2328: Read support for min/max Parquet statistics This change adds support fo

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-22 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 10: Code-Review+2 PS10 is a rebase. Carrying Marcel's +2. -- To view, visit http://gerrit.c

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 10: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/295/ --

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-22 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 9: PS9 addresses a clang-tidy error. -- To view, visit http://gerrit.cloudera.org:8080/6032

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-22 Thread Lars Volker (Code Review)
Hello Marcel Kornacker, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6032 to look at the new patch set (#9). Change subject: IMPALA-2328: Read support for min/max Parquet statistics .

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 8: Verified-1 Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-22 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 8: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/294/ --

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-22 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6032 To unsubscribe,

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-22 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 4: (1 comment) Thanks for the review. I addressed the remaining comments in PS8. There's the

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-22 Thread Lars Volker (Code Review)
Hello Marcel Kornacker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6032 to look at the new patch set (#8). Change subject: IMPALA-2328: Read support for min/max Parquet statistics ..

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-22 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/6032/6/be/src/exec/parquet-column-stats.

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-22 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 7: Code-Review+2 (3 comments) http://gerrit.cloudera.org:8080/#/c/6032/6/be/src/exec/pa

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-22 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 6: (14 comments) Thanks for the review. Please see PS7. http://gerrit.cloudera.org:8080/#/c

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-22 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#7). Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. IMPALA-2328: Read support for min/max Parquet statistics This change adds support for skipping row groups

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-21 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 6: (14 comments) http://gerrit.cloudera.org:8080/#/c/6032/6/be/src/exec/hdfs-parquet-sc

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-21 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6032/4/testdata/workloads/functional-quer

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-21 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#6). Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. IMPALA-2328: Read support for min/max Parquet statistics This change adds support for skipping row groups

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-21 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 4: (26 comments) Thanks for the review. Please see PS6. http://gerrit.cloudera.org:8080/#/c

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-21 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#5). Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. IMPALA-2328: Read support for min/max Parquet statistics This change adds support for skipping row groups

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-21 Thread Mostafa Mokhtar (Code Review)
Mostafa Mokhtar has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6032/4/testdata/workloads/functional-

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-19 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 4: (26 comments) http://gerrit.cloudera.org:8080/#/c/6032/4/be/src/exec/hdfs-parquet-sc

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-17 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 4: PS4 adds the "statistics predicates" to the PlannerTest and removes the custom predicate

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-17 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#4). Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. IMPALA-2328: Read support for min/max Parquet statistics This change adds support for skipping row groups

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-16 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 3: PS3 adds the statistics predicates to EXPLAIN output and adds tests for that. -- To view

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-16 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#3). Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. IMPALA-2328: Read support for min/max Parquet statistics This change adds support for skipping row groups

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-16 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. Patch Set 2: PS2 adds error handling when deserializing statistics -- To view, visit http://gerrit.cl

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-16 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#2). Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. IMPALA-2328: Read support for min/max Parquet statistics This change adds support for skipping row groups

[Impala-ASF-CR] IMPALA-2328: Read support for min/max Parquet statistics

2017-02-15 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/6032 Change subject: IMPALA-2328: Read support for min/max Parquet statistics .. IMPALA-2328: Read support for min/max Parquet statistics This