[Impala-ASF-CR] IMPALA-4893: Efficiently update the rows read counter for sequence file

2017-04-12 Thread anujphadke (Code Review)
anujphadke has posted comments on this change. Change subject: IMPALA-4893: Efficiently update the rows read counter for sequence file .. Patch Set 5: (2 comments)

[Impala-ASF-CR] IMPALA-5073: Part 1: add option to use mmap() for buffer pool

2017-04-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#10). Change subject: IMPALA-5073: Part 1: add option to use mmap() for buffer pool .. IMPALA-5073: Part 1: add option to use mmap() for buffer pool Support allocating with mmap

[Impala-ASF-CR] IMPALA-4815, IMPALA-4817, IMPALA-4819: Populate Parquet Statistics for remaining types

2017-04-12 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-4815, IMPALA-4817, IMPALA-4819: Populate Parquet Statistics for remaining types .. Patch Set 4: (19 comments) Thank you for your comments. Please see my

[Impala-ASF-CR] IMPALA-4815, IMPALA-4817, IMPALA-4819: Populate Parquet Statistics for remaining types

2017-04-12 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#4). Change subject: IMPALA-4815, IMPALA-4817, IMPALA-4819: Populate Parquet Statistics for remaining types .. IMPALA-4815, IMPALA-4817, IMPALA-4819: Populate Parquet Statistics for

[Impala-ASF-CR] IMPALA-5073: Part 1: add option to use mmap() for buffer pool

2017-04-12 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-5073: Part 1: add option to use mmap() for buffer pool .. Patch Set 9: (4 comments)

[Impala-ASF-CR](asf-site) IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF site

2017-04-12 Thread Jim Apple (Code Review)
Jim Apple has submitted this change and it was merged. Change subject: IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF site .. IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF site Add Impala

[Impala-ASF-CR](asf-site) IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF site

2017-04-12 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF site .. Patch Set 3: Code-Review+2 Verified+1 -- To view, visit

[Impala-ASF-CR](asf-site) IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF site

2017-04-12 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF site .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6604 To

[Impala-ASF-CR] IMPALA-5073: Part 1: add option to use mmap() for buffer pool

2017-04-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#9). Change subject: IMPALA-5073: Part 1: add option to use mmap() for buffer pool .. IMPALA-5073: Part 1: add option to use mmap() for buffer pool Support allocating with mmap

[Impala-ASF-CR] IMPALA-5073: Part 1: add option to use mmap() for buffer pool

2017-04-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5073: Part 1: add option to use mmap() for buffer pool .. Patch Set 8: (3 comments)

[Impala-ASF-CR](asf-site) IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF resources.

2017-04-12 Thread Laurel Hale (Code Review)
Laurel Hale has posted comments on this change. Change subject: IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF resources. .. Patch Set 2: > (1 comment) ok--I'll repush -- To view, visit

[Impala-ASF-CR](asf-site) IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF resources.

2017-04-12 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF resources. .. Patch Set 2: (1 comment) Laurel, HTML and PDF look good, but see comment.

[Impala-ASF-CR] IMPALA-4815, IMPALA-4817, IMPALA-4819: Populate Parquet Statistics for remaining types

2017-04-12 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new patch set (#3). Change subject: IMPALA-4815, IMPALA-4817, IMPALA-4819: Populate Parquet Statistics for remaining types .. IMPALA-4815, IMPALA-4817, IMPALA-4819: Populate Parquet Statistics for

[Impala-ASF-CR](asf-site) IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF resources.

2017-04-12 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF resources. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6604/1//COMMIT_MSG

[Impala-ASF-CR](asf-site) IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF resources.

2017-04-12 Thread Laurel Hale (Code Review)
Hello Michael Brown, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6604 to look at the new patch set (#2). Change subject: IMPALA-4181 [DOCS] Publish rendered Impala documentation to ASF resources.

[Impala-ASF-CR] IMPALA-4893: Efficiently update the rows read counter for sequence file

2017-04-12 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4893: Efficiently update the rows read counter for sequence file .. Patch Set 4: (2 comments) Only minor style comments remaining.

[Impala-ASF-CR] IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet

2017-04-12 Thread Attila Jeges (Code Review)
Attila Jeges has uploaded a new patch set (#7). Change subject: IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet .. IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet Before this

[Impala-ASF-CR] IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet

2017-04-12 Thread Attila Jeges (Code Review)
Attila Jeges has posted comments on this change. Change subject: IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet .. Patch Set 7: (9 comments)

[Impala-ASF-CR] IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet

2017-04-12 Thread Attila Jeges (Code Review)
Attila Jeges has uploaded a new patch set (#7). Change subject: IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet .. IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet Before this

[Impala-ASF-CR] IMPALA-5180: Don't balk at non-deterministic exprs

2017-04-12 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5180: Don't balk at non-deterministic exprs .. Patch Set 3: (8 comments) Fix looks much better http://gerrit.cloudera.org:8080/#/c/6575/3//COMMIT_MSG