[Impala-ASF-CR] Fix centos/Java/ORC default timezones in test-with-docker.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12137 ) Change subject: Fix centos/Java/ORC default timezones in test-with-docker. .. Patch Set 1: Build Successful

[Impala-ASF-CR] Fix centos/Java/ORC default timezones in test-with-docker.

2018-12-27 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12137 Change subject: Fix centos/Java/ORC default timezones in test-with-docker. .. Fix centos/Java/ORC default timezones in

[Impala-ASF-CR] IMPALA-6544: S3 tests fail because S3 is eventually consistent.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12127 ) Change subject: IMPALA-6544: S3 tests fail because S3 is eventually consistent. .. Patch Set 2: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6544: S3 tests fail because S3 is eventually consistent.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12127 ) Change subject: IMPALA-6544: S3 tests fail because S3 is eventually consistent. .. IMPALA-6544: S3 tests fail because

[Impala-ASF-CR] IMPALA-7992: Reduce iterations for test decimal fuzz.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12126 ) Change subject: IMPALA-7992: Reduce iterations for test_decimal_fuzz. .. IMPALA-7992: Reduce iterations for

[Impala-ASF-CR] IMPALA-7992: Reduce iterations for test decimal fuzz.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12126 ) Change subject: IMPALA-7992: Reduce iterations for test_decimal_fuzz. .. Patch Set 2: Verified+1 -- To view, visit

[Impala-ASF-CR] Fix some warnings on GCC7

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12132 ) Change subject: Fix some warnings on GCC7 .. Patch Set 2: Build Successful

[Impala-ASF-CR] IMPALA-7666: Adding an opaque client identifier to query options.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12130 ) Change subject: IMPALA-7666: Adding an opaque client identifier to query options. .. Patch Set 2: Build Successful

[Impala-ASF-CR] Fix some warnings on GCC7

2018-12-27 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12132 to look at the new patch set (#2). Change subject: Fix some warnings on GCC7 .. Fix some warnings on GCC7

[Impala-ASF-CR] IMPALA-7666: Adding an opaque client identifier to query options.

2018-12-27 Thread Philip Zeyliger (Code Review)
Hello Lars Volker, Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12130 to look at the new patch set (#2). Change subject: IMPALA-7666: Adding an opaque client identifier to query options.

[Impala-ASF-CR] IMPALA-7968, Part 1: JSON serialization framework

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12079 ) Change subject: IMPALA-7968, Part 1: JSON serialization framework .. Patch Set 7: Build Successful

[Impala-ASF-CR] IMPALA-7841 (Part 1): Refactor SelectStmt for easier debugging

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11915 ) Change subject: IMPALA-7841 (Part 1): Refactor SelectStmt for easier debugging .. Patch Set 2: Build Successful

[Impala-ASF-CR] IMPALA-7648: add tests for OOM from scanning big string

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12123 ) Change subject: IMPALA-7648: add tests for OOM from scanning big string .. IMPALA-7648: add tests for OOM from

[Impala-ASF-CR] IMPALA-7648: add tests for OOM from scanning big string

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12123 ) Change subject: IMPALA-7648: add tests for OOM from scanning big string .. Patch Set 2: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-7265: Add parameter to cache remote HDFS file handles

2018-12-27 Thread Joe McDonnell (Code Review)
Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12111 ) Change subject: IMPALA-7265: Add parameter to cache remote HDFS file handles .. IMPALA-7265: Add parameter to cache remote

[Impala-ASF-CR] IMPALA-7968, Part 1: JSON serialization framework

2018-12-27 Thread Paul Rogers (Code Review)
Hello Fredy Wijaya, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12079 to look at the new patch set (#7). Change subject: IMPALA-7968, Part 1: JSON serialization framework

[Impala-ASF-CR] IMPALA-6544: S3 tests fail because S3 is eventually consistent.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12127 ) Change subject: IMPALA-6544: S3 tests fail because S3 is eventually consistent. .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-7841 (Part 1): Refactor SelectStmt for easier debugging

2018-12-27 Thread Paul Rogers (Code Review)
Paul Rogers has posted comments on this change. ( http://gerrit.cloudera.org:8080/11915 ) Change subject: IMPALA-7841 (Part 1): Refactor SelectStmt for easier debugging .. Patch Set 2: Reduced the scope of the change for

[Impala-ASF-CR] IMPALA-7841 (Part 1): Refactor SelectStmt for easier debugging

2018-12-27 Thread Paul Rogers (Code Review)
Hello Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11915 to look at the new patch set (#2). Change subject: IMPALA-7841 (Part 1): Refactor SelectStmt for easier debugging

[Impala-ASF-CR] IMPALA-6544: S3 tests fail because S3 is eventually consistent.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12127 ) Change subject: IMPALA-6544: S3 tests fail because S3 is eventually consistent. .. Patch Set 2: Code-Review+2 -- To view,

[Impala-ASF-CR] IMPALA-7968, Part 1: JSON serialization framework

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12079 ) Change subject: IMPALA-7968, Part 1: JSON serialization framework .. Patch Set 6: Build Successful

[Impala-ASF-CR] IMPALA-6544: S3 tests fail because S3 is eventually consistent.

2018-12-27 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/12127 ) Change subject: IMPALA-6544: S3 tests fail because S3 is eventually consistent. .. Patch Set 1: Code-Review+2 It's reasonable to

[Impala-ASF-CR] Use Status::Expected for missing Char codegen.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12128 ) Change subject: Use Status::Expected for missing Char codegen. .. Patch Set 1: Verified+1 -- To view, visit

[Impala-ASF-CR] Use Status::Expected for missing Char codegen.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12128 ) Change subject: Use Status::Expected for missing Char codegen. .. Use Status::Expected for missing Char codegen.

[Impala-ASF-CR] IMPALA-7992: Reduce iterations for test decimal fuzz.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12126 ) Change subject: IMPALA-7992: Reduce iterations for test_decimal_fuzz. .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-7992: Reduce iterations for test decimal fuzz.

2018-12-27 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/12126 ) Change subject: IMPALA-7992: Reduce iterations for test_decimal_fuzz. .. Patch Set 2: Code-Review+2 (1 comment) carry after rebase

[Impala-ASF-CR] IMPALA-7968, Part 1: JSON serialization framework

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12079 ) Change subject: IMPALA-7968, Part 1: JSON serialization framework .. Patch Set 6: (2 comments)

[Impala-ASF-CR] IMPALA-7968, Part 1: JSON serialization framework

2018-12-27 Thread Paul Rogers (Code Review)
Hello Fredy Wijaya, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12079 to look at the new patch set (#6). Change subject: IMPALA-7968, Part 1: JSON serialization framework

[Impala-ASF-CR] IMPALA-5973: Provide query plan in JSON format

2018-12-27 Thread Paul Rogers (Code Review)
Paul Rogers has posted comments on this change. ( http://gerrit.cloudera.org:8080/11974 ) Change subject: IMPALA-5973: Provide query plan in JSON format .. Patch Set 3: Update on this one. Been working on a tricky planner

[Impala-ASF-CR] IMPALA-7265: Add parameter to cache remote HDFS file handles

2018-12-27 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/12111 ) Change subject: IMPALA-7265: Add parameter to cache remote HDFS file handles .. Patch Set 2: Code-Review+2 Thanks. -- To view,

[Impala-ASF-CR] IMPALA-7970 : Add support for metastore event based automatic invalidate

2018-12-27 Thread Paul Rogers (Code Review)
Paul Rogers has posted comments on this change. ( http://gerrit.cloudera.org:8080/12118 ) Change subject: IMPALA-7970 : Add support for metastore event based automatic invalidate .. Patch Set 9: (20 comments) Remaining

[Impala-ASF-CR] IMPALA-8016: Fix lifecycle of classloader for UDFs.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12125 ) Change subject: IMPALA-8016: Fix lifecycle of classloader for UDFs. .. Patch Set 4: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-8016: Fix lifecycle of classloader for UDFs.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12125 ) Change subject: IMPALA-8016: Fix lifecycle of classloader for UDFs. .. IMPALA-8016: Fix lifecycle of classloader for

[Impala-ASF-CR] IMPALA-7970 : Add support for metastore event based automatic invalidate

2018-12-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12118 ) Change subject: IMPALA-7970 : Add support for metastore event based automatic invalidate .. Patch Set 8: (1 comment)

[Impala-ASF-CR] IMPALA-7970 : Add support for metastore event based automatic invalidate

2018-12-27 Thread Paul Rogers (Code Review)
Paul Rogers has posted comments on this change. ( http://gerrit.cloudera.org:8080/12118 ) Change subject: IMPALA-7970 : Add support for metastore event based automatic invalidate .. Patch Set 9: (39 comments) This will be a

[Impala-ASF-CR] IMPALA-7648: add tests for OOM from scanning big string

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12123 ) Change subject: IMPALA-7648: add tests for OOM from scanning big string .. Patch Set 2: Build Successful

[Impala-ASF-CR] IMPALA-7867 (Part 4): Collection cleanup in catalog

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12131 ) Change subject: IMPALA-7867 (Part 4): Collection cleanup in catalog .. Patch Set 2: Build Successful

[Impala-ASF-CR] IMPALA-7867 (Part 4): Collection cleanup in catalog

2018-12-27 Thread Paul Rogers (Code Review)
Paul Rogers has posted comments on this change. ( http://gerrit.cloudera.org:8080/12131 ) Change subject: IMPALA-7867 (Part 4): Collection cleanup in catalog .. Patch Set 2: Pre-review tests passed:

[Impala-ASF-CR] IMPALA-7648: add tests for OOM from scanning big string

2018-12-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12123 ) Change subject: IMPALA-7648: add tests for OOM from scanning big string .. Patch Set 1: (2 comments)

[Impala-ASF-CR] IMPALA-7867 (Part 4): Collection cleanup in catalog

2018-12-27 Thread Paul Rogers (Code Review)
Paul Rogers has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12131 Change subject: IMPALA-7867 (Part 4): Collection cleanup in catalog .. IMPALA-7867 (Part 4): Collection cleanup in catalog Continues the

[Impala-ASF-CR] IMPALA-7648: add tests for OOM from scanning big string

2018-12-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12123 ) Change subject: IMPALA-7648: add tests for OOM from scanning big string .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7648: add tests for OOM from scanning big string

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12123 ) Change subject: IMPALA-7648: add tests for OOM from scanning big string .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-7648: add tests for OOM from scanning big string

2018-12-27 Thread Tim Armstrong (Code Review)
Hello Bikramjeet Vig, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12123 to look at the new patch set (#2). Change subject: IMPALA-7648: add tests for OOM from scanning big string

[Impala-ASF-CR] Use Status::Expected for missing Char codegen.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12128 ) Change subject: Use Status::Expected for missing Char codegen. .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-8016: Fix lifecycle of classloader for UDFs.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12125 ) Change subject: IMPALA-8016: Fix lifecycle of classloader for UDFs. .. Patch Set 4: Build Successful

[Impala-ASF-CR] IMPALA-8016: Fix lifecycle of classloader for UDFs.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12125 ) Change subject: IMPALA-8016: Fix lifecycle of classloader for UDFs. .. Patch Set 4: Build started:

[Impala-ASF-CR] IMPALA-8016: Fix lifecycle of classloader for UDFs.

2018-12-27 Thread Philip Zeyliger (Code Review)
Hello Bharath Vissapragada, Paul Rogers, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12125 to look at the new patch set (#4). Change subject: IMPALA-8016: Fix lifecycle of classloader for UDFs.

[Impala-ASF-CR] IMPALA-8016: Fix lifecycle of classloader for UDFs.

2018-12-27 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/12125 ) Change subject: IMPALA-8016: Fix lifecycle of classloader for UDFs. .. Patch Set 4: Code-Review+2 Carrying +2 from Bharath. -- To

[Impala-ASF-CR] IMPALA-8016: Fix lifecycle of classloader for UDFs.

2018-12-27 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/12125 ) Change subject: IMPALA-8016: Fix lifecycle of classloader for UDFs. .. Patch Set 3: (2 comments) Thanks for the reviews!

[Impala-ASF-CR] Adding an opaque client identifier to query options.

2018-12-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12130 ) Change subject: Adding an opaque client identifier to query options. .. Patch Set 1: (1 comment) I think this makes sense...

[Impala-ASF-CR] Use Status::Expected for missing Char codegen.

2018-12-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12128 ) Change subject: Use Status::Expected for missing Char codegen. .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-7905: Hive keywords not quoted for identifiers

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12009 ) Change subject: IMPALA-7905: Hive keywords not quoted for identifiers .. Patch Set 6: Build Successful

[Impala-ASF-CR] IMPALA-6742: Profiles of running queries should include execution summary.

2018-12-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11591 ) Change subject: IMPALA-6742: Profiles of running queries should include execution summary. .. Patch Set 4: Verified+1 -- To