[Impala-ASF-CR] IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork

Hello Michael Ho, Michael Brown, Jim Apple, Bikramjeet Vig, Dan Hecht, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7938 to look at the new patch set (#13). Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive

[Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE

Attila Jeges has uploaded a new patch set (#13). ( http://gerrit.cloudera.org:8080/7564 ) Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE .. IMPALA-3548: Prune runtime filters based on query

[Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE

Attila Jeges has posted comments on this change. ( http://gerrit.cloudera.org:8080/7564 ) Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE .. Patch Set 13: > Uploaded patch set 13: Patch

[Impala-ASF-CR] IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork

Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/7938 ) Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork .. Patch Set 13: (1 comment)

[Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/7564 ) Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE .. Patch Set 13: Build started:

[Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/7564 ) Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE .. Patch Set 13: Verified+1 -- To view,

[Impala-ASF-CR] IMPALA-5607: Add additional units to EXTRACT, DATE PART, TRUNC

Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8311 ) Change subject: IMPALA-5607: Add additional units to EXTRACT, DATE_PART, TRUNC .. Patch Set 2: (6 comments) Overall this looks good -

[Impala-ASF-CR] IMPALA-5307: Part 2: copy out strings in uncompressed Avro

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8146 ) Change subject: IMPALA-5307: Part 2: copy out strings in uncompressed Avro .. Patch Set 13: (1 comment)

[Impala-ASF-CR] IMPALA-5017: Error on decimal overflow

Taras Bobrovytsky has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8404 Change subject: IMPALA-5017: Error on decimal overflow .. IMPALA-5017: Error on decimal overflow Before this patch, decimal

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 5: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 11: (7 comments)

[Impala-ASF-CR] IMPALA-2494: Support for byte array encoded decimals in Parquet scanner

Hello Lars Volker, Matthew Jacobs, Tim Armstrong, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7822 to look at the new patch set (#7). Change subject: IMPALA-2494: Support for byte array encoded decimals in Parquet scanner

[Impala-ASF-CR] IMPALA-2494: Support for byte array encoded decimals in Parquet scanner

Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/7822 ) Change subject: IMPALA-2494: Support for byte array encoded decimals in Parquet scanner .. Patch Set 6: (7 comments)

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 5: (4 comments)

[Impala-ASF-CR] IMPALA-5607: Add additional units to EXTRACT, DATE PART, TRUNC

Zach Amsden has posted comments on this change. ( http://gerrit.cloudera.org:8080/8311 ) Change subject: IMPALA-5607: Add additional units to EXTRACT, DATE_PART, TRUNC .. Patch Set 2: Looks like we want NANOSECONDS after all,

[Impala-ASF-CR] IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/7938 ) Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork .. Patch Set 14: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork

Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/7938 ) Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork .. IMPALA-5129: Use Kudu's Kinit code to

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

Hello Thomas Tauber-Marshall, Tim Armstrong, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8226 to look at the new patch set (#5). Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 5: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 5: Build started:

[Impala-ASF-CR] IMPALA-1291: Parquet read fails if io buffer size is less than the footer size

Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8366 to look at the new patch set (#3). Change subject: IMPALA-1291: Parquet read fails if io buffer size is less than the footer size

[Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE

Alex Behm has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/7564 ) Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE .. IMPALA-3548: Prune runtime filters based on

[Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE

Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/7564 ) Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE .. Patch Set 13: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-5017: Error on decimal overflow

Taras Bobrovytsky has posted comments on this change. ( http://gerrit.cloudera.org:8080/8404 ) Change subject: IMPALA-5017: Error on decimal overflow .. Patch Set 1: Ooops, canceled -- To view, visit

[Impala-ASF-CR] IMPALA-5017: Error on decimal overflow

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8404 ) Change subject: IMPALA-5017: Error on decimal overflow .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

Hello Sailesh Mukil, Joe McDonnell, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8303 to look at the new patch set (#12). Change subject: IMPALA-1575: Part 1: eagerly release query exec resources

[Impala-ASF-CR] IMPALA-5307: Part 2: copy out strings in uncompressed Avro

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8146 ) Change subject: IMPALA-5307: Part 2: copy out strings in uncompressed Avro .. Patch Set 13: (1 comment)

[Impala-ASF-CR] IMPALA-5307: Part 2: copy out strings in uncompressed Avro

Hello Michael Ho, Thomas Tauber-Marshall, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8146 to look at the new patch set (#14). Change subject: IMPALA-5307: Part 2: copy out strings in uncompressed Avro

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

Hello Sailesh Mukil, Joe McDonnell, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8303 to look at the new patch set (#13). Change subject: IMPALA-1575: Part 1: eagerly release query exec resources

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 11: (8 comments)

[Impala-ASF-CR] IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork

Jim Apple has posted comments on this change. ( http://gerrit.cloudera.org:8080/7938 ) Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork .. Patch Set 13: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-1291: Parquet read fails if io buffer size is less than the footer size

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8366 ) Change subject: IMPALA-1291: Parquet read fails if io buffer size is less than the footer size .. Patch Set 4: Code-Review+2 -- To

[Impala-ASF-CR] IMPALA-1291: Parquet read fails if io buffer size is less than the footer size

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8366 ) Change subject: IMPALA-1291: Parquet read fails if io buffer size is less than the footer size .. Patch Set 3: Code-Review+2 -- To

[Impala-ASF-CR] IMPALA-1291: Parquet read fails if io buffer size is less than the footer size

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8366 ) Change subject: IMPALA-1291: Parquet read fails if io buffer size is less than the footer size .. Patch Set 4: Build started:

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

Hello Thomas Tauber-Marshall, Tim Armstrong, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8226 to look at the new patch set (#4). Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 4: (2 comments)

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

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 2: (7 comments) Thanks for tackling this! As a

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

Tim Armstrong 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 2: (5 comments) Thanks for doing this

[Impala-ASF-CR] IMPALA-5429: Multi threaded block metadata loading

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8235 ) Change subject: IMPALA-5429: Multi threaded block metadata loading .. Patch Set 11: Build started:

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 10: (4 comments) Just have a few minor

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 3: Code-Review+1 (2 comments) Change looks good to me -

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 4: Tim, can you do the +2 review for this one? -- To view,

[Impala-ASF-CR] IMPALA-5429: Multi threaded block metadata loading

Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/8235 ) Change subject: IMPALA-5429: Multi threaded block metadata loading .. Patch Set 11: Code-Review+2 Rebased, carrying +2. Thanks

[Impala-ASF-CR] IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/7938 ) Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork .. Patch Set 13: Build started:

[Impala-ASF-CR] IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/7938 ) Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork .. Patch Set 13: Verified-1 Build failed:

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

Zoltan Borok-Nagy has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8368 Change subject: IMPALA-2235: Fix current db when shell auto-reconnects .. IMPALA-2235: Fix current db when shell auto-reconnects

[Impala-ASF-CR] IMPALA-5473: [DOCS] Document TLS min version & cipher options

John Russell has posted comments on this change. ( http://gerrit.cloudera.org:8080/8401 ) Change subject: IMPALA-5473: [DOCS] Document TLS min version & cipher options .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-1291: Parquet read fails if io buffer size is less than the footer size

Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8366 ) Change subject: IMPALA-1291: Parquet read fails if io buffer size is less than the footer size .. IMPALA-1291: Parquet

[Impala-ASF-CR] IMPALA-1291: Parquet read fails if io buffer size is less than the footer size

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8366 ) Change subject: IMPALA-1291: Parquet read fails if io buffer size is less than the footer size .. Patch Set 4: Verified+1 --

[Impala-ASF-CR] IMPALA-5473: [DOCS] Document TLS min version & cipher options

Michael Brown has posted comments on this change. ( http://gerrit.cloudera.org:8080/8401 ) Change subject: IMPALA-5473: [DOCS] Document TLS min version & cipher options .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-5473: [DOCS] Document TLS min version & cipher options

John Russell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8401 Change subject: IMPALA-5473: [DOCS] Document TLS min version & cipher options .. IMPALA-5473: [DOCS] Document TLS min version & cipher

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

Hello Michael Ho, Lars Volker, Matthew Jacobs, Anonymous Coward #345, Tim Armstrong, Todd Lipcon, Mostafa Mokhtar, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7793 to look at the new patch set (#8). Change subject: IMPALA-4252: Min-max runtime filters

[Impala-ASF-CR] IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork

Hello Michael Ho, Michael Brown, Jim Apple, Bikramjeet Vig, Dan Hecht, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7938 to look at the new patch set (#14). Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive

[Impala-ASF-CR] IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/7938 ) Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork .. Patch Set 14: Build started:

[Impala-ASF-CR] IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork

Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/7938 ) Change subject: IMPALA-5129: Use Kudu's Kinit code to avoid expensive fork .. Patch Set 14: Code-Review+2 Fix minor clang-tidy issues.

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

Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 7: I know that He Lifu at NetEase also prototyped some

[Impala-ASF-CR] IMPALA-5473: [DOCS] Document TLS min version & cipher options

John Russell has posted comments on this change. ( http://gerrit.cloudera.org:8080/8401 ) Change subject: IMPALA-5473: [DOCS] Document TLS min version & cipher options .. Patch Set 1: I added a couple of questions under

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

Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 8: (10 comments) > I noticed that -

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

Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/7793 ) Change subject: IMPALA-4252: Min-max runtime filters for Kudu .. Patch Set 8: > > Patch Set 7: > > > > > > Patch Set 7: >

[Impala-ASF-CR] IMPALA-5429: Multi threaded block metadata loading

Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8235 ) Change subject: IMPALA-5429: Multi threaded block metadata loading .. Patch Set 11: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-5429: Multi threaded block metadata loading

Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8235 ) Change subject: IMPALA-5429: Multi threaded block metadata loading .. IMPALA-5429: Multi threaded block metadata

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

Hello Sailesh Mukil, Joe McDonnell, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8303 to look at the new patch set (#11). Change subject: IMPALA-1575: Part 1: eagerly release query exec resources

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 11: Code-Review+1 (4 comments) Thanks for the

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 10: (1 comment)

[Impala-ASF-CR] IMPALA-5473: [DOCS] Document TLS min version & cipher options

Bharath Vissapragada has posted comments on this change. ( http://gerrit.cloudera.org:8080/8401 ) Change subject: IMPALA-5473: [DOCS] Document TLS min version & cipher options .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 10: (1 comment)

[Impala-ASF-CR] IMPALA-2758: Remove BufferedTupleStream::GetRows

Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 4: Code-Review+2 (4 comments) Looks good, just did a

[Impala-ASF-CR] IMPALA-4236: Codegen CopyRows() for select nodes

Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/8196 ) Change subject: IMPALA-4236: Codegen CopyRows() for select nodes .. Patch Set 11: (1 comment)

[Impala-ASF-CR] IMPALA-1575: Part 1: eagerly release query exec resources

Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. Patch Set 11: Code-Review+1 (1 comment) Thanks for the