[Impala-CR](cdh5-trunk) IMPALA-3754: fix TestParquet.test corrupt rle counts flakiness

2016-06-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/3402 Change subject: IMPALA-3754: fix TestParquet.test_corrupt_rle_counts flakiness .. IMPALA-3754: fix TestParquet.test_corrupt_rle_counts

[Impala-CR](cdh5-trunk) IMPALA-3757: Add missing lock in RuntimeProfile::ComputeTimeInProfile

2016-06-16 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3757: Add missing lock in RuntimeProfile::ComputeTimeInProfile .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3397 To

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

2016-06-16 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#7). 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) Update thirdparty dependencies

2016-06-16 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: Update thirdparty dependencies .. Patch Set 1: Code-Review+2 Verified+1 I ran a private build with the new thirdparty and it ran successfully:

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

2016-06-16 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#6). 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-3757: Add missing lock in RuntimeProfile::ComputeTimeInProfile

2016-06-16 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#2). Change subject: IMPALA-3757: Add missing lock in RuntimeProfile::ComputeTimeInProfile .. IMPALA-3757: Add missing lock in RuntimeProfile::ComputeTimeInProfile We've seen a

[Impala-CR](cdh5-trunk) IMPALA-3757: Add missing lock in RuntimeProfile::ComputeTimeInProfile

2016-06-16 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/3397 Change subject: IMPALA-3757: Add missing lock in RuntimeProfile::ComputeTimeInProfile .. IMPALA-3757: Add missing lock in

[Impala-CR](cdh5-trunk) IMPALA-2878: Fix Base64Decode error and remove duplicate codes.

2016-06-16 Thread Yuanhao Luo (Code Review)
Yuanhao Luo has posted comments on this change. Change subject: IMPALA-2878: Fix Base64Decode error and remove duplicate codes. .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/3311/4/be/src/util/url-coding.cc

[Impala-CR](cdh5-trunk) IMPALA-2878: Fix Base64Decode error and remove duplicate codes.

2016-06-16 Thread Yuanhao Luo (Code Review)
Hello Jim Apple, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3311 to look at the new patch set (#5). Change subject: IMPALA-2878: Fix Base64Decode error and remove duplicate codes. ..

[Impala-CR](cdh5-trunk) IMPALA-2347: Reuse metastore client connections in Catalog

2016-06-16 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-2347: Reuse metastore client connections in Catalog .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/3381/3//COMMIT_MSG Commit

[Impala-CR](cdh5-trunk) IMPALA-3737: Local filesystem build failed loading custom schemas

2016-06-16 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3737: Local filesystem build failed loading custom schemas .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3384 To

[Impala-CR](cdh5-trunk) IMPALA-3737: Local filesystem build failed loading custom schemas

2016-06-16 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3737: Local filesystem build failed loading custom schemas .. Patch Set 2: Code-Review+2 Carry +2. Fix syntax. Private build passed:

[Impala-CR](cdh5-trunk) IMPALA-3737: Local filesystem build failed loading custom schemas

2016-06-16 Thread Sailesh Mukil (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3384 to look at the new patch set (#2). Change subject: IMPALA-3737: Local filesystem build failed loading custom schemas

[Impala-CR](cdh5-trunk) IMPALA-2836: Change permissions for ADD/DROP PARTITION

2016-06-16 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-2836: Change permissions for ADD/DROP PARTITION .. Patch Set 5: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3327 To

[Impala-CR](cdh5-trunk) IMPALA-3711: Check privileges when necessary

2016-06-16 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-3711: Check privileges when necessary .. Patch Set 6: (2 comments)

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

2016-06-16 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3223: Supports download of CDH components from S3. .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/ To unsubscribe,

[Impala-CR](cdh5-trunk) IMPALA-3711: Check privileges when necessary

2016-06-16 Thread Huaisi Xu (Code Review)
Huaisi Xu has posted comments on this change. Change subject: IMPALA-3711: Check privileges when necessary .. Patch Set 6: (9 comments)

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

2016-06-16 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-3223: Supports download of CDH components from S3. .. Patch Set 4: (5 comments) http://gerrit.cloudera.org:8080/#/c//4//COMMIT_MSG Commit Message:

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

2016-06-16 Thread Michael Ho (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/ to look at the new patch set (#5). Change subject: IMPALA-3223: Supports download of CDH components from S3. ..

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

2016-06-16 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: Use AVX2 operations to speedup Bloom filters by 10-100%. .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/3338/7/be/src/util/bloom-filter.h File

[Impala-CR](cdh5-trunk) IMPALA-3499: Split catalog update

2016-06-16 Thread Huaisi Xu (Code Review)
Huaisi Xu has posted comments on this change. Change subject: IMPALA-3499: Split catalog update .. Patch Set 25: just realized gvm does not work.. I will do a private build -- To view, visit

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

2016-06-16 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 21: Hi Jim. If you don’t mind, I want to show you some performance data of

Re: Fw: LLVM codegen functionality

2016-06-16 Thread Nishidha Panpaliya
Hi Henry, Once we verify other tests with this setup (Impala 2.6, gcc 4.9.3 on ppc64le) and ensure things work, we can try to push the code back to Apache Impala project. But it would have some complications as we are not in sync with the latest cdh5-trunk branch. Latest cdh5-trunk had issues

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

2016-06-16 Thread Youwei Wang (Code Review)
Youwei Wang has uploaded a new patch set (#21). 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-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2.

2016-06-16 Thread Youwei Wang (Code Review)
Youwei Wang has uploaded a new patch set (#20). 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-1654: Support general predicates in most partition DDL operations.

2016-06-16 Thread Amos Bird (Code Review)
Amos Bird has posted comments on this change. Change subject: IMPALA-1654: Support general predicates in most partition DDL operations. .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/1563/4//COMMIT_MSG

[Impala-CR](cdh5-trunk) IMPALA-1654: Support general predicates in most partition DDL operations.

2016-06-16 Thread Amos Bird (Code Review)
Amos Bird has uploaded a new patch set (#5). Change subject: IMPALA-1654: Support general predicates in most partition DDL operations. .. IMPALA-1654: Support general predicates in most partition DDL operations. This commit

[Impala-CR](cdh5-trunk) IMPALA-3751: fix clang build errors and warnings

2016-06-16 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3751: fix clang build errors and warnings .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3391 To unsubscribe, visit

[Impala-CR](cdh5-trunk) IMPALA-3751: fix clang build errors and warnings

2016-06-16 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3751: fix clang build errors and warnings .. IMPALA-3751: fix clang build errors and warnings Fix misc clang errors and warnings. Change-Id:

[Impala-CR](cdh5-2.6.0 5.8.0) xfail s3a keys do not work custom cluster test

2016-06-16 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has submitted this change and it was merged. Change subject: xfail s3a_keys_do_not_work custom cluster test .. xfail s3a_keys_do_not_work custom cluster test This test will be re-enabled in the following release

[Impala-CR](cdh5-trunk) IMPALA-3587: Get rid of not default fs skip marker

2016-06-16 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has submitted this change and it was merged. Change subject: IMPALA-3587: Get rid of not_default_fs skip marker .. IMPALA-3587: Get rid of not_default_fs skip marker The SkipIf.not_default_fs used in