[Impala-CR] IMPALA-2878: Fix Base64Decode error and remove duplicate codes.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-2878: Fix Base64Decode error and remove duplicate codes. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4209 To unsubscribe, visit

[Impala-CR] IMPALA-3567: Part 1: groundwork to make Join build sides DataSinks

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3567: Part 1: groundwork to make Join build sides DataSinks .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4212 To unsubscribe, visit

[Impala-CR] IMPALA-3881: Add DataTables 1.10.12 to www/

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3881: Add DataTables 1.10.12 to www/ .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4227 To unsubscribe, visit

[Impala-CR] IMPALA-2347: Reuse metastore client connections in Catalog

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-2347: Reuse metastore client connections in Catalog .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4229 To unsubscribe, visit

[Impala-CR] IMPALA-3952: Clear scratch batch mem pool if Open() failed.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3952: Clear scratch batch mem pool if Open() failed. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4236 To unsubscribe, visit

[Impala-CR] IMPALA-1619, IMPALA-3018: Address various small memory allocation related bugs

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-1619, IMPALA-3018: Address various small memory allocation related bugs .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4235 To

[Impala-CR] IMPALA-3964: Fix crash when a count(*) is performed on a nested collection.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3964: Fix crash when a count(*) is performed on a nested collection. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4237 To

[Impala-CR] IMPALA-3843: Update warning for non-SSSE3 CPUs

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3843: Update warning for non-SSSE3 CPUs .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4218 To unsubscribe, visit

[Impala-CR] Fix stress test runner bug introduced by IMPALA-3969

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: Fix stress test runner bug introduced by IMPALA-3969 .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4243 To unsubscribe, visit

[Impala-CR] IMPALA-3992: bad shell error message when running nonexistent file

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3992: bad shell error message when running nonexistent file .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4240 To unsubscribe, visit

[Impala-CR] IMPALA-3856,IMPALA-3871: Fix BinaryPredicate normalization for Kudu

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3856,IMPALA-3871: Fix BinaryPredicate normalization for Kudu .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4241 To unsubscribe, visit

[Impala-CR] IMPALA-3376: Extra definition level when writing Parquet files

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3376: Extra definition level when writing Parquet files .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4207 To unsubscribe, visit

[Impala-CR] IMPALA-3918: Remove Cloudera copyrights and add ASF license header

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3918: Remove Cloudera copyrights and add ASF license header .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4213 To unsubscribe, visit

[Impala-CR] IMPALA-3924: Ubuntu16 support

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3924: Ubuntu16 support .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4216 To unsubscribe, visit

[Impala-CR] IMPALA-3790: Fix admission control flaky stress test

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3790: Fix admission control flaky stress test .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4210 To unsubscribe, visit

[Impala-CR] IMPALA-3946: fix MemPool integrity issues with empty chunks

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3946: fix MemPool integrity issues with empty chunks .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4222 To unsubscribe, visit

[Impala-CR] IMPALA-3918: Remove Cloudera copyrights and add ASF license header

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3918: Remove Cloudera copyrights and add ASF license header .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4211 To unsubscribe, visit

[Impala-CR] IMPALA-3611: track unused Disk IO buffer memory

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3611: track unused Disk IO buffer memory .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4208 To unsubscribe, visit

[Impala-CR] IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4224 To unsubscribe, visit

[Impala-CR] IMPALA-2700: ASCII NUL characters are doubled on insert into text tables

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-2700: ASCII NUL characters are doubled on insert into text tables .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4215 To unsubscribe,

[Impala-CR] IMPALA-1112: Remove some unncessary code from cross-compilation

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-1112: Remove some unncessary code from cross-compilation .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4217 To unsubscribe, visit

[Impala-CR] IMPALA-3965: TSSLSocketWithWildcardSAN.py not exported as part of impala-shell build lib

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3965: TSSLSocketWithWildcardSAN.py not exported as part of impala-shell build lib .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4214

[Impala-CR] IMPALA-3786: Remove "Cloudera" from impalad webpage title

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3786: Remove "Cloudera" from impalad webpage title .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4219 To unsubscribe, visit

[Impala-CR] IMPALA-3957: Test failure in S3 build: TestLoadData.test load

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3957: Test failure in S3 build: TestLoadData.test_load .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4221 To unsubscribe, visit

[Impala-CR] IMPALA-3940: Fix getting column stats through views.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3940: Fix getting column stats through views. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4223 To unsubscribe, visit

[Impala-CR] IMPALA-3201: buffer pool header only

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3201: buffer pool header only .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4233 To unsubscribe, visit

[Impala-CR] Remove dead and untested code

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: Remove dead and untested code .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4232 To unsubscribe, visit

[Impala-CR] IMPALA-3470: DecompressorTest is flaky.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3470: DecompressorTest is flaky. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4234 To unsubscribe, visit

[Impala-CR] IMPALA-3090: always log memory limit errors

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3090: always log memory limit errors .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4245 To unsubscribe, visit

[Impala-CR] Add functional and targeted perf tests for joins with empty builds

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: Add functional and targeted perf tests for joins with empty builds .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4247 To unsubscribe, visit

[Impala-CR] IMPALA-3828: Enable inversion for inner joins.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3828: Enable inversion for inner joins. .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4246 To unsubscribe, visit

[Impala-CR] IMPALA-3962: Clean up scratch tuple batch on scan failures

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3962: Clean up scratch tuple batch on scan failures .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4248 To unsubscribe, visit

[Impala-CR] IMPALA-2988: Refactor HdfsTableSink::Close() so that it cannot fail

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-2988: Refactor HdfsTableSink::Close() so that it cannot fail .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4249 To unsubscribe, visit

[Impala-CR] IMPALA-3662: Don't double allocate tuples buffer in parquet scanner

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3662: Don't double allocate tuples buffer in parquet scanner .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4250 To unsubscribe, visit

[Impala-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4251 To unsubscribe,

[Impala-CR] IMPALA-3996: Migrate to updated Kudu insert string API

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3996: Migrate to updated Kudu insert string API .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4252 To unsubscribe, visit

[Impala-CR] IMPALA-4004: Don't access nested types in test failpoints.py

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-4004: Don't access nested types in test_failpoints.py .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4253 To unsubscribe, visit

[Impala-CR] IMPALA-3988: Only use first 96 bits of query id

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3988: Only use first 96 bits of query id .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4254 To unsubscribe, visit

[Impala-CR] IMPALA-3470: DecompressorTest is flaky.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Juan Yu, Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4234 to review the following change. Change subject: IMPALA-3470: DecompressorTest is flaky. .. IMPALA-3470:

[Impala-CR] IMPALA-3662: Don't double allocate tuples buffer in parquet scanner

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4250 to review the following change. Change subject: IMPALA-3662: Don't double allocate tuples buffer in parquet scanner

[Impala-CR] IMPALA-3940: Fix getting column stats through views.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Alex Behm, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4223 to review the following change. Change subject: IMPALA-3940: Fix getting column stats through views.

[Impala-CR] Add functional and targeted perf tests for joins with empty builds

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4247 to review the following change. Change subject: Add functional and targeted perf tests for joins with empty builds

[Impala-CR] IMPALA-2988: Refactor HdfsTableSink::Close() so that it cannot fail

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Dan Hecht, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4249 to review the following change. Change subject: IMPALA-2988: Refactor HdfsTableSink::Close() so that it cannot fail

[Impala-CR] IMPALA-3957: Test failure in S3 build: TestLoadData.test load

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Sailesh Mukil, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4221 to review the following change. Change subject: IMPALA-3957: Test failure in S3 build: TestLoadData.test_load

[Impala-CR] IMPALA-3843: Update warning for non-SSSE3 CPUs

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4218 to review the following change. Change subject: IMPALA-3843: Update warning for non-SSSE3 CPUs ..

[Impala-CR] IMPALA-3964: Fix crash when a count(*) is performed on a nested collection.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Internal Jenkins, Dan Hecht, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4237 to review the following change. Change subject: IMPALA-3964: Fix crash when a count(*) is performed on a nested collection.

[Impala-CR] IMPALA-3567: Part 1: groundwork to make Join build sides DataSinks

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Marcel Kornacker, Internal Jenkins, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4212 to review the following change. Change subject: IMPALA-3567: Part 1: groundwork to make Join build sides DataSinks

[Impala-CR] IMPALA-3996: Migrate to updated Kudu insert string API

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Matthew Jacobs, Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4252 to review the following change. Change subject: IMPALA-3996: Migrate to updated Kudu insert string API

[Impala-CR] IMPALA-3650: DISTRIBUTE BY required for managed Kudu tables

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Matthew Jacobs, Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4244 to review the following change. Change subject: IMPALA-3650: DISTRIBUTE BY required for managed Kudu tables

[Impala-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4251 to review the following change. Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

[Impala-CR] IMPALA-3972: Improve display of /varz page

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Henry Robinson, Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4228 to review the following change. Change subject: IMPALA-3972: Improve display of /varz page ..

[Impala-CR] Fix stress test runner bug introduced by IMPALA-3969

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Brown, Internal Jenkins, Dan Hecht, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4243 to review the following change. Change subject: Fix stress test runner bug introduced by IMPALA-3969

[Impala-CR] IMPALA-3992: bad shell error message when running nonexistent file

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Henry Robinson, Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4240 to review the following change. Change subject: IMPALA-3992: bad shell error message when running nonexistent file

[Impala-CR] IMPALA-3988: Only use first 96 bits of query id

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Marcel Kornacker, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4254 to review the following change. Change subject: IMPALA-3988: Only use first 96 bits of query id .. IMPALA-3988:

[Impala-CR] Remove dead and untested code

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Dan Hecht, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4232 to review the following change. Change subject: Remove dead and untested code .. Remove dead and

[Impala-CR] IMPALA-4004: Don't access nested types in test failpoints.py

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Alex Behm, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4253 to review the following change. Change subject: IMPALA-4004: Don't access nested types in test_failpoints.py

[Impala-CR] IMPALA-3828: Enable inversion for inner joins.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Alex Behm, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4246 to review the following change. Change subject: IMPALA-3828: Enable inversion for inner joins. ..

[Impala-CR] IMPALA-1619, IMPALA-3018: Address various small memory allocation related bugs

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Dan Hecht, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4235 to review the following change. Change subject: IMPALA-1619, IMPALA-3018: Address various small memory allocation related bugs

[Impala-CR] IMPALA-3856,IMPALA-3871: Fix BinaryPredicate normalization for Kudu

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Matthew Jacobs, Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4241 to review the following change. Change subject: IMPALA-3856,IMPALA-3871: Fix BinaryPredicate normalization for Kudu

[Impala-CR] IMPALA-3952: Clear scratch batch mem pool if Open() failed.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Alex Behm, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4236 to review the following change. Change subject: IMPALA-3952: Clear scratch batch mem pool if Open() failed.

[Impala-CR] IMPALA-3923: fix overflow in BufferedTupleStream::GetRows()

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4220 to review the following change. Change subject: IMPALA-3923: fix overflow in BufferedTupleStream::GetRows()

[Impala-CR] IMPALA-3918: Remove Cloudera copyrights and add ASF license header

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Dan Hecht, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4211 to review the following change. Change subject: IMPALA-3918: Remove Cloudera copyrights and add ASF license header

[Impala-CR] IMPALA-3953: Fixes for KuduScanNode BE test failure

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Dan Hecht, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4231 to review the following change. Change subject: IMPALA-3953: Fixes for KuduScanNode BE test failure

[Impala-CR] IMPALA-3376: Extra definition level when writing Parquet files

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Matthew Jacobs, Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4207 to review the following change. Change subject: IMPALA-3376: Extra definition level when writing Parquet files

[Impala-CR] IMPALA-3969: stress test: add option to set common query options

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello David Knupp, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4225 to review the following change. Change subject: IMPALA-3969: stress test: add option to set common query options

[Impala-CR] IMPALA-3063: Separate join inversion from join ordering.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Alex Behm, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4239 to review the following change. Change subject: IMPALA-3063: Separate join inversion from join ordering.

[Impala-CR] IMPALA-1112: Remove some unncessary code from cross-compilation

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4217 to review the following change. Change subject: IMPALA-1112: Remove some unncessary code from cross-compilation

[Impala-CR] IMPALA-3924: Ubuntu16 support

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Matthew Jacobs, Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4216 to review the following change. Change subject: IMPALA-3924: Ubuntu16 support .. IMPALA-3924:

[Impala-CR] IMPALA-3918: remove Cloudera copyright from the shell welcome message

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4238 to review the following change. Change subject: IMPALA-3918: remove Cloudera copyright from the shell welcome message

[Impala-CR] IMPALA-3881: Add DataTables 1.10.12 to www/

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4227 to review the following change. Change subject: IMPALA-3881: Add DataTables 1.10.12 to www/ ..

[Impala-CR] IMPALA-2700: ASCII NUL characters are doubled on insert into text tables

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4215 to review the following change. Change subject: IMPALA-2700: ASCII NUL characters are doubled on insert into text tables

[Impala-CR] IMPALA-3918: Remove Cloudera copyrights and add ASF license header

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Dan Hecht, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4213 to review the following change. Change subject: IMPALA-3918: Remove Cloudera copyrights and add ASF license header

[Impala-CR] IMPALA-2347: Reuse metastore client connections in Catalog

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Bharath Vissapragada, Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4229 to review the following change. Change subject: IMPALA-2347: Reuse metastore client connections in Catalog

[Impala-CR] IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4224 to review the following change. Change subject: IMPALA-3764,3914: fuzz test HDFS scanners and fix parquet bugs found

[Impala-CR] IMPALA-3829: OpenSession() logs errors on valid configuration keys

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4226 to review the following change. Change subject: IMPALA-3829: OpenSession() logs errors on valid configuration keys

[Impala-CR] IMPALA-3201: buffer pool header only

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4233 to review the following change. Change subject: IMPALA-3201: buffer pool header only ..

[Impala-CR] IMPALA-2878: Fix Base64Decode error and remove duplicate codes.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Jim Apple, Yuanhao Luo, Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4209 to review the following change. Change subject: IMPALA-2878: Fix Base64Decode error and remove duplicate codes.

[Impala-CR] IMPALA-3946: fix MemPool integrity issues with empty chunks

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4222 to review the following change. Change subject: IMPALA-3946: fix MemPool integrity issues with empty chunks

[Impala-CR] IMPALA-3936: BufferedBlockMgr fixes for Pin() while write in flight.

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Tim Armstrong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4230 to review the following change. Change subject: IMPALA-3936: BufferedBlockMgr fixes for Pin() while write in flight.

[Impala-CR] IMPALA-3611: track unused Disk IO buffer memory

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Hello Internal Jenkins, Dan Hecht, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4208 to review the following change. Change subject: IMPALA-3611: track unused Disk IO buffer memory ..

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 2)

2016-09-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 2) .. Patch Set 4: Currently, the on demand packaging build won't run because there are no sles64

[Impala-ASF-CR] IMPALA-3725 Support Kudu UPSERT in Impala

2016-08-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-3725 Support Kudu UPSERT in Impala .. IMPALA-3725 Support Kudu UPSERT in Impala This patch introduces a new query statement, UPSERT, for Kudu

[Impala-ASF-CR] IMPALA-3725 Support Kudu UPSERT in Impala

2016-08-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3725 Support Kudu UPSERT in Impala .. Patch Set 1: (9 comments) http://gerrit.cloudera.org:8080/#/c/4047/1//COMMIT_MSG Commit Message: PS1,

[Impala-ASF-CR] Bump Impala version to 2.8

2016-08-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Impala version to 2.8 .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4170 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 2)

2016-08-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 2) .. Patch Set 4: I just rebased both patches, and the new version also passes a private jenkins

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 2)

2016-08-30 Thread Thomas Tauber-Marshall (Code Review)
Hello Jim Apple, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3937 to look at the new patch set (#4). Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 2) ..

[Impala-ASF-CR] IMPALA-3786: Replace "cloudera" with "apache" (part 1)

2016-08-30 Thread Thomas Tauber-Marshall (Code Review)
Hello Jim Apple, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3936 to look at the new patch set (#3). Change subject: IMPALA-3786: Replace "cloudera" with "apache" (part 1) ..

[Impala-ASF-CR] IMPALA-3725 Support Kudu UPSERT in Impala

2016-08-26 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-3725 Support Kudu UPSERT in Impala .. IMPALA-3725 Support Kudu UPSERT in Impala This patch introduces a new query statement, UPSERT, for Kudu

[Impala-ASF-CR] IMPALA-2932: Extend DistributedPlanner to account for hash table build cost

2016-08-26 Thread Thomas Tauber-Marshall (Code Review)
Hello Marcel Kornacker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4098 to look at the new patch set (#4). Change subject: IMPALA-2932: Extend DistributedPlanner to account for hash table build cost

[Impala-ASF-CR] IMPALA-4009: qgen: add docs, utility for installing Oracle as ref database

2016-08-24 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4009: qgen: add docs, utility for installing Oracle as ref database .. Patch Set 3: Code-Review+1 Looks good. -- To view, visit

[Impala-ASF-CR] IMPALA-2932: Extend DistributedPlanner to account for hash table build cost

2016-08-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-2932: Extend DistributedPlanner to account for hash table build cost .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-2932: Extend DistributedPlanner to account for hash table build cost

2016-08-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-2932: Extend DistributedPlanner to account for hash table build cost .. IMPALA-2932: Extend DistributedPlanner to account for hash table build

[Impala-ASF-CR] IMPALA-4009: qgen: add documentation for installing Oracle as ref database

2016-08-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4009: qgen: add documentation for installing Oracle as ref database .. Patch Set 1: (1 comment) I performed the steps (with the docker

[Impala-ASF-CR] IMPALA-2932: Extend DistributedPlanner to account for hash table build cost

2016-08-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-2932: Extend DistributedPlanner to account for hash table build cost .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-2932: Extend DistributedPlanner to account for hash table build cost

2016-08-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-2932: Extend DistributedPlanner to account for hash table build cost .. IMPALA-2932: Extend DistributedPlanner to account for hash table build

[Impala-ASF-CR] IMPALA-2932: Extend DistributedPlanner to account for hash table build cost

2016-08-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/4098 Change subject: IMPALA-2932: Extend DistributedPlanner to account for hash table build cost .. IMPALA-2932: Extend

[Impala-ASF-CR] IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

2016-08-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner .. Patch Set 1: Code-Review+1 Replaces: https://gerrit.cloudera.org/#/c/3774/ --

[Impala-CR](cdh5-trunk) IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

2016-08-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has abandoned this change. Change subject: IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner .. Abandoned Replaced with: https://gerrit.cloudera.org/#/c/4093/ -- To view, visit

[Impala-ASF-CR] IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

2016-08-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/4093 Change subject: IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner .. IMPALA-3629: Codegen

[Impala-CR](cdh5-trunk) IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner

2016-08-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3629: Codegen TransferScratchTuples() in hdfs-parquet-scanner .. Patch Set 10: (3 comments)

  1   2   3   4   >