[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-13 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 14: Code-Review+1 Update comments and trivial change. Carry forward +1 -- To view, visit http

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-12 Thread Juan Yu (Code Review)
errit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Juan Yu Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Juan Yu Gerrit-Reviewer: Skye Wanderman-Milne

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-12 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 13: (8 comments) http://gerrit.cloudera.org:8080/#/c/2543/13/be/src/util/decompress-test.cc File

[Impala-CR](cdh5-trunk) IMPALA-2076: Correct execution time tracking for DataStreamSender.

2016-04-12 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#3). Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender. .. IMPALA-2076: Correct execution time tracking for DataStreamSender. DataStreamSender uses multiple

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-12 Thread Juan Yu (Code Review)
errit-Branch: cdh5-trunk Gerrit-Owner: Juan Yu Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Juan Yu Gerrit-Reviewer: Skye Wanderman-Milne

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-12 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 13: Code-Review+1 Carry Skye's +1 -- To view, visit http://gerrit.cloudera.org:8080/25

[Impala-CR](cdh5-trunk) IMPALA-2076: Correct execution time for DataStreamSender.

2016-04-08 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2076: Correct execution time for DataStreamSender. .. Patch Set 2: @Henry, Thanks for reviewing and suggestion. I'll think about adding a new counter typ

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-07 Thread Juan Yu (Code Review)
errit-Branch: cdh5-trunk Gerrit-Owner: Juan Yu Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Juan Yu Gerrit-Reviewer: Skye Wanderman-Milne

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-05 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/2543/10/be/src/util/decompress.cc File be/src

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-05 Thread Juan Yu (Code Review)
errit-Branch: cdh5-trunk Gerrit-Owner: Juan Yu Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Juan Yu Gerrit-Reviewer: Skye Wanderman-Milne

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-04-05 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/2543/10/be/src/util/decompress.cc File be/src

[Impala-CR](cdh5-trunk) IMPALA-2076: Correct execution time for DataStreamSender.

2016-04-01 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2076: Correct execution time for DataStreamSender. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/2578/2/be/src/runtime/client-cache.h File

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-30 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#10). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. IMPALA-3038: Add multistream gzip/bzip2 test coverage Change-Id: I9b0e1971145dd457e71fc9c00ce7c06fff8dea88

[Impala-CR](cdh5-trunk) IMPALA-2076: Correct execution time for DataStreamSender.

2016-03-30 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2076: Correct execution time for DataStreamSender. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/2578/2/be/src/runtime/client-cache.h File

[Impala-CR](cdh5-trunk) PREVIEW IMPALA-2076: Part2 - Show DataStreamSender in ExecSummary under EXCHANGE node.

2016-03-25 Thread Juan Yu (Code Review)
Juan Yu has abandoned this change. Change subject: PREVIEW IMPALA-2076: Part2 - Show DataStreamSender in ExecSummary under EXCHANGE node. .. Abandoned will take a different approach to add non-plan node profile to ExecSummary

[Impala-CR](cdh5-trunk) IMPALA-2076: Correct execution time for DataStreamSender.

2016-03-25 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#2). Change subject: IMPALA-2076: Correct execution time for DataStreamSender. .. IMPALA-2076: Correct execution time for DataStreamSender. Ignore receiver waiting time and only count

[Impala-CR](cdh5-2.2.0_5.4.x) CDH-38160: Prevent destructing uninitialized avro schema pointers

2016-03-25 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: CDH-38160: Prevent destructing uninitialized avro schema pointers .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2582 To

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-24 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#9). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. IMPALA-3038: Add multistream gzip/bzip2 test coverage Change-Id: I9b0e1971145dd457e71fc9c00ce7c06fff8dea88

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-24 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/2543/7/be/src/util/decompress-test.cc File be

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-24 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#8). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. IMPALA-3038: Add multistream gzip/bzip2 test coverage Change-Id: I9b0e1971145dd457e71fc9c00ce7c06fff8dea88

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-24 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/2543/7/be/src/util/decompress-test.cc File be

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-23 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#7). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. IMPALA-3038: Add multistream gzip/bzip2 test coverage Change-Id: I9b0e1971145dd457e71fc9c00ce7c06fff8dea88

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-23 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 6: (5 comments) http://gerrit.cloudera.org:8080/#/c/2543/6/be/src/util/decompress-test.cc File be

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-23 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 5: (7 comments) http://gerrit.cloudera.org:8080/#/c/2543/5/be/src/util/decompress-test.cc File be

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-23 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#6). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. IMPALA-3038: Add multistream gzip/bzip2 test coverage Change-Id: I9b0e1971145dd457e71fc9c00ce7c06fff8dea88

[Impala-CR](cdh5-trunk) IMPALA-2853: introduce PARQUET_RESOLVE_BY_NAME query option

2016-03-23 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2853: introduce PARQUET_RESOLVE_BY_NAME query option .. Patch Set 4: (4 comments) http://gerrit.cloudera.org:8080/#/c/2384/4/testdata/workloads/functional

[Impala-CR](cdh5-trunk) IMPALA-2853: introduce PARQUET_RESOLVE_BY_NAME query option

2016-03-23 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2853: introduce PARQUET_RESOLVE_BY_NAME query option .. Patch Set 4: This is for IMPALA-2835, isn't it? -- To view, visit http://gerrit.cloudera.org

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-22 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 4: (6 comments) http://gerrit.cloudera.org:8080/#/c/2543/4/be/src/util/decompress-test.cc File be

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-22 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#5). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. IMPALA-3038: Add multistream gzip/bzip2 test coverage Change-Id: I9b0e1971145dd457e71fc9c00ce7c06fff8dea88

[Impala-CR](cdh5-trunk) IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE

2016-03-22 Thread Juan Yu (Code Review)
chSet: 6 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Bharath Vissapragada Gerrit-Reviewer: Bharath Vissapragada Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Juan Yu Gerrit-Reviewer: Marcel Kornacker

[Impala-CR](cdh5-trunk) IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE

2016-03-22 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/2580/5/testdata/workloads

[Impala-CR](cdh5-trunk) IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE

2016-03-22 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/2580/4/testdata/workloads

[Impala-CR](cdh5-trunk) IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE

2016-03-22 Thread Juan Yu (Code Review)
oject: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Bharath Vissapragada Gerrit-Reviewer: Bharath Vissapragada Gerrit-Reviewer: Juan Yu Gerrit-Reviewer: Marcel Kornacker

[Impala-CR](cdh5-trunk) IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE

2016-03-22 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE .. Patch Set 4: -Code-Review (1 comment) we need to fix the test http://gerrit.cloudera.org

[Impala-CR](cdh5-trunk) IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE

2016-03-22 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/2580/4/testdata/workloads

[Impala-CR](cdh5-trunk) IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE

2016-03-22 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE .. Patch Set 4: Code-Review+2 rebased, carry Marcel's +2 -- To view, visit

[Impala-CR](cdh5-trunk) IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE

2016-03-22 Thread Juan Yu (Code Review)
Hello Marcel Kornacker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2580 to look at the new patch set (#4). Change subject: IMPALA-3194: Allow queries materializing scalar type columns in RC_FILE

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-21 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. Patch Set 3: (10 comments) http://gerrit.cloudera.org:8080/#/c/2543/3/be/src/testutil/gtest-util.h File be

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-21 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#4). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. IMPALA-3038: Add multistream gzip/bzip2 test coverage Change-Id: I9b0e1971145dd457e71fc9c00ce7c06fff8dea88

[Impala-CR](cdh5-2.2.0_5.4.x) CDH-38160: Prevent destructing uninitialized avro schema pointers

2016-03-21 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: CDH-38160: Prevent destructing uninitialized avro schema pointers .. Patch Set 4: Code-Review+2 In the commit message, could you mention this issue only present in

[Impala-CR](cdh5-2.2.0_5.4.x) CDH-38160: Prevent destructing uninitialized avro schema pointers

2016-03-21 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: CDH-38160: Prevent destructing uninitialized avro schema pointers .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/2582/2/testdata/workloads

Re: error starting up statestored

2016-03-19 Thread Juan Yu
Iirc, start a new shell should fix this as well. On Mar 16, 2016 5:30 PM, "Marcel Kornacker" wrote: > On Wed, Mar 16, 2016 at 5:24 PM, Tim Armstrong > wrote: > > It should be set by impala-config.sh: > > > > if [[ -n "$IMPALA_TOOLCHAIN" ]]; then > > > > > LD_LIBRARY_PATH="${LD_LIBRARY_PATH}:${IM

[Impala-CR](cdh5-2.5.0_5.7.0) Runtime filters tests

2016-03-19 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: Runtime filters tests .. Patch Set 5: do you plan to commit this to 5.7? or trunk -- To view, visit http://gerrit.cloudera.org:8080/2297 To unsubscribe, visit http

[Impala-CR](cdh5-trunk) PREVIEW IMPALA-2076: Part 1 - Correct execution time for DataStreamSender.

2016-03-19 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new change for review. http://gerrit.cloudera.org:8080/2578 Change subject: PREVIEW IMPALA-2076: Part 1 - Correct execution time for DataStreamSender. .. PREVIEW IMPALA-2076: Part 1 - Correct execution

[Impala-CR](cdh5-2.2.0_5.4.x) CDH-38160: Prevent destructing uninitialized avro schema pointers

2016-03-19 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: CDH-38160: Prevent destructing uninitialized avro schema pointers .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/2582/2/testdata

[Impala-CR](cdh5-trunk) PREVIEW IMPALA-2076: Part2 - Show DataStreamSender in ExecSummary under EXCHANGE node.

2016-03-19 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new change for review. http://gerrit.cloudera.org:8080/2579 Change subject: PREVIEW IMPALA-2076: Part2 - Show DataStreamSender in ExecSummary under EXCHANGE node. .. PREVIEW IMPALA-2076: Part2 - Show

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-18 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#3). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. IMPALA-3038: Add multistream gzip/bzip2 test coverage Change-Id: I9b0e1971145dd457e71fc9c00ce7c06fff8dea88

[Impala-CR](cdh5-2.2.0_5.4.x) CDH-38160: Prevent destructing uninitialized avro schema pointers

2016-03-18 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: CDH-38160: Prevent destructing uninitialized avro schema pointers .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/2582/2/testdata/workloads

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-14 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#2). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. IMPALA-3038: Add multistream gzip/bzip2 test coverage Change-Id: I9b0e1971145dd457e71fc9c00ce7c06fff8dea88

[Impala-CR](cdh5-trunk) IMPALA-3038: Add multistream gzip/bzip2 test coverage

2016-03-14 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new change for review. http://gerrit.cloudera.org:8080/2543 Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage .. IMPALA-3038: Add multistream gzip/bzip2 test coverage Change-Id

[Impala-CR](cdh5-trunk) IMPALA-1850 allow fs.defaultFS to be set to a non-HDFS filesystem

2016-03-09 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-1850 allow fs.defaultFS to be set to a non-HDFS filesystem .. Patch Set 7: (5 comments) http://gerrit.cloudera.org:8080/#/c/1121/7/fe/src/main/java/com

[Impala-CR](cdh5-2.4.0_5.6.x) IMPALA-2093: Disallow NOT IN aggregate subqueries with a constant lhs expr

2016-03-09 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2093: Disallow NOT IN aggregate subqueries with a constant lhs expr .. Patch Set 1: @Bharath, did you run test job for it? If all tests passed, please commit

[Impala-CR](cdh5-2.3.0_5.5.x) IMPALA-2093: Disallow NOT IN aggregate subqueries with a constant lhs expr

2016-03-09 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2093: Disallow NOT IN aggregate subqueries with a constant lhs expr .. Patch Set 1: @Bharath, did you run test job for it? If all tests passed, please commit

[Impala-CR](cdh5-2.5.0_5.7.0) OPSAPS-32457: Fix CatalogService Thrift changes to be backwards compatible.

2016-03-04 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: OPSAPS-32457: Fix CatalogService Thrift changes to be backwards compatible. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/2455/1/fe/src/main/java

[Impala-CR](cdh5-2.5.0_5.7.0) OPSAPS-32457: Fix CatalogService Thrift changes to be backwards compatible.

2016-03-04 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: OPSAPS-32457: Fix CatalogService Thrift changes to be backwards compatible. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/2455/1/fe/src/main/java

[Impala-CR](cdh5-2.5.0_5.7.0) OPSAPS-32457: Fix CatalogService Thrift changes to be backwards compatible.

2016-03-04 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: OPSAPS-32457: Fix CatalogService Thrift changes to be backwards compatible. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/2455/1/fe/src/main/java

[Impala-CR](cdh5-2.4.0_5.6.x) Validate the service while authorizing an internal connection

2016-03-03 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: Validate the service while authorizing an internal connection .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2444 To unsubscribe

[Impala-CR](cdh5-2.3.0_5.5.x) Validate the service while authorizing an internal connection

2016-03-03 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: Validate the service while authorizing an internal connection .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2438 To unsubscribe

[Impala-CR](cdh5-trunk) IMPALA-1939: Impalad should not require a co-located DataNode

2016-03-03 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-1939: Impalad should not require a co-located DataNode .. Patch Set 6: (3 comments) http://gerrit.cloudera.org:8080/#/c/1121/6/fe/src/main/java/com/cloudera

[Impala-CR](cdh5-2.2.0_5.4.x) Clean up FE pom.xml to eliminate console spew.

2016-03-03 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: Clean up FE pom.xml to eliminate console spew. .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2437 To unsubscribe, visit http

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-3085: Unregister data sinks' MemTrackers at their Close() functions.

2016-03-02 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3085: Unregister data sinks' MemTrackers at their Close() functions. .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/24

[Impala-CR](cdh5-trunk) IMPALA-1939: Impalad should not require a co-located DataNode

2016-03-02 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-1939: Impalad should not require a co-located DataNode .. Patch Set 5: (2 comments) Could you also rebase? http://gerrit.cloudera.org:8080/#/c/1121/5/fe/src

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-3034: Verify all consumed memory of a MemTracker is always released at destruction time.

2016-03-02 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3034: Verify all consumed memory of a MemTracker is always released at destruction time. .. Patch Set 1: Code-Review+2 -- To view, visit http

[Impala-CR](cdh5-2.2.0_5.4.x) Fixing indentation and cleaning up unwanted code after cherry-picking

2016-03-02 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: Fixing indentation and cleaning up unwanted code after cherry-picking .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2408 To

[Impala-CR](cdh5-trunk) Bump Impala version to 2.6.0

2016-03-02 Thread Juan Yu (Code Review)
Juan Yu has submitted this change and it was merged. Change subject: Bump Impala version to 2.6.0 .. Bump Impala version to 2.6.0 Change-Id: Ia1b8b6de080f7bdc59a5e350dadd36502aaecc4e Reviewed-on: http://gerrit.cloudera.org

[Impala-CR](cdh5-trunk) Bump Impala version to 2.6.0

2016-03-02 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: Bump Impala version to 2.6.0 .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/2409 To unsubscribe, visit http://gerrit.cloudera.org:8080

[Impala-CR](cdh5-trunk) Bump Impala version to 2.6.0

2016-03-02 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new change for review. http://gerrit.cloudera.org:8080/2409 Change subject: Bump Impala version to 2.6.0 .. Bump Impala version to 2.6.0 Change-Id: Ia1b8b6de080f7bdc59a5e350dadd36502aaecc4e --- M bin

[Impala-CR](cdh5-2.2.0_5.4.x) This is a combination of 6 commits. Fixing 5.4.x branch jenkins job error. Using virtualenv's impala-python.

2016-03-01 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: This is a combination of 6 commits. Fixing 5.4.x branch jenkins job error. Using virtualenv's impala-python. .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-2914: Fix DCHECK Check failed: HasDateOrTime()

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2914: Fix DCHECK Check failed: HasDateOrTime() .. Patch Set 2: Code-Review+2 (1 comment) I think we don't need to backport the new API ToUnixTimeInUTC()

[Impala-CR](cdh5-2.2.0_5.4.x) CDH-37841 Impala - Fix exhaustive test failure for 5.4.x branch Skipping test_compute_stats_impala_2201 in the 5.4.x branch

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: CDH-37841 Impala - Fix exhaustive test failure for 5.4.x branch Skipping test_compute_stats_impala_2201 in the 5.4.x branch .. Patch Set 1: Code-Review+2 -- To view

[Impala-CR](cdh5-2.2.0_5.4.x) Validate the service while authorizing an internal connection

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: Validate the service while authorizing an internal connection .. Patch Set 1: Code-Review+2 I see, Thanks. @Anuj, please backport IMPALA-3095 as well. -- To view

[Impala-CR](cdh5-2.2.0_5.4.x) Validate the service while authorizing an internal connection

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: Validate the service while authorizing an internal connection .. Patch Set 1: I don't know if customer uses internal hadoop users to access Impala's interna

[Impala-CR](cdh5-2.2.0_5.4.x) Validate the service while authorizing an internal connection

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: Validate the service while authorizing an internal connection .. Patch Set 1: -Code-Review even with IMPALA-3095, user still needs to whitelist all internal hadoop

[Impala-CR](cdh5-2.2.0_5.4.x) Validate the service while authorizing an internal connection

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: Validate the service while authorizing an internal connection .. Patch Set 1: @Sailesh, do you mean this commit could have some behavior regression? -- To view

[Impala-CR](cdh5-2.2.0_5.4.x) Validate the service while authorizing an internal connection

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: Validate the service while authorizing an internal connection .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2354 To unsubscribe

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-2643: Prevent migrating incorrectly inferred identity predicates into inline views.

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2643: Prevent migrating incorrectly inferred identity predicates into inline views. .. Patch Set 1: Code-Review+2 -- To view, visit http

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-2621: Fix flaky UNIX_TIMESTAMP() test

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2621: Fix flaky UNIX_TIMESTAMP() test .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2340 To unsubscribe, visit http

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-2914: Fix DCHECK Check failed: HasDateOrTime()

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2914: Fix DCHECK Check failed: HasDateOrTime() .. Patch Set 1: Looks like this change depends on some other commits (IMPALA-2190 and maybe more), could you

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-1702: Fix union query NPE if the inline view is constant select.

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-1702: Fix union query NPE if the inline view is constant select. .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2351 To

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-2264: fix edge cases for decimal/integer cast

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2264: fix edge cases for decimal/integer cast .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2344 To unsubscribe, visit http

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-1702: Fix crash when IMPALA-1702 hit even with workaround patch

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-1702: Fix crash when IMPALA-1702 hit even with workaround patch .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2350 To

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-1702: Fix NPE in validateTableIds()

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-1702: Fix NPE in validateTableIds() .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2348 To unsubscribe, visit http

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-1702: Dump fragment's thrift descriptor when a table's partition info is corrupt

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-1702: Dump fragment's thrift descriptor when a table's partition info is corrupt .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cl

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-3093: ReopenClient() could NULL out 'client_key' causing a crash

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3093: ReopenClient() could NULL out 'client_key' causing a crash .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:80

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-1702: Check for duplicate table IDs at the end of analysis

2016-02-29 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-1702: Check for duplicate table IDs at the end of analysis .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/2347 To

[Impala-CR](cdh5-2.2.0_5.4.x) This is a combination of 6 commits. Fixing 5.4.x branch jenkins job error. Using virtualenv's impala-python.

2016-02-28 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: This is a combination of 6 commits. Fixing 5.4.x branch jenkins job error. Using virtualenv's impala-python. .. Patch Set 1: (1 comment) Is this the only way t

[Impala-CR](cdh5-2.2.0_5.4.x) This is a combination of 6 commits. Fixing 5.4.x branch jenkins job error. Using virtualenv's impala-python.

2016-02-28 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: This is a combination of 6 commits. Fixing 5.4.x branch jenkins job error. Using virtualenv's impala-python. .. Patch Set 1: are there lots of merge conflict? W

[Impala-CR](cdh5-2.5.0_5.7.0) IMPALA-1886/IMPALA-2154: Add support for multi-stream bz2/gzip compressed files.

2016-02-27 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-1886/IMPALA-2154: Add support for multi-stream bz2/gzip compressed files. .. Patch Set 18: Code-Review+2 Exhaustive run passed, http

[Impala-CR](cdh5-2.5.0_5.7.0) IMPALA-1886/IMPALA-2154: Add support for multi-stream bz2/gzip compressed files.

2016-02-27 Thread Juan Yu (Code Review)
chset Gerrit-Change-Id: Icbe617d03a69953f0bf3aa0f7c30d34bc612f9f8 Gerrit-PatchSet: 18 Gerrit-Project: Impala Gerrit-Branch: cdh5-2.5.0_5.7.0 Gerrit-Owner: Juan Yu Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Juan Yu Gerrit-Reviewer: Skye Wanderman-Milne

[Impala-CR](cdh5-2.2.0_5.4.x) IMPALA-2798: After adding a column to avro table, Impala returns weird result if codegen is enabled.

2016-02-26 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2798: After adding a column to avro table, Impala returns weird result if codegen is enabled. .. Patch Set 1: Code-Review+2 They don't, we need to modif

[Impala-CR](cdh5-2.5.0_5.7.0) IMPALA-1886/IMPALA-2154: Add support for multi-stream bz2/gzip compressed files.

2016-02-26 Thread Juan Yu (Code Review)
chset Gerrit-Change-Id: Icbe617d03a69953f0bf3aa0f7c30d34bc612f9f8 Gerrit-PatchSet: 17 Gerrit-Project: Impala Gerrit-Branch: cdh5-2.5.0_5.7.0 Gerrit-Owner: Juan Yu Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Juan Yu Gerrit-Reviewer: Skye Wanderman-Milne

[Impala-CR](cdh5-2.5.0_5.7.0) IMPALA-1886/IMPALA-2154: Add support for multi-stream bz2/gzip compressed files.

2016-02-25 Thread Juan Yu (Code Review)
chset Gerrit-Change-Id: Icbe617d03a69953f0bf3aa0f7c30d34bc612f9f8 Gerrit-PatchSet: 16 Gerrit-Project: Impala Gerrit-Branch: cdh5-2.5.0_5.7.0 Gerrit-Owner: Juan Yu Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Juan Yu Gerrit-Reviewer: Skye Wanderman-Milne

[Impala-CR](cdh5-2.5.0_5.7.0) IMPALA-1886/IMPALA-2154: Add support for multi-stream bz2/gzip compressed files.

2016-02-25 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-1886/IMPALA-2154: Add support for multi-stream bz2/gzip compressed files. .. Patch Set 14: (9 comments) http://gerrit.cloudera.org:8080/#/c/2219/14/be/src

<    1   2