[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 .

[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) http://gerrit.cloudera.org:

[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 an

[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: https://jenkins.impala.io/j

[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 http://gerrit.clo

[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 kno

[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) http://gerrit.cloudera.org:8080/#/c/8

[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-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 http://ger

[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 http://gerr

[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) http://gerrit.cloudera.org:8

[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) http://gerrit.cloudera

[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-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-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, visit

[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: https://jenkins.impala.io/job/g

[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 drop

[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) http://gerrit.cloudera.org:8080/#/c/8404/1//COMMIT_M

[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-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" d

[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) http://gerrit.cloudera.org:8080/#/

[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-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) http://gerrit.cloudera.org:8080/#/c/8023/3/be/

[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: https://jenkins.i

[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 Philip Zeyliger (Code Review)
Hello Tim Armstrong, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8211 to look at the new patch set (#8). Change subject: IMPALA-5243: Speed up code gen for wide Avro tables. .

[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. Al

[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-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) http://gerrit.cloudera.org:808

[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) http://gerrit.cloudera.org

[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-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-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 (correct)

[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 (re-instate

[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-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) http://gerrit.cloudera.org:8080/#/c/839

[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 http://gerrit.cl

[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 http://ger

[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: https://jenkins.impala.io/j

[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 http://gerrit.cl

[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 Th

[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) http://gerrit.clouder

[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 s

[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) http://gerrit.cloudera.org:8080/#/c/8023/6/be

[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) http://gerrit.clo

[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 http

[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: https://jenkins.impal

[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: https://jenkins.impala.io/j

[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 http://gerrit.clo

[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 the

[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) http://gerrit.cloude

[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 clos

[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 retu

[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) http://gerrit.cloudera.org:8080/#/c/8309/1//COMMIT_

[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) http://gerrit.cl

[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 tes

[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) http://gerrit.cloud

[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 http://gerri

[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 TestHashJoinT

[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) http://gerrit.cl

[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 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 un

[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: https:/

[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 http:/

[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-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 tab

[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 http://gerri

[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-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) http://gerrit.cloudera.org:8

[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-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 i

[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 coordinat

[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) http://gerrit.clouder

[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) http://gerrit.cloud

[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) http://gerrit.cloud

[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 http://gerrit.cloudera.org:8

[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: Verified+1 -- To view

[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 submitted this change and it was merged. ( 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 orde