[Impala-ASF-CR] IMPALA-4123: Fast bit unpacking

2016-10-05 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4123: Fast bit unpacking .. Patch Set 4: (15 comments) http://gerrit.cloudera.org:8080/#/c/4494/4/be/src/benchmarks/bit-packing-benchmark.cc File

[Impala-ASF-CR] IMPALA-4213: Fix Kudu predicates that need constant folding

2016-10-05 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4213: Fix Kudu predicates that need constant folding .. IMPALA-4213: Fix Kudu predicates that need constant folding Folding const exprs where

[Impala-ASF-CR] IMPALA-1702: Enforce table level consistency accross service

2016-10-05 Thread Huaisi Xu (Code Review)
Huaisi Xu has posted comments on this change. Change subject: IMPALA-1702: Enforce table level consistency accross service .. Patch Set 7: (6 comments)

[Impala-ASF-CR] IMPALA-4204: Remove KuduScanNodeTest

2016-10-05 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4204: Remove KuduScanNodeTest .. IMPALA-4204: Remove KuduScanNodeTest In IMPALA-4204, the limit test could fail depending on the timing of

[Impala-ASF-CR] IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables

2016-10-05 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables .. Patch Set 3: (14 comments) next batch... I'm probably about 1/2 way through.

[Impala-ASF-CR] IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

2016-10-05 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend .. Patch Set 21: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-1702: Enforce table level consistency accross service

2016-10-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-1702: Enforce table level consistency accross service .. Patch Set 7: (6 comments)

[Impala-ASF-CR] IMPALA-3920: TotalStorageWaitTime counter not populated for fragments with Kudu scan node

2016-10-05 Thread anujphadke (Code Review)
anujphadke has uploaded a new change for review. http://gerrit.cloudera.org:8080/4639 Change subject: IMPALA-3920: TotalStorageWaitTime counter not populated for fragments with Kudu scan node .. IMPALA-3920:

[Impala-ASF-CR] IMPALA-4123: Fast bit unpacking

2016-10-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#5). Change subject: IMPALA-4123: Fast bit unpacking .. IMPALA-4123: Fast bit unpacking Adds utility functions for fast unpacking of batches of bit-packed values. These support

[Impala-ASF-CR] IMPALA-4213: Fix Kudu predicates that need constant folding

2016-10-05 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4213: Fix Kudu predicates that need constant folding .. Patch Set 4: Code-Review+2 Carrying Alex's +2 -- To view, visit

[Impala-ASF-CR] IMPALA-4213: Fix Kudu predicates that need constant folding

2016-10-05 Thread Matthew Jacobs (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4613 to look at the new patch set (#4). Change subject: IMPALA-4213: Fix Kudu predicates that need constant folding ..

[Impala-ASF-CR] IMPALA-4123: Fast bit unpacking

2016-10-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4123: Fast bit unpacking .. Patch Set 2: (33 comments) http://gerrit.cloudera.org:8080/#/c/4494/2/be/src/benchmarks/bit-packing-benchmark.cc File

[Impala-ASF-CR] IMPALA-4123: Fast bit unpacking

2016-10-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#4). Change subject: IMPALA-4123: Fast bit unpacking .. IMPALA-4123: Fast bit unpacking Adds utility functions for fast unpacking of batches of bit-packed values. These support

[Impala-ASF-CR] IMPALA-4213: Planner not pushing some Kudu predicates

2016-10-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4213: Planner not pushing some Kudu predicates .. Patch Set 3: Code-Review+2 (3 comments) http://gerrit.cloudera.org:8080/#/c/4613/3//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-3229: Don't assume that AUX exists just because of shell env

2016-10-05 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3229: Don't assume that AUX exists just because of shell env .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4563 To

[Impala-ASF-CR] IMPALA-1702: Enforce table level consistency accross service

2016-10-05 Thread Huaisi Xu (Code Review)
Huaisi Xu has posted comments on this change. Change subject: IMPALA-1702: Enforce table level consistency accross service .. Patch Set 7: (16 comments) http://gerrit.cloudera.org:8080/#/c/4349/6/common/thrift/Types.thrift

[Impala-ASF-CR] IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables

2016-10-05 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables .. Patch Set 3: (11 comments) Not even close to done but want to start giving you feedback in

[Impala-ASF-CR](asf-site) IMPALA-3808: Add downloads page

2016-10-05 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3808: Add downloads page .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4544 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4023: don't attach buffered tuple streams to batches

2016-10-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4023: don't attach buffered tuple streams to batches .. Patch Set 6: (2 comments)

[Impala-ASF-CR] IMPALA-3342: Adding new timer to accurately measure the TotalCpuTime

2016-10-05 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-3342: Adding new timer to accurately measure the TotalCpuTime .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/4633/1//COMMIT_MSG

[Impala-ASF-CR](asf-site) IMPALA-3808: Add downloads page

2016-10-05 Thread Jim Apple (Code Review)
Jim Apple has submitted this change and it was merged. Change subject: IMPALA-3808: Add downloads page .. IMPALA-3808: Add downloads page This should not be pushed to the repo until a release is approved by the IPMC and the

[Impala-ASF-CR] IMPALA-4023: don't attach buffered tuple streams to batches

2016-10-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4023: don't attach buffered tuple streams to batches .. Patch Set 6: (2 comments)

[Impala-ASF-CR] IMPALA-4213: Planner not pushing some Kudu predicates

2016-10-05 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4213: Planner not pushing some Kudu predicates .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-2905: Handle coordinator fragment lifecycle like all others

2016-10-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-2905: Handle coordinator fragment lifecycle like all others .. Patch Set 6: (1 comment) Will wait for rebase to continue.

[Impala-ASF-CR] IMPALA-4213: Planner not pushing some Kudu predicates

2016-10-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4213: Planner not pushing some Kudu predicates .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-1702: Enforce table level consistency accross service

2016-10-05 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-1702: Enforce table level consistency accross service .. Patch Set 6: (12 comments) So much better!

[Impala-ASF-CR] IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

2016-10-05 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend .. Patch Set 21: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

2016-10-05 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend .. Patch Set 18: Code-Review+1 (1 comment) Changes since PS16

[Impala-ASF-CR] IMPALA-1702: Enforce table level consistency accross service

2016-10-05 Thread Huaisi Xu (Code Review)
Huaisi Xu has posted comments on this change. Change subject: IMPALA-1702: Enforce table level consistency accross service .. Patch Set 6: (5 comments)

[Impala-ASF-CR] IMPALA-1702: Enforce table level consistency accross service

2016-10-05 Thread Huaisi Xu (Code Review)
Huaisi Xu has uploaded a new patch set (#6). Change subject: IMPALA-1702: Enforce table level consistency accross service .. IMPALA-1702: Enforce table level consistency accross service Problems: 1.

[Impala-ASF-CR] IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

2016-10-05 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend .. Patch Set 18: (1 comment)

[Impala-ASF-CR] IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

2016-10-05 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend .. Patch Set 18: (2 comments)

[Impala-ASF-CR] IMPALA-4135: Thrift threaded server times-out connections during high load

2016-10-05 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4135: Thrift threaded server times-out connections during high load .. Patch Set 7: (1 comment)

[Impala-ASF-CR] IMPALA-3202: DiskIoMgr improvements for new buffer pool

2016-10-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/4631 Change subject: IMPALA-3202: DiskIoMgr improvements for new buffer pool .. IMPALA-3202: DiskIoMgr improvements for new buffer pool The

[Impala-ASF-CR] IMPALA-3229: Don't assume that AUX exists just because of shell env

2016-10-05 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3229: Don't assume that AUX exists just because of shell env .. Patch Set 5: Build failed:

[Impala-ASF-CR] IMPALA-3229: Don't assume that AUX exists just because of shell env

2016-10-05 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3229: Don't assume that AUX exists just because of shell env .. Patch Set 5: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-3229: Don't assume that AUX exists just because of shell env

2016-10-05 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3229: Don't assume that AUX exists just because of shell env .. Patch Set 5: Code-Review+2 rebase, carry +2 -- To view, visit

[Impala-ASF-CR] IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

2016-10-05 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend .. Patch Set 18: the latest patch contained another bug fix.