[Impala-ASF-CR] IMPALA-4577: Adjust maximum size of row batch queue with MT DOP.

2016-12-02 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4577: Adjust maximum size of row batch queue with MT_DOP. .. IMPALA-4577: Adjust maximum size of row batch queue with MT_DOP. When MT_DOP is

[Impala-ASF-CR] IMPALA-4363: Add Parquet timestamp validation

2016-12-02 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4363: Add Parquet timestamp validation .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5343 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4363: Add Parquet timestamp validation

2016-12-02 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4363: Add Parquet timestamp validation .. IMPALA-4363: Add Parquet timestamp validation Before this patch, we would simply read the INT96

[Impala-ASF-CR] IMPALA-4572: Run COMPUTE STATS on Parquet tables with MT DOP=4.

2016-12-02 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4572: Run COMPUTE STATS on Parquet tables with MT_DOP=4. .. Patch Set 3: Code-Review+2 rebase -- To view, visit http://gerrit.cloudera.org:8080/5315 To

[Impala-ASF-CR] Cleanup of logging output, part 2: downgrading 'debug' to 'trace'

2016-12-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Cleanup of logging output, part 2: downgrading 'debug' to 'trace' .. Patch Set 1: Build started:

[Impala-ASF-CR] IMPALA-4570: shell tarball breaks with certain setuptools versions

2016-12-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4570: shell tarball breaks with certain setuptools versions .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5314

[Impala-ASF-CR] IMPALA-4570: shell tarball breaks with certain setuptools versions

2016-12-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4570: shell tarball breaks with certain setuptools versions .. IMPALA-4570: shell tarball breaks with certain setuptools versions The

[Impala-ASF-CR] Cleanup of logging output, part 2: downgrading 'debug' to 'trace'

2016-12-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Cleanup of logging output, part 2: downgrading 'debug' to 'trace' .. Patch Set 1: Build started:

[Impala-ASF-CR] Fix typo in DDL statement for loading Kudu table in stress test

2016-12-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Fix typo in DDL statement for loading Kudu table in stress test .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5158 To

[Impala-ASF-CR] IMPALA-4363: Add Parquet timestamp validation

2016-12-02 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4363: Add Parquet timestamp validation .. Patch Set 2: Code-Review+2 Carrying the +2. -- To view, visit http://gerrit.cloudera.org:8080/5343 To

[Impala-ASF-CR] IMPALA-4363: Add Parquet timestamp validation

2016-12-02 Thread Taras Bobrovytsky (Code Review)
Hello Internal Jenkins, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5343 to look at the new patch set (#2). Change subject: IMPALA-4363: Add Parquet timestamp validation

[Impala-ASF-CR] IMPALA-4363: Add Parquet timestamp validation

2016-12-02 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4363: Add Parquet timestamp validation .. Patch Set 1: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-4577: Adjust maximum size of row batch queue with MT DOP.

2016-12-02 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4577: Adjust maximum size of row batch queue with MT_DOP. .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5330 To

[Impala-ASF-CR] IMPALA-4572: Run COMPUTE STATS on Parquet tables with MT DOP=4.

2016-12-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4572: Run COMPUTE STATS on Parquet tables with MT_DOP=4. .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5315 To

[Impala-ASF-CR] IMPALA-4363: Add Parquet timestamp validation

2016-12-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4363: Add Parquet timestamp validation .. Patch Set 1: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-4572: Run COMPUTE STATS on Parquet tables with MT DOP=4.

2016-12-02 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new patch set (#2). Change subject: IMPALA-4572: Run COMPUTE STATS on Parquet tables with MT_DOP=4. .. IMPALA-4572: Run COMPUTE STATS on Parquet tables with MT_DOP=4. COMPUTE STATS on Parquet tables is

[Impala-ASF-CR] IMPALA-4572: Run COMPUTE STATS on Parquet tables with MT DOP=4.

2016-12-02 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4572: Run COMPUTE STATS on Parquet tables with MT_DOP=4. .. Patch Set 1: (4 comments)

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-12-02 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 6: (11 comments) Some more comments about the refcounting, many in response to yours.

[Impala-ASF-CR] IMPALA-4363: Add Parquet timestamp validation

2016-12-02 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4363: Add Parquet timestamp validation .. Patch Set 1: The previous review https://gerrit.cloudera.org/#/c/4968/ has been abandoned because I

[Impala-ASF-CR] IMPALA-4363: Add Parquet timestamp validation

2016-12-02 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new change for review. http://gerrit.cloudera.org:8080/5343 Change subject: IMPALA-4363: Add Parquet timestamp validation .. IMPALA-4363: Add Parquet timestamp validation Before this patch,

[Impala-ASF-CR] IMPALA-3875: Thrift threaded server hang in some cases

2016-12-02 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3875: Thrift threaded server hang in some cases .. Patch Set 4: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-4363: Add Parquet timestamp validation

2016-12-02 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4363: Add Parquet timestamp validation .. Patch Set 9: (3 comments) http://gerrit.cloudera.org:8080/#/c/4968/9/be/src/exec/parquet-column-readers.cc

[Impala-ASF-CR] IMPALA-4172/IMPALA-3653: Improvements to block metadata loading

2016-12-02 Thread Bharath Vissapragada (Code Review)
Hello Dimitris Tsirogiannis, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5148 to look at the new patch set (#18). Change subject: IMPALA-4172/IMPALA-3653: Improvements to block metadata loading

[Impala-ASF-CR] IMPALA-4577: Adjust maximum size of row batch queue with MT DOP.

2016-12-02 Thread Alex Behm (Code Review)
Hello Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5330 to look at the new patch set (#3). Change subject: IMPALA-4577: Adjust maximum size of row batch queue with MT_DOP.

[Impala-ASF-CR] Cleanup of logging output, part 2: downgrading 'debug' to 'trace'

2016-12-02 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: Cleanup of logging output, part 2: downgrading 'debug' to 'trace' .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5342 To

[Impala-ASF-CR] Cleanup of logging output, part 2: downgrading 'debug' to 'trace'

2016-12-02 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has uploaded a new change for review. http://gerrit.cloudera.org:8080/5342 Change subject: Cleanup of logging output, part 2: downgrading 'debug' to 'trace' .. Cleanup of logging output, part 2: downgrading

[Impala-ASF-CR] IMPALA-4363: Add Parquet timestamp validation

2016-12-02 Thread Taras Bobrovytsky (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4968 to look at the new patch set (#10). Change subject: IMPALA-4363: Add Parquet timestamp validation .. IMPALA-4363: Add

[Impala-ASF-CR] IMPALA-4566: Kudu client glog contention can cause timeouts

2016-12-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4566: Kudu client glog contention can cause timeouts .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-4566: Kudu client glog contention can cause timeouts

2016-12-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-4566: Kudu client glog contention can cause timeouts .. IMPALA-4566: Kudu client glog contention can cause timeouts Under stressful workloads,

[Impala-ASF-CR] Additional functional testing for default values on Kudu tables

2016-12-02 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Additional functional testing for default values on Kudu tables .. Patch Set 1: (1 comment)

[Impala-ASF-CR] Additional functional testing for default values on Kudu tables

2016-12-02 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has uploaded a new change for review. http://gerrit.cloudera.org:8080/5337 Change subject: Additional functional testing for default values on Kudu tables .. Additional functional testing for default

[Impala-ASF-CR] IMPALA-4566: Kudu client glog contention can cause timeouts

2016-12-02 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4566: Kudu client glog contention can cause timeouts .. Patch Set 1: (3 comments) > I saw Todd has a change to address this on the client side.. is >

[Impala-ASF-CR] IMPALA-4579: SHOW CREATE VIEW fails for view containing a subquery

2016-12-02 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-4579: SHOW CREATE VIEW fails for view containing a subquery .. Patch Set 2: Code-Review+2 Rebase. Carry +2 -- To view, visit

[Impala-ASF-CR] IMPALA-4580: Fix crash with FETCH FIRST when #rows < result cache size

2016-12-02 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4580: Fix crash with FETCH_FIRST when #rows < result cache size .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5335 To

[Impala-ASF-CR] IMPALA-4566: Kudu client glog contention can cause timeouts

2016-12-02 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4566: Kudu client glog contention can cause timeouts .. Patch Set 1: I saw Todd has a change to address this on the client side.. is this just a bandaid

[Impala-ASF-CR] IMPALA-4580: Fix crash with FETCH FIRST when #rows < result cache size

2016-12-02 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-4580: Fix crash with FETCH_FIRST when #rows < result cache size .. Patch Set 2: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-4580: Fix crash with FETCH FIRST when #rows < result cache size

2016-12-02 Thread Henry Robinson (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5335 to look at the new patch set (#2). Change subject: IMPALA-4580: Fix crash with FETCH_FIRST when #rows < result cache size

[Impala-ASF-CR] IMPALA-4566: Kudu client glog contention can cause timeouts

2016-12-02 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4566: Kudu client glog contention can cause timeouts .. Patch Set 1: Thanks, I'll test this on the stress cluster over the weekend -- To view, visit

[Impala-ASF-CR] IMPALA-4580: Fix crash with FETCH FIRST when #rows < result cache size

2016-12-02 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4580: Fix crash with FETCH_FIRST when #rows < result cache size .. Patch Set 1: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-3200: Implement suballocator for splitting buffers

2016-12-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3200: Implement suballocator for splitting buffers .. Patch Set 11: (1 comment)

[Impala-ASF-CR] IMPALA-3200: Implement suballocator for splitting buffers

2016-12-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#11). Change subject: IMPALA-3200: Implement suballocator for splitting buffers .. IMPALA-3200: Implement suballocator for splitting buffers This is useful for situations like hash

[Impala-ASF-CR] IMPALA-3200: Implement suballocator for splitting buffers

2016-12-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3200: Implement suballocator for splitting buffers .. Patch Set 9: (5 comments) http://gerrit.cloudera.org:8080/#/c/4715/4//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-3200: Implement suballocator for splitting buffers

2016-12-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#10). Change subject: IMPALA-3200: Implement suballocator for splitting buffers .. IMPALA-3200: Implement suballocator for splitting buffers This is useful for situations like hash

[Impala-ASF-CR] IMPALA-3200: Implement suballocator for splitting buffers

2016-12-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#9). Change subject: IMPALA-3200: Implement suballocator for splitting buffers .. IMPALA-3200: Implement suballocator for splitting buffers This is useful for situations like hash

[Impala-ASF-CR] Add Apache license header to files in doc directory

2016-12-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add Apache license header to files in doc directory .. Patch Set 2: (1 comment) > (1 comment) > > Looks good. (Does the +2 initiate any further transfer of

[Impala-ASF-CR] Add Apache license header to files in doc directory

2016-12-02 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: Add Apache license header to files in doc directory .. Patch Set 2: Code-Review+2 (1 comment) Looks good. (Does the +2 initiate any further transfer of files or

[Impala-ASF-CR] Add Apache license header to files in doc directory

2016-12-02 Thread John Russell (Code Review)
John Russell has submitted this change and it was merged. Change subject: Add Apache license header to files in doc directory .. Add Apache license header to files in doc directory This now gives a clean RAT check with

[Impala-ASF-CR] IMPALA-4570: shell tarball breaks with certain setuptools versions

2016-12-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4570: shell tarball breaks with certain setuptools versions .. Patch Set 3: Build started:

[Impala-ASF-CR] IMPALA-4570: shell tarball breaks with certain setuptools versions

2016-12-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4570: shell tarball breaks with certain setuptools versions .. Patch Set 3: Code-Review+2 Fixed syntax bug and rebased. -- To view, visit

[Impala-ASF-CR] IMPALA-4570: shell tarball breaks with certain setuptools versions

2016-12-02 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5314 to look at the new patch set (#3). Change subject: IMPALA-4570: shell tarball breaks with certain setuptools versions

[Impala-ASF-CR] IMPALA-4580: Fix crash with FETCH FIRST when #rows < result cache size

2016-12-02 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/5335 Change subject: IMPALA-4580: Fix crash with FETCH_FIRST when #rows < result cache size .. IMPALA-4580: Fix crash with FETCH_FIRST when

[Impala-ASF-CR] IMPALA-4570: shell tarball breaks with certain setuptools versions

2016-12-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4570: shell tarball breaks with certain setuptools versions .. Patch Set 2: Verified-1 Build failed:

[Impala-ASF-CR] Fix typo in DDL statement for loading Kudu table in stress test

2016-12-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Fix typo in DDL statement for loading Kudu table in stress test .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-4566: Kudu client glog contention can cause timeouts

2016-12-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/5334 Change subject: IMPALA-4566: Kudu client glog contention can cause timeouts .. IMPALA-4566: Kudu client glog contention can

[Impala-ASF-CR] IMPALA-3875: Thrift threaded server hang in some cases

2016-12-02 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3875: Thrift threaded server hang in some cases .. Patch Set 3: Code-Review+2 (2 comments) Carry +2.

[Impala-ASF-CR] IMPALA-3875: Thrift threaded server hang in some cases

2016-12-02 Thread Sailesh Mukil (Code Review)
Hello Henry Robinson, Internal Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5263 to look at the new patch set (#4). Change subject: IMPALA-3875: Thrift threaded server hang in some cases

[Impala-ASF-CR] IMPALA-3875: Thrift threaded server hang in some cases

2016-12-02 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-3875: Thrift threaded server hang in some cases .. Patch Set 3: Code-Review+2 (2 comments)

[Impala-ASF-CR] IMPALA-4303: Do not reset() qualifier of union operands.

2016-12-02 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4303: Do not reset() qualifier of union operands. .. Patch Set 2: Code-Review+2 (3 comments)

[Impala-ASF-CR] IMPALA-4577: Adjust maximum size of row batch queue with MT DOP.

2016-12-02 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4577: Adjust maximum size of row batch queue with MT_DOP. .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-4579: SHOW CREATE VIEW fails for view containing a subquery

2016-12-02 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has uploaded a new change for review. http://gerrit.cloudera.org:8080/5333 Change subject: IMPALA-4579: SHOW CREATE VIEW fails for view containing a subquery .. IMPALA-4579: SHOW CREATE VIEW fails for

[Impala-ASF-CR] IMPALA-4572: Run COMPUTE STATS on Parquet tables with MT DOP=4.

2016-12-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4572: Run COMPUTE STATS on Parquet tables with MT_DOP=4. .. Patch Set 1: (3 comments)

[Impala-ASF-CR] Add Apache license header to files in doc directory

2016-12-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add Apache license header to files in doc directory .. Patch Set 2: > Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit/4/ This job is just

[Impala-ASF-CR] Add Apache license header to files in doc directory

2016-12-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Add Apache license header to files in doc directory .. Patch Set 2: Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit/4/ -- To view,

[Impala-ASF-CR] IMPALA-4561: Replace DISTRIBUTE BY with PARTITION BY in CREATE TABLE

2016-12-02 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has uploaded a new patch set (#2). Change subject: IMPALA-4561: Replace DISTRIBUTE BY with PARTITION BY in CREATE TABLE .. IMPALA-4561: Replace DISTRIBUTE BY with PARTITION BY in CREATE TABLE Change-Id:

[Impala-ASF-CR] IMPALA-4561: Replace DISTRIBUTE BY with PARTITION BY in CREATE TABLE

2016-12-02 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-4561: Replace DISTRIBUTE BY with PARTITION BY in CREATE TABLE .. Patch Set 1: (7 comments)

[Impala-ASF-CR] IMPALA-4577: Adjust maximum size of row batch queue with MT DOP.

2016-12-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4577: Adjust maximum size of row batch queue with MT_DOP. .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/5330 To

[Impala-ASF-CR] IMPALA-4357: Fix DROP TABLE to pass analysis if the table fails to load

2016-12-02 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4357: Fix DROP TABLE to pass analysis if the table fails to load .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5144

[Impala-ASF-CR] IMPALA-3875: Thrift threaded server hang in some cases

2016-12-02 Thread Sailesh Mukil (Code Review)
Hello Henry Robinson, Internal Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5263 to look at the new patch set (#3). Change subject: IMPALA-3875: Thrift threaded server hang in some cases

[Impala-ASF-CR] IMPALA-4570: shell tarball breaks with certain setuptools versions

2016-12-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4570: shell tarball breaks with certain setuptools versions .. Patch Set 2: Build started:

[Impala-ASF-CR] IMPALA-4577: Adjust maximum size of row batch queue with MT DOP.

2016-12-02 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new patch set (#2). Change subject: IMPALA-4577: Adjust maximum size of row batch queue with MT_DOP. .. IMPALA-4577: Adjust maximum size of row batch queue with MT_DOP. When MT_DOP is set, non-Parquet

[Impala-ASF-CR] IMPALA-4577: Adjust maximum size of row batch queue with MT DOP.

2016-12-02 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4577: Adjust maximum size of row batch queue with MT_DOP. .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-4577: Adjust maximum size of row batch queue with MT DOP.

2016-12-02 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4577: Adjust maximum size of row batch queue with MT_DOP. .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-4574: Do not treat UUID() like a constant expr.

2016-12-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4574: Do not treat UUID() like a constant expr. .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-4343,IMPALA-4354: qgen: model INSERTs; write INSERTs from query model

2016-12-02 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4343,IMPALA-4354: qgen: model INSERTs; write INSERTs from query model .. Patch Set 9: Thanks Taras! Do you mind submitting this for me since you're a

[Impala-ASF-CR] IMPALA-4343,IMPALA-4354: qgen: model INSERTs; write INSERTs from query model

2016-12-02 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4343,IMPALA-4354: qgen: model INSERTs; write INSERTs from query model .. Patch Set 9: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-4577: Adjust maximum size of row batch queue with MT DOP.

2016-12-02 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new change for review. http://gerrit.cloudera.org:8080/5330 Change subject: IMPALA-4577: Adjust maximum size of row batch queue with MT_DOP. .. IMPALA-4577: Adjust maximum size of row batch queue with

[Impala-ASF-CR] IMPALA-3200: Implement suballocator for splitting buffers

2016-12-02 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3200: Implement suballocator for splitting buffers .. Patch Set 8: (4 comments) http://gerrit.cloudera.org:8080/#/c/4715/4//COMMIT_MSG Commit Message: PS4,

[Impala-ASF-CR] IMPALA-2057: Better error message for incorrect avro decimal column declaration

2016-12-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-2057: Better error message for incorrect avro decimal column declaration .. Patch Set 2: (3 comments)

[Impala-ASF-CR] IMPALA-4574: Do not treat UUID() like a constant expr.

2016-12-02 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4574: Do not treat UUID() like a constant expr. .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/5324/1//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-4343,IMPALA-4354: qgen: model INSERTs; write INSERTs from query model

2016-12-02 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4343,IMPALA-4354: qgen: model INSERTs; write INSERTs from query model .. Patch Set 9: Code-Review+1 rebase -- To view, visit

[Impala-ASF-CR] IMPALA-4525: follow-on: cleanup error handling

2016-12-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4525: follow-on: cleanup error handling .. IMPALA-4525: follow-on: cleanup error handling Testing: Ran exhaustive build. There is

[Impala-ASF-CR] IMPALA-4525: follow-on: cleanup error handling

2016-12-02 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4525: follow-on: cleanup error handling .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5212 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-2057: Better error message for incorrect avro decimal column declaration

2016-12-02 Thread anujphadke (Code Review)
anujphadke has posted comments on this change. Change subject: IMPALA-2057: Better error message for incorrect avro decimal column declaration .. Patch Set 2: 1. check if a logical type is present, and throw an exception if

[Impala-ASF-CR] IMPALA-2057: Better error message for incorrect avro decimal column declaration

2016-12-02 Thread anujphadke (Code Review)
anujphadke has uploaded a new patch set (#2). Change subject: IMPALA-2057: Better error message for incorrect avro decimal column declaration .. IMPALA-2057: Better error message for incorrect avro decimal column declaration

[Impala-ASF-CR] IMPALA-4574: Do not treat UUID() like a constant expr.

2016-12-02 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new change for review. http://gerrit.cloudera.org:8080/5324 Change subject: IMPALA-4574: Do not treat UUID() like a constant expr. .. IMPALA-4574: Do not treat UUID() like a constant expr. A recent