[Impala-CR](cdh5-trunk) IMPALA-3762: Download Python requirements before they are needed.

2016-06-21 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3762: Download Python requirements before they are needed. .. Patch Set 2: Build failed: http://sandbox.jenkins.cloudera.com/job/mikeb-gvm/32/ --

Re: Issues with Impala dataloading (power8 gcc4.9.3 Impala2.6)

2016-06-21 Thread Valencia Serrao
Thanks, Jim! I'll check the logs you mentioned. From: Jim Apple To: dev@impala.incubator.apache.org Cc: Manish Patil/Austin/Contr/IBM@IBMUS, Sudarshan Jagadale/Austin/Contr/IBM@IBMUS, Nishidha Panpaliya/Austin/Contr/IBM@IBMUS, Valencia

[Impala-CR](cdh5-trunk) Cleanup Python test import statements

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: Cleanup Python test import statements .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3444 To unsubscribe, visit

[Impala-CR](cdh5-trunk) IMPALA-3762: Download Python requirements before they are needed.

2016-06-21 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3762: Download Python requirements before they are needed. .. Patch Set 2: Build failed: http://sandbox.jenkins.cloudera.com/job/mikeb-gvm/31/ --

[Impala-CR](cdh5-trunk) IMPALA-2428: Support multiple-character string as the field delimiter

2016-06-21 Thread Yuanhao Luo (Code Review)
Yuanhao Luo has posted comments on this change. Change subject: IMPALA-2428: Support multiple-character string as the field delimiter .. Patch Set 1: (1 comment)

[Impala-CR](cdh5-trunk) IMPALA-2428: Support multiple-character string as the field delimiter

2016-06-21 Thread Yuanhao Luo (Code Review)
Yuanhao Luo has uploaded a new patch set (#4). Change subject: IMPALA-2428: Support multiple-character string as the field delimiter .. IMPALA-2428: Support multiple-character string as the field delimiter This commit add

[Impala-CR](cdh5-trunk) IMPALA-3575: Add retry to backend connection request and rpc timeout

2016-06-21 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#11). Change subject: IMPALA-3575: Add retry to backend connection request and rpc timeout .. IMPALA-3575: Add retry to backend connection request and rpc timeout This patch adds a

[Impala-CR](cdh5-trunk) IMPALA-3575: Add retry to backend connection request and rpc timeout

2016-06-21 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3575: Add retry to backend connection request and rpc timeout .. Patch Set 10: (4 comments)

[Impala-CR](cdh5-trunk) Cleanup Python test import statements

2016-06-21 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new change for review. http://gerrit.cloudera.org:8080/3444 Change subject: Cleanup Python test import statements .. Cleanup Python test import statements Many of our test scripts have import

[Impala-CR](cdh5-trunk) IMPALA-3767: bootstrap virtualenv fails to find cython distribution

2016-06-21 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3767: bootstrap_virtualenv fails to find cython distribution .. IMPALA-3767: bootstrap_virtualenv fails to find cython distribution This

[Impala-CR](cdh5-trunk) IMPALA-3767: bootstrap virtualenv fails to find cython distribution

2016-06-21 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3767: bootstrap_virtualenv fails to find cython distribution .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3427 To

[Impala-CR](cdh5-trunk) IMPALA-3674: Lazy materialization of LLVM module bitcode.

2016-06-21 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-3674: Lazy materialization of LLVM module bitcode. .. Patch Set 3: (14 comments) http://gerrit.cloudera.org:8080/#/c/3220/3/be/src/codegen/llvm-codegen.cc

[Impala-CR](cdh5-trunk) IMPALA-1619: Support 64-bit allocations.

2016-06-21 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-1619: Support 64-bit allocations. .. Patch Set 7: (15 comments) http://gerrit.cloudera.org:8080/#/c/2781/7//COMMIT_MSG Commit Message: PS7, Line 10: bug

[Impala-CR](cdh5-trunk) IMPALA-3674: Lazy materialization of LLVM module bitcode.

2016-06-21 Thread Michael Ho (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3220 to look at the new patch set (#4). Change subject: IMPALA-3674: Lazy materialization of LLVM module bitcode. ..

[Impala-CR](cdh5-trunk) IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2.

2016-06-21 Thread Youwei Wang (Code Review)
Youwei Wang has uploaded a new patch set (#25). Change subject: IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2. .. IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2. Using SSSE3/AVX2

[Impala-CR](cdh5-trunk) IMPALA-3753: Disable create table test for old aggs and joins

2016-06-21 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3753: Disable create table test for old aggs and joins .. Patch Set 1: (1 comment)

[Impala-CR](cdh5-trunk) IMPALA-3753: Disable create table test for old aggs and joins

2016-06-21 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3753: Disable create table test for old aggs and joins .. Patch Set 1: (1 comment)

[Impala-CR](cdh5-trunk) IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2.

2016-06-21 Thread Youwei Wang (Code Review)
Youwei Wang has uploaded a new patch set (#23). Change subject: IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2. .. IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2. Using SSSE3/AVX2

[Impala-CR](cdh5-trunk) IMPALA-3664: S3A test keys do not work fails

2016-06-21 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has submitted this change and it was merged. Change subject: IMPALA-3664: S3A test_keys_do_not_work fails .. IMPALA-3664: S3A test_keys_do_not_work fails Previously when we created a new connection to S3 via the

[Impala-CR](cdh5-trunk) IMPALA-3664: S3A test keys do not work fails

2016-06-21 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3664: S3A test_keys_do_not_work fails .. Patch Set 5: Code-Review+2 Verified+1 Rebase, carry +2. -- To view, visit http://gerrit.cloudera.org:8080/3392

[Impala-CR](cdh5-trunk) IMPALA-3664: S3A test keys do not work fails

2016-06-21 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3664: S3A test_keys_do_not_work fails .. Patch Set 4: Code-Review+2 Verified+1 > > (1 comment) > > Well, the harm is more complexity and potential for

[Impala-CR](cdh5-trunk) Use AVX2 operations to speedup Bloom filters by 10-100%.

2016-06-21 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: Use AVX2 operations to speedup Bloom filters by 10-100%. .. Patch Set 10: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3338 To unsubscribe,

[Impala-CR](cdh5-trunk) Use AVX2 operations to speedup Bloom filters by 10-100%.

2016-06-21 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: Use AVX2 operations to speedup Bloom filters by 10-100%. .. Use AVX2 operations to speedup Bloom filters by 10-100%. As a reminder, our Bloom

[Impala-CR](cdh5-trunk) IMPALA-1766: Misc. statistical functions. Implemented aggregate corr().

2016-06-21 Thread Anonymous Coward (Code Review)
Anonymous Coward #200 has posted comments on this change. Change subject: IMPALA-1766: Misc. statistical functions. Implemented aggregate corr(). .. Patch Set 1: > Abdur, are you still interested in working on this? Jim, i

[Impala-CR](cdh5-trunk) IMPALA-3753: Disable create table test for old aggs and joins

2016-06-21 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3753: Disable create table test for old aggs and joins .. Patch Set 1: (1 comment)

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#4). Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. IMPALA-3729: batch_size=1 coverage for avro scanner Also fix a stale comment in the avro scanner header.

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 2: (3 comments)

[Impala-CR](cdh5-trunk) IMPALA-3698: Fix Isilon permissions test

2016-06-21 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. Change subject: IMPALA-3698: Fix Isilon permissions test .. Patch Set 1: Code-Review+2 Do we typically wait for a successful jenkins verification before giving a +2? I read the

[Impala-CR](cdh5-trunk) IMPALA-3611: track unused Disk IO buffer memory

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3611: track unused Disk IO buffer memory .. Patch Set 9: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3246 To unsubscribe, visit

[Impala-CR](cdh5-trunk) IMPALA-3637,IMPALA-3636: refactor codegen constant replacement

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3637,IMPALA-3636: refactor codegen constant replacement .. Patch Set 5: (14 comments) Thanks for the review. I'll do a local perf run on targeted-perf

[Impala-CR](cdh5-trunk) IMPALA-3637,IMPALA-3636: refactor codegen constant replacement

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#5). Change subject: IMPALA-3637,IMPALA-3636: refactor codegen constant replacement .. IMPALA-3637,IMPALA-3636: refactor codegen constant replacement Merge the disparate codegen

[Impala-CR](cdh5-trunk) IMPALA-3698: Fix Isilon permissions test

2016-06-21 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new change for review. http://gerrit.cloudera.org:8080/3433 Change subject: IMPALA-3698: Fix Isilon permissions test .. IMPALA-3698: Fix Isilon permissions test Isilon OneFS 8.0 introduces a

[Impala-CR](cdh5-trunk) IMPALA-3762: Download Python requirements before they are needed.

2016-06-21 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3762: Download Python requirements before they are needed. .. Patch Set 2: Verified-1 Build failed:

[Impala-CR](cdh5-2.5.0 5.7.x) CDH-41243: Parquet scanner regression on wide tables

2016-06-21 Thread Huaisi Xu (Code Review)
Huaisi Xu has uploaded a new patch set (#2). Change subject: CDH-41243: Parquet scanner regression on wide tables .. CDH-41243: Parquet scanner regression on wide tables IMPALA-2473 introduced a check that prevent row batches

[Impala-CR](cdh5-trunk) IMPALA-3680: Cleanup the scan range state after failed hdfs cache reads

2016-06-21 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. Change subject: IMPALA-3680: Cleanup the scan range state after failed hdfs cache reads .. Patch Set 5: (1 comment)

[Impala-CR](cdh5-2.5.0 5.7.x) Revert "IMPALA-2473: reduce scanner memory usage"

2016-06-21 Thread Huaisi Xu (Code Review)
Huaisi Xu has posted comments on this change. Change subject: Revert "IMPALA-2473: reduce scanner memory usage" .. Patch Set 1: > Reverting the whole commit makes sense in 5.4.x, but there are > other changes in 5.7.x that

[Impala-CR](cdh5-trunk) IMPALA-3762: Download Python requirements before they are needed.

2016-06-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3762: Download Python requirements before they are needed. .. Patch Set 2: packaging build passed -- To view, visit http://gerrit.cloudera.org:8080/3426

[Impala-CR](cdh5-2.5.0 5.7.x) Revert "IMPALA-2473: reduce scanner memory usage"

2016-06-21 Thread Huaisi Xu (Code Review)
Huaisi Xu has posted comments on this change. Change subject: Revert "IMPALA-2473: reduce scanner memory usage" .. Patch Set 1: What about in 5.4.x? -- To view, visit http://gerrit.cloudera.org:8080/3429 To unsubscribe,

[Impala-CR](cdh5-trunk) IMPALA-3637,IMPALA-3636: refactor codegen constant replacement

2016-06-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3637,IMPALA-3636: refactor codegen constant replacement .. Patch Set 4: (14 comments)

[Impala-CR](cdh5-2.5.0 5.7.x) Revert "IMPALA-2473: reduce scanner memory usage"

2016-06-21 Thread Huaisi Xu (Code Review)
Huaisi Xu has posted comments on this change. Change subject: Revert "IMPALA-2473: reduce scanner memory usage" .. Patch Set 1: > (4 comments) > > I think we should only backport the changes to hdfs-parquet-scanner.cc >

[Impala-CR](cdh5-2.5.0 5.7.x) Revert "IMPALA-2473: reduce scanner memory usage"

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: Revert "IMPALA-2473: reduce scanner memory usage" .. Patch Set 1: (4 comments) I think we should only backport the changes to hdfs-parquet-scanner.cc and

[Impala-CR](cdh5-2.5.0 5.7.x) Revert "IMPALA-2473: reduce scanner memory usage"

2016-06-21 Thread Huaisi Xu (Code Review)
Huaisi Xu has posted comments on this change. Change subject: Revert "IMPALA-2473: reduce scanner memory usage" .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3429/1/be/src/runtime/row-batch.h File

[Impala-CR](cdh5-2.5.0 5.7.x) Revert "IMPALA-2473: reduce scanner memory usage"

2016-06-21 Thread Huaisi Xu (Code Review)
Huaisi Xu has posted comments on this change. Change subject: Revert "IMPALA-2473: reduce scanner memory usage" .. Patch Set 1: Local testing in a vm shows reverting this on 57x is 2x slower than on 5.4.10. I verified that

[Impala-CR](cdh5-trunk) IMPALA-3767: bootstrap virtualenv fails to find cython distribution

2016-06-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3767: bootstrap_virtualenv fails to find cython distribution .. Patch Set 2: Code-Review+2 (1 comment) Carry Tim's +2

[Impala-CR](cdh5-2.5.0 5.7.x) Revert "IMPALA-2473: reduce scanner memory usage"

2016-06-21 Thread Huaisi Xu (Code Review)
Huaisi Xu has uploaded a new change for review. http://gerrit.cloudera.org:8080/3429 Change subject: Revert "IMPALA-2473: reduce scanner memory usage" .. Revert "IMPALA-2473: reduce scanner memory usage" This reverts commit

[Impala-CR](cdh5-trunk) IMPALA-3767: bootstrap virtualenv fails to find cython distribution

2016-06-21 Thread Jim Apple (Code Review)
Hello Michael Ho, Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3427 to look at the new patch set (#2). Change subject: IMPALA-3767: bootstrap_virtualenv fails to find cython distribution

[Impala-CR](cdh5-trunk) IMPALA-3767: bootstrap virtualenv fails to find cython distribution

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3767: bootstrap_virtualenv fails to find cython distribution .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3427 To

[Impala-CR](cdh5-trunk) IMPALA-3767: bootstrap virtualenv fails to find cython distribution

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3767: bootstrap_virtualenv fails to find cython distribution .. Patch Set 1: (1 comment)

[Impala-CR](cdh5-trunk) IMPALA-3201: headers and reservation logic for new buffer pool

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#15). Change subject: IMPALA-3201: headers and reservation logic for new buffer pool .. IMPALA-3201: headers and reservation logic for new buffer pool This patch implements the core

[Impala-CR](cdh5-trunk) IMPALA-3201: headers and reservation logic for new buffer pool

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3201: headers and reservation logic for new buffer pool .. Patch Set 14: Rebased, fixed merge conflict and fixed a couple of minor typos/formatting

[Impala-CR](cdh5-trunk) IMPALA-3201: headers and reservation logic for new buffer pool

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#14). Change subject: IMPALA-3201: headers and reservation logic for new buffer pool .. IMPALA-3201: headers and reservation logic for new buffer pool This patch implements the core

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/3413/1//COMMIT_MSG Commit Message: PS1,

[Impala-CR](cdh5-trunk) IMPALA-3729: batch size=1 coverage for avro scanner

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#2). Change subject: IMPALA-3729: batch_size=1 coverage for avro scanner .. IMPALA-3729: batch_size=1 coverage for avro scanner Also fix a stale comment in the avro scanner header.

[Impala-CR](cdh5-trunk) IMPALA-3767: bootstrap virtualenv fails to find cython distribution

2016-06-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3767: bootstrap_virtualenv fails to find cython distribution .. Patch Set 1: Passed packaging build -- To view, visit http://gerrit.cloudera.org:8080/3427

[Impala-CR](cdh5-trunk) IMPALA-3767: bootstrap virtualenv fails to find cython distribution

2016-06-21 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-3767: bootstrap_virtualenv fails to find cython distribution .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3427 To

[Toolchain-CR] Add Kudu 0.9.0

2016-06-21 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has abandoned this change. Change subject: Add Kudu 0.9.0 .. Abandoned Looks like 0.9.0 is the same bits as 0.9.0 RC1, so this isn't needed. -- To view, visit http://gerrit.cloudera.org:8080/3404 To

[Impala-CR](cdh5-trunk) IMPALA-3637,IMPALA-3636: refactor codegen constant replacement

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3637,IMPALA-3636: refactor codegen constant replacement .. Patch Set 4: Yeah, the regression was because we started calling Expr::GetConstant() on

[Impala-CR](cdh5-trunk) IMPALA-3637,IMPALA-3636: refactor codegen constant replacement

2016-06-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3637,IMPALA-3636: refactor codegen constant replacement .. Patch Set 4: Does this fix the regression in IMPALA-3636? -- To view, visit

[Impala-CR](cdh5-trunk) IMPALA-3611: track unused Disk IO buffer memory

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3611: track unused Disk IO buffer memory .. Patch Set 9: Code-Review+2 Change 1L to 1LL and rebased -- To view, visit

[Impala-CR](cdh5-trunk) IMPALA-3575: Add retry to backend connection request and rpc timeout

2016-06-21 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3575: Add retry to backend connection request and rpc timeout .. Patch Set 10: (4 comments)

[Impala-CR](cdh5-trunk) IMPALA-3611: track unused Disk IO buffer memory

2016-06-21 Thread Tim Armstrong (Code Review)
Hello Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3246 to look at the new patch set (#8). Change subject: IMPALA-3611: track unused Disk IO buffer memory .. IMPALA-3611:

[Impala-CR](cdh5-trunk) IMPALA-3762: Download Python requirements before they are needed.

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3762: Download Python requirements before they are needed. .. Patch Set 2: Code-Review+2 Looks good, assuming packaging build etc passes. -- To view,

[Impala-CR](cdh5-trunk) IMPALA-3762: Download Python requirements before they are needed.

2016-06-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3762: Download Python requirements before they are needed. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3426/1/buildall.sh File

[Impala-CR](cdh5-trunk) IMPALA-3762: Download Python requirements before they are needed.

2016-06-21 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#2). Change subject: IMPALA-3762: Download Python requirements before they are needed. .. IMPALA-3762: Download Python requirements before they are needed. This is needed for ASF builds.

[Impala-CR](cdh5-trunk) IMPALA-3767: bootstrap virtualenv fails to find cython distribution

2016-06-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3767: bootstrap_virtualenv fails to find cython distribution .. Patch Set 1: testing packagin build now -- To view, visit

[Impala-CR](cdh5-trunk) IMPALA-3767: bootstrap virtualenv fails to find cython distribution

2016-06-21 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/3427 Change subject: IMPALA-3767: bootstrap_virtualenv fails to find cython distribution .. IMPALA-3767: bootstrap_virtualenv fails to find

[Impala-CR](cdh5-trunk) IMPALA-3762: Download Python requirements before they are needed.

2016-06-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3762: Download Python requirements before they are needed. .. Patch Set 1: Testing in packaging build now -- To view, visit

[Impala-CR](cdh5-trunk) IMPALA-3762: Download Python requirements before they are needed.

2016-06-21 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/3426 Change subject: IMPALA-3762: Download Python requirements before they are needed. .. IMPALA-3762: Download Python requirements before they

[Impala-CR](cdh5-trunk) Use AVX2 operations to speedup Bloom filters by 10-100%.

2016-06-21 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: Use AVX2 operations to speedup Bloom filters by 10-100%. .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/3338/10//COMMIT_MSG Commit Message:

[Impala-CR](cdh5-trunk) Use AVX2 operations to speedup Bloom filters by 10-100%.

2016-06-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Use AVX2 operations to speedup Bloom filters by 10-100%. .. Patch Set 10: Code-Review+2 (1 comment) Carry +2

[Impala-CR](cdh5-trunk) Use AVX2 operations to speedup Bloom filters by 10-100%.

2016-06-21 Thread Jim Apple (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3338 to look at the new patch set (#10). Change subject: Use AVX2 operations to speedup Bloom filters by 10-100%. ..

[Impala-CR](cdh5-trunk) Use AVX2 operations to speedup Bloom filters by 10-100%.

2016-06-21 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: Use AVX2 operations to speedup Bloom filters by 10-100%. .. Patch Set 8: Code-Review+2 (3 comments) Looks good, upgrading to +2 since it's all in the utils

[Impala-CR](cdh5-trunk) IMPALA-3680: Cleanup the scan range state after failed hdfs cache reads

2016-06-21 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3680: Cleanup the scan range state after failed hdfs cache reads .. Patch Set 5: Code-Review+2 (3 comments)

[Impala-CR](cdh5-trunk) IMPALA-3664: S3A test keys do not work fails

2016-06-21 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3664: S3A test_keys_do_not_work fails .. Patch Set 4: Code-Review+2 > (1 comment) Well, the harm is more complexity and potential for test failures, like

[Impala-CR](cdh5-trunk) IMPALA-3664: S3A test keys do not work fails

2016-06-21 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3664: S3A test_keys_do_not_work fails .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/3392/3/tests/custom_cluster/test_s3a_access.py

[Impala-CR](cdh5-trunk) IMPALA-3687: Prefer Avro field name during schema reconciliation

2016-06-21 Thread Huaisi Xu (Code Review)
Huaisi Xu has posted comments on this change. Change subject: IMPALA-3687: Prefer Avro field name during schema reconciliation .. Patch Set 6: (3 comments) > (1 comment) > > I still think that we're missing some testing

[Impala-CR](cdh5-trunk) IMPALA-3664: S3A test keys do not work fails

2016-06-21 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3664: S3A test_keys_do_not_work fails .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3392/3/tests/custom_cluster/test_s3a_access.py File

[Impala-CR](cdh5-trunk) IMPALA-3664: S3A test keys do not work fails

2016-06-21 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3664: S3A test_keys_do_not_work fails .. Patch Set 3: Code-Review+2 (1 comment) Carry +2.

[Impala-CR](cdh5-trunk) IMPALA-3664: S3A test keys do not work fails

2016-06-21 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3664: S3A test_keys_do_not_work fails .. Patch Set 3: Code-Review+2 (1 comment)

[Impala-CR](cdh5-trunk) IMPALA-2428: Support multiple-character string as the field delimiter

2016-06-21 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-2428: Support multiple-character string as the field delimiter .. Patch Set 1: (1 comment)

[Impala-CR](cdh5-trunk) IMPALA-2428: Support multiple-character string as the field delimiter

2016-06-21 Thread Yuanhao Luo (Code Review)
Yuanhao Luo has uploaded a new patch set (#3). Change subject: IMPALA-2428: Support multiple-character string as the field delimiter .. IMPALA-2428: Support multiple-character string as the field delimiter This commit add

[Impala-CR](cdh5-trunk) IMPALA-2428: Support multiple-character string as the field delimiter

2016-06-21 Thread Yuanhao Luo (Code Review)
Yuanhao Luo has posted comments on this change. Change subject: IMPALA-2428: Support multiple-character string as the field delimiter .. Patch Set 1: (11 comments) http://gerrit.cloudera.org:8080/#/c/3314/1//COMMIT_MSG

[Impala-CR](cdh5-trunk) IMPALA-3680: Cleanup the scan range state after failed hdfs cache reads

2016-06-21 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#5). Change subject: IMPALA-3680: Cleanup the scan range state after failed hdfs cache reads .. IMPALA-3680: Cleanup the scan range state after failed hdfs cache reads

[Impala-CR](cdh5-trunk) IMPALA-3680: Cleanup the scan range state after failed hdfs cache reads

2016-06-21 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-3680: Cleanup the scan range state after failed hdfs cache reads .. Patch Set 4: (1 comment)

[Impala-CR](cdh5-trunk) IMPALA-2428: Support multiple-character string as the field delimiter

2016-06-21 Thread Yuanhao Luo (Code Review)
Yuanhao Luo has uploaded a new patch set (#2). Change subject: IMPALA-2428: Support multiple-character string as the field delimiter .. IMPALA-2428: Support multiple-character string as the field delimiter This commit add

[Impala-CR](cdh5-trunk) IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2.

2016-06-21 Thread Youwei Wang (Code Review)
Youwei Wang has posted comments on this change. Change subject: IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2. .. Patch Set 22: (1 comment)

[Impala-CR](cdh5-trunk) IMPALA-3678: Impala FE SortNode analyzer conjuncts failure

2016-06-21 Thread Feng Guangyuan (Code Review)
Feng Guangyuan has uploaded a new change for review. http://gerrit.cloudera.org:8080/3420 Change subject: IMPALA-3678: Impala FE SortNode analyzer conjuncts_ failure .. IMPALA-3678: Impala FE SortNode analyzer conjuncts_

[Impala-CR](cdh5-trunk) IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2.

2016-06-21 Thread Youwei Wang (Code Review)
Youwei Wang has posted comments on this change. Change subject: IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2. .. Patch Set 22: (4 comments)

[Impala-CR](cdh5-trunk) IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2.

2016-06-21 Thread Youwei Wang (Code Review)
Youwei Wang has uploaded a new patch set (#22). Change subject: IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2. .. IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2. Using SSSE3/AVX2

[Impala-CR](cdh5-trunk) IMPALA-3763: download requirements fixes

2016-06-21 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3763: download_requirements fixes .. IMPALA-3763: download_requirements fixes * Download to infra/python/deps instead of the current

[Impala-CR](cdh5-trunk) IMPALA-3223: Supports download of CDH components from S3.

2016-06-21 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3223: Supports download of CDH components from S3. .. IMPALA-3223: Supports download of CDH components from S3. This change updates the

[Impala-CR](cdh5-trunk) IMPALA-3763: download requirements fixes

2016-06-21 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3763: download_requirements fixes .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3416 To unsubscribe, visit

[Impala-CR](cdh5-trunk) IMPALA-3223: Supports download of CDH components from S3.

2016-06-21 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3223: Supports download of CDH components from S3. .. Patch Set 7: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/ To unsubscribe,