[Impala-ASF-CR] IMPALA-4817: Populate Parquet Statistics for Strings

2017-04-10 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4817: Populate Parquet Statistics for Strings .. Patch Set 2: (7 comments) I had a few high-level comments/questions.

[Impala-ASF-CR] IMPALA-4817: Populate Parquet Statistics for Strings

2017-04-10 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4817: Populate Parquet Statistics for Strings .. Patch Set 2: (15 comments)

[Impala-ASF-CR] IMPALA-4817: Populate Parquet Statistics for Strings

2017-04-10 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4817: Populate Parquet Statistics for Strings .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6563/2/common/thrift/parquet.thrift

[Impala-ASF-CR] IMPALA-4817: Populate Parquet Statistics for Strings

2017-04-10 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4817: Populate Parquet Statistics for Strings .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6563/2/common/thrift/parquet.thrift File

[Impala-ASF-CR] IMPALA-4817: Populate Parquet Statistics for Strings

2017-04-06 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#2). Change subject: IMPALA-4817: Populate Parquet Statistics for Strings .. IMPALA-4817: Populate Parquet Statistics for Strings This change adds functionality to populate the new

[Impala-ASF-CR] IMPALA-4817: Populate Parquet Statistics for Strings

2017-04-05 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/6563 Change subject: IMPALA-4817: Populate Parquet Statistics for Strings .. IMPALA-4817: Populate Parquet Statistics for Strings This change