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

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

2016-10-05 Thread Marcel Kornacker (Code Review)
Hello Matthew Jacobs, Internal Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4054 to look at the new patch set (#18). Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend ...

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

[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: http://sandbox.jenkins.cloudera.com/job/impala-

[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: http://sandbox.jenkins.cloudera.com/job/impala-external-ge

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

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

2016-10-05 Thread Thomas Tauber-Marshall (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4519 to look at the new patch set (#7). Change subject: IMPALA-4135: Thrift threaded server times-out connections during high load ...

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

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

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

[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. CatalogServiceCatalog::reset(

[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) http://gerrit.cloudera.org:8080/#/c/4349/4/fe/src/main/java/org/apache/imp

[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-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: Code-Review+1 (1 comment) http://gerrit.cloud

[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: (4 comments) http://gerrit.cloudera.org:8080/#/c/4349/6/common/thrift/Types.thrift File

[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: (1 comment) http://gerrit.cloudera.org:8080/#/c/4349/6/fe/src/main/java/org/apache/impa

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

2016-10-05 Thread anujphadke (Code Review)
anujphadke has uploaded a new change for review. http://gerrit.cloudera.org:8080/4633 Change subject: IMPALA-3342: Adding new timer to accurately measure the TotalCpuTime .. IMPALA-3342: Adding new timer to accurately measure

[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: (3 comments) http://gerrit.cloudera.org:8080/#/c/4613/2/fe/src/main/java/org/apache/impala/ana

[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: (3 comments) http://gerrit.cloudera.org:8080/#/c/4613/2/fe/src/main/java/org/apache/impal

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

2016-10-05 Thread Marcel Kornacker (Code Review)
Hello Henry Robinson, Matthew Jacobs, Internal Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4054 to look at the new patch set (#19). Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

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

2016-10-05 Thread Marcel Kornacker (Code Review)
Hello Henry Robinson, Matthew Jacobs, Internal Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4054 to look at the new patch set (#20). Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

[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 20: Code-Review+2 (2 comments) rebased http://

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

2016-10-05 Thread Yonghyun Hwang (Code Review)
Yonghyun Hwang has posted comments on this change. Change subject: IMPALA-3342: Adding new timer to accurately measure the TotalCpuTime .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4633/1/be/src/util/stopwa

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

2016-10-05 Thread Marcel Kornacker (Code Review)
Hello Henry Robinson, Matthew Jacobs, Internal Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4054 to look at the new patch set (#21). Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend

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

[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! http://gerrit.cloudera.org:8080/#/c/4349/6/common/thrift

[Impala-ASF-CR] IMPALA-4239: fix buffer pool test failures in release build

2016-10-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4239: fix buffer pool test failures in release build .. Patch Set 2: Code-Review+2 Upgrade to +2 -- To view, visit http://gerrit.cloudera.org:8080/4596

[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) http://gerrit.cloudera.org:8080/#/c/4613/2/fe/src/main/java/org/apache/impala/ana

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

[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) http://gerrit.cloudera.org:8080/#/c/4613/2/fe/src/main/java/org/apache/impal

[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) http://gerrit.cloudera.org:8080/#/c/4448/6/be/src/exec/analytic-eval-node.c

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

2016-10-05 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#3). Change subject: IMPALA-4213: Planner not pushing some Kudu predicates .. IMPALA-4213: Planner not pushing some Kudu predicates Folding const exprs where there were implicit casts

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

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

[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) http://gerrit.cloudera.org:8080/#/c/4448/6/be/src/exec/analytic-eval-no

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

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

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

2016-10-05 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4204: Remove KuduScanNodeTest .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4609 To unsubscribe, visit http://gerrit

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

[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 (#7). Change subject: IMPALA-1702: Enforce table level consistency accross service .. IMPALA-1702: Enforce table level consistency accross service Problems: 1. CatalogServiceCatalog::reset(

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

2016-10-05 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4204: Remove KuduScanNodeTest .. Patch Set 2: Verified-1 Build failed: http://sandbox.jenkins.cloudera.com/job/impala-external-gerrit-verify-merge-ASF

[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 submitted this change and it was merged. Change subject: IMPALA-3229: Don't assume that AUX exists just because of shell env .. IMPALA-3229: Don't assume that AUX exists just because of shell env This caus

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

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

[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 be/src/

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

[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: (1 comment) http://gerrit.cloudera.org:8080/#/c/4448/6/be/src/exec/analytic-eval-node.cc

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

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

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

[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: TotalStorageWaitT

[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) http://gerrit.cloudera.org:8080/#/c/4349/7/fe/src/main/java/org/apache/imp

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

[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 submitted this change and it was merged. Change subject: IMPALA-3902: Scheduler improvements for running multiple fragment instances on a single backend .. IMPALA-3902: Scheduler improvements for running mu

[Impala-ASF-CR] IMPALA-4239: fix buffer pool test failures in release build

2016-10-05 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4239: fix buffer pool test failures in release build .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4596 To unsubscribe,

[Impala-ASF-CR] IMPALA-4239: fix buffer pool test failures in release build

2016-10-05 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4239: fix buffer pool test failures in release build .. IMPALA-4239: fix buffer pool test failures in release build Gtest's ASSERT_DEBUG_DEATH

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

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

2016-10-05 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4204: Remove KuduScanNodeTest .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4609 To unsubscribe, visit http://gerrit.cl

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

[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) http://gerrit.cloudera.org:8080/#/c/4349/7/fe/src/main/java/org/apache/imp

[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 (#8). Change subject: IMPALA-1702: Enforce table level consistency accross service .. IMPALA-1702: Enforce table level consistency accross service Problems: 1. CatalogServiceCatalog::reset(

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

2016-10-05 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4213: Fix Kudu predicates that need constant folding .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4613 To unsubscribe,

[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-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 be/src/benc

[Impala-ASF-CR] IMPALA-4231: fix codegen time regression

2016-10-05 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4231: fix codegen time regression .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/4623/1//COMMIT_MSG Commit Message: PS1, Line 26: y, whic