[Impala-ASF-CR] IMPALA-6118: Fix assertion failure in coordinator bloom filter updating

2017-10-27 Thread Tianyi Wang (Code Review)
Tianyi Wang has abandoned this change. ( http://gerrit.cloudera.org:8080/8410 ) Change subject: IMPALA-6118: Fix assertion failure in coordinator bloom filter updating .. Abandoned -- To view, visit

[Impala-ASF-CR] IMPALA-6118: Fix assertion failure in coordinator bloom filter updating

2017-10-27 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/8410 ) Change subject: IMPALA-6118: Fix assertion failure in coordinator bloom filter updating .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-6118: Fix assertion failure in coordinator bloom filter updating

2017-10-27 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/8410 ) Change subject: IMPALA-6118: Fix assertion failure in coordinator bloom filter updating .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-6118: Fix assertion failure in coordinator bloom filter updating

2017-10-27 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8410 ) Change subject: IMPALA-6118: Fix assertion failure in coordinator bloom filter updating .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-6118: Fix assertion failure in coordinator bloom filter updating

2017-10-27 Thread Tianyi Wang (Code Review)
Tianyi Wang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8410 Change subject: IMPALA-6118: Fix assertion failure in coordinator bloom filter updating .. IMPALA-6118: Fix assertion failure in

[Impala-ASF-CR] IMPALA-6123: Fix column order of a query test in test inline view limit

2017-10-27 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8409 ) Change subject: IMPALA-6123: Fix column order of a query test in test_inline_view_limit .. Patch Set 2: Any idea how this test succeeded

[Impala-ASF-CR] IMPALA-6123: Fix column order of a query test in test inline view limit

2017-10-27 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8409 ) Change subject: IMPALA-6123: Fix column order of a query test in test_inline_view_limit .. Patch Set 2: Actually nvm, I see it only fails

[Impala-ASF-CR] IMPALA-6108, IMPALA-6070: Parallel data load (re-instated).

2017-10-27 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/8405 ) Change subject: IMPALA-6108, IMPALA-6070: Parallel data load (re-instated). .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-6108, IMPALA-6070: Parallel data load (re-instated).

2017-10-27 Thread Philip Zeyliger (Code Review)
Hello Joe McDonnell, Tim Armstrong, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8405 to look at the new patch set (#3). Change subject: IMPALA-6108, IMPALA-6070: Parallel data load (re-instated).

[Impala-ASF-CR] IMPALA-5243: Speed up code gen for wide Avro tables.

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8211 ) Change subject: IMPALA-5243: Speed up code gen for wide Avro tables. .. Patch Set 8: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-5243: Speed up code gen for wide Avro tables.

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8211 ) Change subject: IMPALA-5243: Speed up code gen for wide Avro tables. .. IMPALA-5243: Speed up code gen for wide Avro

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

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8303 ) Change subject: IMPALA-1575: Part 1: eagerly release query exec resources .. IMPALA-1575: Part 1: eagerly release query

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

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins 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 16: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6123: Fix column order of a query test in test inline view limit

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8409 ) Change subject: IMPALA-6123: Fix column order of a query test in test_inline_view_limit .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-6123: Fix column order of a query test in test inline view limit

2017-10-27 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/8409 ) Change subject: IMPALA-6123: Fix column order of a query test in test_inline_view_limit .. Patch Set 2: Code-Review+2 I'll +2 this to

[Impala-ASF-CR] IMPALA-6123: Fix column order of a query test in test inline view limit

2017-10-27 Thread Bikramjeet Vig (Code Review)
Hello Lars Volker, Tim Armstrong, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8409 to look at the new patch set (#2). Change subject: IMPALA-6123: Fix column order of a query test in test_inline_view_limit

[Impala-ASF-CR] IMPALA-6123: Fix column order of a query test in test inline view limit

2017-10-27 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8409 ) Change subject: IMPALA-6123: Fix column order of a query test in test_inline_view_limit .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-6093: diagnostics for flaky TestHashJoinTimer

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8403 ) Change subject: IMPALA-6093: diagnostics for flaky TestHashJoinTimer .. IMPALA-6093: diagnostics for flaky

[Impala-ASF-CR] IMPALA-6093: diagnostics for flaky TestHashJoinTimer

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8403 ) Change subject: IMPALA-6093: diagnostics for flaky TestHashJoinTimer .. Patch Set 3: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6123: Fix column order of a query test in test inline view limit

2017-10-27 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/8409 ) Change subject: IMPALA-6123: Fix column order of a query test in test_inline_view_limit .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-6123: Fix column order of a query test in test inline view limit

2017-10-27 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8409 Change subject: IMPALA-6123: Fix column order of a query test in test_inline_view_limit .. IMPALA-6123: Fix column order of a query

[Impala-ASF-CR] IMPALA-6108, IMPALA-6070: Parallel data load (re-instated).

2017-10-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8405 ) Change subject: IMPALA-6108, IMPALA-6070: Parallel data load (re-instated). .. Patch Set 2: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-5019: Decimal V2 addition

2017-10-27 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/8309 ) Change subject: IMPALA-5019: Decimal V2 addition .. IMPALA-5019: Decimal V2 addition In this patch, we implement the new decimal

[Impala-ASF-CR] IMPALA-5019: Decimal V2 addition

2017-10-27 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. ( http://gerrit.cloudera.org:8080/8309 ) Change subject: IMPALA-5019: Decimal V2 addition .. Patch Set 2: (12 comments)

[Impala-ASF-CR] IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder

2017-10-27 Thread Joe McDonnell (Code Review)
Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/8034 ) Change subject: IMPALA-5522:Use tracked memory for DictDecoder and DictEncoder .. Patch Set 12: (5 comments) I think this is very

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

2017-10-27 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 3: (1 comment)

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

2017-10-27 Thread John Russell (Code Review)
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 2: (2 comments) I'll punt to Sailesh for

[Impala-ASF-CR] IMPALA-5243: Speed up code gen for wide Avro tables.

2017-10-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8211 ) Change subject: IMPALA-5243: Speed up code gen for wide Avro tables. .. Patch Set 8: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-5243: Speed up code gen for wide Avro tables.

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8211 ) Change subject: IMPALA-5243: Speed up code gen for wide Avro tables. .. Patch Set 8: Build started:

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

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins 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 16: Build started:

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

2017-10-27 Thread Tim Armstrong (Code Review)
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 16: Code-Review+2 carry -- To view, visit

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

2017-10-27 Thread Tim Armstrong (Code Review)
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 (#15). Change subject: IMPALA-1575: Part 1: eagerly release query exec resources

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

2017-10-27 Thread Tim Armstrong (Code Review)
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 14: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-4856: Port data stream service to KRPC

2017-10-27 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/8023 ) Change subject: IMPALA-4856: Port data stream service to KRPC .. Patch Set 6: (15 comments)

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

2017-10-27 Thread Tim Wood (Code Review)
Tim Wood has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/8372 ) Change subject: IMPALA-6087: Revisit tests withheld from TPC-DS suite for use of TRUNCATE .. IMPALA-6087: Revisit tests withheld from TPC-DS

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

2017-10-27 Thread Dan Hecht (Code Review)
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 14: Code-Review+2 (1 comment)

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

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. IMPALA-2758: Remove BufferedTupleStream::GetRows

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

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 6: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6093: diagnostics for flaky TestHashJoinTimer

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8403 ) Change subject: IMPALA-6093: diagnostics for flaky TestHashJoinTimer .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-6093: diagnostics for flaky TestHashJoinTimer

2017-10-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8403 ) Change subject: IMPALA-6093: diagnostics for flaky TestHashJoinTimer .. Patch Set 3: Code-Review+2 carry -- To view, visit

[Impala-ASF-CR] IMPALA-6093: diagnostics for flaky TestHashJoinTimer

2017-10-27 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8403 ) Change subject: IMPALA-6093: diagnostics for flaky TestHashJoinTimer .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-3436: Return a decimal when rounding a double

2017-10-27 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8398 ) Change subject: IMPALA-3436: Return a decimal when rounding a double .. Patch Set 1: (13 comments)

[Impala-ASF-CR] IMPALA-6108, IMPALA-6070: Parallel data load (re-instated).

2017-10-27 Thread Philip Zeyliger (Code Review)
Hello Joe McDonnell, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8405 to look at the new patch set (#2). Change subject: IMPALA-6108, IMPALA-6070: Parallel data load (re-instated).

[Impala-ASF-CR] IMPALA-6108, IMPALA-6070: Parallel data load (re-instated).

2017-10-27 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8405 Change subject: IMPALA-6108, IMPALA-6070: Parallel data load (re-instated). .. IMPALA-6108, IMPALA-6070: Parallel data load

[Impala-ASF-CR] IMPALA-5243: Speed up code gen for wide Avro tables.

2017-10-27 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/8211 ) Change subject: IMPALA-5243: Speed up code gen for wide Avro tables. .. Patch Set 8: > Patch Set 8: > > > Oof. GVO caught a

[Impala-ASF-CR] IMPALA-5243: Speed up code gen for wide Avro tables.

2017-10-27 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. ( http://gerrit.cloudera.org:8080/8211 ) Change subject: IMPALA-5243: Speed up code gen for wide Avro tables. .. Patch Set 8: > Oof. GVO caught a (correct) clang-tidy issue. Alas,

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

2017-10-27 Thread Thomas Tauber-Marshall (Code Review)
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: TPCDS results:

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

2017-10-27 Thread Henry Robinson (Code Review)
Henry Robinson 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 2: (2 comments)

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

2017-10-27 Thread Tim Armstrong (Code Review)
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: (3 comments)

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

2017-10-27 Thread Tim Armstrong (Code Review)
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 (#14). Change subject: IMPALA-1575: Part 1: eagerly release query exec resources

[Impala-ASF-CR] IMPALA-5243: Speed up code gen for wide Avro tables.

2017-10-27 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/8211 ) Change subject: IMPALA-5243: Speed up code gen for wide Avro tables. .. Patch Set 8: Oof. GVO caught a (correct) clang-tidy issue.

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

2017-10-27 Thread Dan Hecht (Code Review)
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 14: Code-Review+1 Thanks. This approach looks okay

[Impala-ASF-CR] IMPALA-5243: Speed up code gen for wide Avro tables.

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8211 ) Change subject: IMPALA-5243: Speed up code gen for wide Avro tables. .. Patch Set 7: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-4856: Port data stream service to KRPC

2017-10-27 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/8023 ) Change subject: IMPALA-4856: Port data stream service to KRPC .. Patch Set 6: (6 comments)

[Impala-ASF-CR] IMPALA-4856: Port data stream service to KRPC

2017-10-27 Thread Michael Ho (Code Review)
Hello Sailesh Mukil, Mostafa Mokhtar, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8023 to look at the new patch set (#6). Change subject: IMPALA-4856: Port data stream service to KRPC

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

2017-10-27 Thread Dan Hecht (Code Review)
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: (3 comments)

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

2017-10-27 Thread John Russell (Code Review)
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 2: (4 comments) I included the "long-form"

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

2017-10-27 Thread John Russell (Code Review)
Hello Bharath Vissapragada, Henry Robinson, Michael Brown, Sailesh Mukil, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8401 to look at the new patch set (#2). Change subject: IMPALA-5473: [DOCS] Document TLS min version & cipher options

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

2017-10-27 Thread John Sherman (Code Review)
John Sherman has posted comments on this change. ( http://gerrit.cloudera.org:8080/8404 ) Change subject: IMPALA-5017: Error on decimal overflow .. Patch Set 1: (1 comment)

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

2017-10-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 6: Code-Review+2 Glad we have automated checking for

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

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

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

2017-10-27 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/8226 ) Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows .. Patch Set 6: Sorry I forgot to handle some error. -- To view,

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

2017-10-27 Thread Tianyi Wang (Code Review)
Hello Thomas Tauber-Marshall, Tim Armstrong, Dan Hecht, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8226 to look at the new patch set (#6). Change subject: IMPALA-2758: Remove BufferedTupleStream::GetRows

[Impala-ASF-CR] IMPALA-4856: Port data stream service to KRPC

2017-10-27 Thread Michael Ho (Code Review)
Hello Sailesh Mukil, Mostafa Mokhtar, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8023 to look at the new patch set (#5). Change subject: IMPALA-4856: Port data stream service to KRPC

[Impala-ASF-CR] IMPALA-6093: diagnostics for flaky TestHashJoinTimer

2017-10-27 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. ( http://gerrit.cloudera.org:8080/8403 ) Change subject: IMPALA-6093: diagnostics for flaky TestHashJoinTimer .. Patch Set 2: Code-Review+1 (1 comment)

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

2017-10-27 Thread Sailesh Mukil (Code Review)
Sailesh Mukil 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

2017-10-27 Thread Tim Armstrong (Code Review)
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 13: Code-Review+1 -- To view, visit

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

2017-10-27 Thread Tim Armstrong (Code Review)
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 14: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-6093: diagnostics for flaky TestHashJoinTimer

2017-10-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8403 ) Change subject: IMPALA-6093: diagnostics for flaky TestHashJoinTimer .. Patch Set 2: I implemented Bikram's suggestion from the JIRA.

[Impala-ASF-CR] IMPALA-6093: diagnostics for flaky TestHashJoinTimer

2017-10-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8403 ) Change subject: IMPALA-6093: diagnostics for flaky TestHashJoinTimer .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-6093: diagnostics for flaky TestHashJoinTimer

2017-10-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/8403 ) Change subject: IMPALA-6093: diagnostics for flaky TestHashJoinTimer .. IMPALA-6093: diagnostics for flaky TestHashJoinTimer We don't

[Impala-ASF-CR] IMPALA-5243: Speed up code gen for wide Avro tables.

2017-10-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8211 ) Change subject: IMPALA-5243: Speed up code gen for wide Avro tables. .. Patch Set 7: Build started:

[Impala-ASF-CR] IMPALA-5243: Speed up code gen for wide Avro tables.

2017-10-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8211 ) Change subject: IMPALA-5243: Speed up code gen for wide Avro tables. .. Patch Set 7: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-5243: Speed up code gen for wide Avro tables.

2017-10-27 Thread Philip Zeyliger (Code Review)
Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/8211 ) Change subject: IMPALA-5243: Speed up code gen for wide Avro tables. .. Patch Set 7: Code-Review+1 Carrying the +1. I've not made

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

2017-10-27 Thread Kim Jin Chul (Code Review)
Kim Jin Chul 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 3: (2 comments)

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

2017-10-27 Thread Kim Jin Chul (Code Review)
Hello Greg Rahn, Zach Amsden, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8311 to look at the new patch set (#3). Change subject: IMPALA-5607: Add additional units to EXTRACT, DATE_PART, TRUNC