[Impala-ASF-CR] IMPALA-5754: Improve randomness of rand()/random()

2017-11-13 Thread Kim Jin Chul (Code Review)
Hello Jim Apple, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8355 to look at the new patch set (#5). Change subject: IMPALA-5754: Improve randomness of rand()/random() .. IMPALA-5754:

[Impala-ASF-CR](asf-site) Update Impala docs for 2.10 release

2017-11-13 Thread Michael Brown (Code Review)
Michael Brown has removed a vote on this change. Change subject: Update Impala docs for 2.10 release .. Removed Verified-1 by Impala Public Jenkins (255) -- To view, visit http://gerrit.cloudera.org:8080/8511 To unsubscribe,

[Impala-ASF-CR] IMPALA-6087: Revisit tests withheld from TPC-DS suite for use of TRUNCATE

2017-11-13 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. ( http://gerrit.cloudera.org:8080/8372 ) Change subject: IMPALA-6087: Revisit tests withheld from TPC-DS suite for use of TRUNCATE .. Patch Set 5: (2 comments)

[Impala-ASF-CR] IMPALA-4985: use parquet stats of nested types for dynamic pruning

2017-11-13 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/8480 ) Change subject: IMPALA-4985: use parquet stats of nested types for dynamic pruning .. Patch Set 1: (8 comments)

[Impala-ASF-CR] IMPALA-6054: Parquet dictionary pages should be freed on dictionary construction

2017-11-13 Thread Csaba Ringhofer (Code Review)
Hello Lars Volker, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8436 to look at the new patch set (#5). Change subject: IMPALA-6054: Parquet dictionary pages should be freed on dictionary construction

[Impala-ASF-CR] IMPALA-3613: Avoid topic updates to unregistered subscriber instances

2017-11-13 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/8449 ) Change subject: IMPALA-3613: Avoid topic updates to unregistered subscriber instances .. Patch Set 6: (1 comment)

[Impala-ASF-CR](asf-site) Update Impala docs for 2.10 release

2017-11-13 Thread John Russell (Code Review)
John Russell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8511 ) Change subject: Update Impala docs for 2.10 release .. Update Impala docs for 2.10 release Commit hash =

[Impala-ASF-CR] IMPALA-6087: Revisit tests withheld from TPC-DS suite for use of TRUNCATE

2017-11-13 Thread Mostafa Mokhtar (Code Review)
Mostafa Mokhtar has posted comments on this change. ( http://gerrit.cloudera.org:8080/8372 ) Change subject: IMPALA-6087: Revisit tests withheld from TPC-DS suite for use of TRUNCATE .. Patch Set 5: (2 comments) Do you have

[Impala-ASF-CR] Expose $IMPALA MAVEN OPTIONS for configuring Maven.

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8496 ) Change subject: Expose $IMPALA_MAVEN_OPTIONS for configuring Maven. .. Expose $IMPALA_MAVEN_OPTIONS for configuring

[Impala-ASF-CR] IMPALA-6087: Revisit tests withheld from TPC-DS suite for use of TRUNCATE

2017-11-13 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. ( http://gerrit.cloudera.org:8080/8372 ) Change subject: IMPALA-6087: Revisit tests withheld from TPC-DS suite for use of TRUNCATE .. Patch Set 6: (2 comments) > Patch Set 5: > >

[Impala-ASF-CR] IMPALA-5976: Remove equivalence class computation in FE

2017-11-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8317 ) Change subject: IMPALA-5976: Remove equivalence class computation in FE .. Patch Set 4: (24 comments)

[Impala-ASF-CR] IMPALA-3613: Avoid topic updates to unregistered subscriber instances

2017-11-13 Thread Bharath Vissapragada (Code Review)
Hello Sailesh Mukil, Dimitris Tsirogiannis, Alex Behm, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8449 to look at the new patch set (#7). Change subject: IMPALA-3613: Avoid topic updates to unregistered subscriber instances

[Impala-ASF-CR] Expose $IMPALA MAVEN OPTIONS for configuring Maven.

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8496 ) Change subject: Expose $IMPALA_MAVEN_OPTIONS for configuring Maven. .. Patch Set 4: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6087: Revisit tests withheld from TPC-DS suite for use of TRUNCATE

2017-11-13 Thread Tim Wood (Code Review)
Hello Greg Rahn, Matthew Mulder, Michael Brown, Mostafa Mokhtar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8372 to look at the new patch set (#6). Change subject: IMPALA-6087: Revisit tests withheld from TPC-DS suite for use of TRUNCATE

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-11-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 14: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-3613: Avoid topic updates to unregistered subscriber instances

2017-11-13 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8449 ) Change subject: IMPALA-3613: Avoid topic updates to unregistered subscriber instances .. Patch Set 7: (1 comment)

[Impala-ASF-CR] IMPALA-5754: Improve randomness of rand()/random()

2017-11-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. ( http://gerrit.cloudera.org:8080/8355 ) Change subject: IMPALA-5754: Improve randomness of rand()/random() .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-6054: Parquet dictionary pages should be freed on dictionary construction

2017-11-13 Thread Csaba Ringhofer (Code Review)
Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/8436 ) Change subject: IMPALA-6054: Parquet dictionary pages should be freed on dictionary construction .. Patch Set 5: (4 comments)

[Impala-ASF-CR] IMPALA-4704: Turns on client connections when local catalog initialized.

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8202 ) Change subject: IMPALA-4704: Turns on client connections when local catalog initialized. .. Patch Set 24: Build started:

[Impala-ASF-CR] IMPALA-4704: Turns on client connections when local catalog initialized.

2017-11-13 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8202 ) Change subject: IMPALA-4704: Turns on client connections when local catalog initialized. .. Patch Set 24: Code-Review+2 -- To view,

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-11-13 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 14: (2 comments) looked through the headers and thrift and it

[Impala-ASF-CR] [DOCS] Remove unnecessary 'incubator' from URLs

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8524 ) Change subject: [DOCS] Remove unnecessary 'incubator' from URLs .. Patch Set 2: Verified+1 -- To view, visit

[Impala-ASF-CR] [DOCS] Remove unnecessary 'incubator' from URLs

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8524 ) Change subject: [DOCS] Remove unnecessary 'incubator' from URLs .. [DOCS] Remove unnecessary 'incubator' from URLs

[Impala-ASF-CR] IMPALA-2235: Fix current db when shell auto-reconnects

2017-11-13 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/8368 ) Change subject: IMPALA-2235: Fix current db when shell auto-reconnects .. Patch Set 3: Code-Review+1 (3 comments) Thanks for the

[Impala-ASF-CR] IMPALA-6160: Rework query execution to handle multiple statements in a Query object.

2017-11-13 Thread Tim Wood (Code Review)
Hello Matthew Mulder, Michael Brown, Jim Apple, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8513 to look at the new patch set (#2). Change subject: IMPALA-6160: Rework query execution to handle multiple statements in a Query object.

[Impala-ASF-CR] [PREVIEW] IMPALA-4886: Expose table metrics in the catalog web UI.

2017-11-13 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8529 Change subject: [PREVIEW] IMPALA-4886: Expose table metrics in the catalog web UI. .. [PREVIEW] IMPALA-4886: Expose table

[Impala-ASF-CR] [PREVIEW] IMPALA-4886: Expose table metrics in the catalog web UI.

2017-11-13 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. ( http://gerrit.cloudera.org:8080/8529 ) Change subject: [PREVIEW] IMPALA-4886: Expose table metrics in the catalog web UI. .. Patch Set 1: Screenshots of the catalog

[Impala-ASF-CR] IMPALA-6160: Rework query execution to handle multiple statements in a Query object.

2017-11-13 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. ( http://gerrit.cloudera.org:8080/8513 ) Change subject: IMPALA-6160: Rework query execution to handle multiple statements in a Query object. .. Patch Set 2: (7 comments)

[Impala-ASF-CR] [DOCS] Remove unnecessary 'incubator' from URLs

2017-11-13 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8524 ) Change subject: [DOCS] Remove unnecessary 'incubator' from URLs .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] [DOCS] Remove unnecessary 'incubator' from URLs

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8524 ) Change subject: [DOCS] Remove unnecessary 'incubator' from URLs .. Patch Set 2: Build started:

[Impala-ASF-CR] Pin gen build version's git handling to typical git dir.

2017-11-13 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8500 ) Change subject: Pin gen_build_version's git handling to typical git dir. .. Patch Set 1: Sorry, didn't realize this was the upstream

[Impala-ASF-CR] IMPALA-4177,IMPALA-6039: batched bit reading and rle decoding

2017-11-13 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/8267 ) Change subject: IMPALA-4177,IMPALA-6039: batched bit reading and rle decoding .. Patch Set 14: Code-Review+2 Yes, I had another look at

[Impala-ASF-CR](asf-site) Update Impala docs for 2.10 release

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8511 ) Change subject: Update Impala docs for 2.10 release .. Patch Set 3: Build started:

[Impala-ASF-CR](asf-site) Update Impala docs for 2.10 release

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8511 ) Change subject: Update Impala docs for 2.10 release .. Patch Set 3: Verified-1 Build failed:

[Impala-ASF-CR](asf-site) Update Impala docs for 2.10 release

2017-11-13 Thread John Russell (Code Review)
John Russell has posted comments on this change. ( http://gerrit.cloudera.org:8080/8511 ) Change subject: Update Impala docs for 2.10 release .. Patch Set 3: Verified+1 Oops, ran the verification job by mistake instead of

[Impala-ASF-CR] Pin gen build version's git handling to typical git dir.

2017-11-13 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/8500 ) Change subject: Pin gen_build_version's git handling to typical git dir. .. Patch Set 1: Ping? Thanks! -- To view, visit

[Impala-ASF-CR] IMPALA-6160: Allow multiple statements in a Query object.

2017-11-13 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. ( http://gerrit.cloudera.org:8080/8513 ) Change subject: IMPALA-6160: Allow multiple statements in a Query object. .. Patch Set 4: (2 comments)

[Impala-ASF-CR] IMPALA-6160: Allow multiple statements in a Query object.

2017-11-13 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. ( http://gerrit.cloudera.org:8080/8513 ) Change subject: IMPALA-6160: Allow multiple statements in a Query object. .. Patch Set 3: (7 comments)

[Impala-ASF-CR] IMPALA-6160: Allow multiple statements in a Query object.

2017-11-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. ( http://gerrit.cloudera.org:8080/8513 ) Change subject: IMPALA-6160: Allow multiple statements in a Query object. .. Patch Set 3: (2 comments) Just nits left

[Impala-ASF-CR] Pin gen build version's git handling to typical git dir.

2017-11-13 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8500 ) Change subject: Pin gen_build_version's git handling to typical git dir. .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-6160: Allow multiple statements in a Query object.

2017-11-13 Thread Tim Wood (Code Review)
Hello Matthew Mulder, Michael Brown, Jim Apple, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8513 to look at the new patch set (#4). Change subject: IMPALA-6160: Allow multiple statements in a Query object.

[Impala-ASF-CR] IMPALA-6160: Allow multiple statements in a Query object.

2017-11-13 Thread Tim Wood (Code Review)
Hello Matthew Mulder, Michael Brown, Jim Apple, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8513 to look at the new patch set (#3). Change subject: IMPALA-6160: Allow multiple statements in a Query object.

[Impala-ASF-CR] IMPALA-6160: Rework query execution to handle multiple statements in a Query object.

2017-11-13 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. ( http://gerrit.cloudera.org:8080/8513 ) Change subject: IMPALA-6160: Rework query execution to handle multiple statements in a Query object. .. Patch Set 2: (2 comments)

[Impala-ASF-CR] Pin gen build version's git handling to typical git dir.

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8500 ) Change subject: Pin gen_build_version's git handling to typical git dir. .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-4704: Turns on client connections when local catalog initialized.

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8202 ) Change subject: IMPALA-4704: Turns on client connections when local catalog initialized. .. IMPALA-4704: Turns on

[Impala-ASF-CR] IMPALA-4704: Turns on client connections when local catalog initialized.

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8202 ) Change subject: IMPALA-4704: Turns on client connections when local catalog initialized. .. Patch Set 24: Verified+1 -- To

[Impala-ASF-CR] IMPALA-6160: Allow multiple statements in a Query object.

2017-11-13 Thread Tim Wood (Code Review)
Tim Wood has posted comments on this change. ( http://gerrit.cloudera.org:8080/8513 ) Change subject: IMPALA-6160: Allow multiple statements in a Query object. .. Patch Set 3: (2 comments)

[Impala-ASF-CR] IMPALA-3613: Avoid topic updates to unregistered subscriber instances

2017-11-13 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8449 ) Change subject: IMPALA-3613: Avoid topic updates to unregistered subscriber instances .. Patch Set 6: (1 comment)

[Impala-ASF-CR] IMPALA-6121: remove I/O mgr request context cache

2017-11-13 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8408 ) Change subject: IMPALA-6121: remove I/O mgr request context cache .. Patch Set 8: Code-Review+2 (3 comments)

[Impala-ASF-CR] IMPALA-4252: Min-max runtime filters for Kudu

2017-11-13 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 14: Code-Review+1 (1 comment)

[Impala-ASF-CR] Pin gen build version's git handling to typical git dir.

2017-11-13 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8500 ) Change subject: Pin gen_build_version's git handling to typical git dir. .. Patch Set 1: Sounds like

[Impala-ASF-CR](asf-site) Update Impala docs for 2.10 release

2017-11-13 Thread John Russell (Code Review)
John Russell has posted comments on this change. ( http://gerrit.cloudera.org:8080/8511 ) Change subject: Update Impala docs for 2.10 release .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-6067: Enable s3 access via IAM roles for EC2 VMs

2017-11-13 Thread Laszlo Gaal (Code Review)
Laszlo Gaal has posted comments on this change. ( http://gerrit.cloudera.org:8080/8294 ) Change subject: IMPALA-6067: Enable s3 access via IAM roles for EC2 VMs .. Patch Set 3: (3 comments) > I also don't want to block

[Impala-ASF-CR](asf-site) Update Impala docs for 2.10 release

2017-11-13 Thread John Russell (Code Review)
Hello Bharath Vissapragada, Michael Brown, Laurel Hale, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8511 to look at the new patch set (#3). Change subject: Update Impala docs for 2.10 release

[Impala-ASF-CR] Remove unused/defunct Maven repositories.

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8497 ) Change subject: Remove unused/defunct Maven repositories. .. Patch Set 1: Verified-1 Build failed:

[Impala-ASF-CR] Expose $IMPALA MAVEN OPTIONS for configuring Maven.

2017-11-13 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/8496 ) Change subject: Expose $IMPALA_MAVEN_OPTIONS for configuring Maven. .. Patch Set 4: Code-Review+2 Rebase, carry + 2 -- To view,

[Impala-ASF-CR] Expose $IMPALA MAVEN OPTIONS for configuring Maven.

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8496 ) Change subject: Expose $IMPALA_MAVEN_OPTIONS for configuring Maven. .. Patch Set 4: Build started:

[Impala-ASF-CR] IMPALA-6067: Enable s3 access via IAM roles for EC2 VMs

2017-11-13 Thread Laszlo Gaal (Code Review)
Hello Lars Volker, Michael Brown, Jim Apple, Philip Zeyliger, Sailesh Mukil, David Knupp, Joe McDonnell, Tim Armstrong, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8294 to look at the new patch set (#3). Change subject: IMPALA-6067: Enable

[Impala-ASF-CR] Pin gen build version's git handling to typical git dir.

2017-11-13 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8500 ) Change subject: Pin gen_build_version's git handling to typical git dir. .. Patch Set 1: Verified-1 Build failed:

[Impala-ASF-CR] Pin gen build version's git handling to typical git dir.

2017-11-13 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. ( http://gerrit.cloudera.org:8080/8500 ) Change subject: Pin gen_build_version's git handling to typical git dir. .. Patch Set 1: > Sounds like

[Impala-ASF-CR](asf-site) Update Impala docs for 2.10 release

2017-11-13 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8511 ) Change subject: Update Impala docs for 2.10 release .. Patch Set 3: Code-Review+2 Thank you for doing this! -- To view, visit