[Impala-ASF-CR] IMPALA-3912: test random rpc timeout is flaky.

2016-09-06 Thread Juan Yu (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4080 to look at the new patch set (#4). Change subject: IMPALA-3912: test_random_rpc_timeout is flaky. .. IMPALA-3912:

[Impala-ASF-CR] IMPALA-3912: test random rpc timeout is flaky.

2016-09-06 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3912: test_random_rpc_timeout is flaky. .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4080/3//COMMIT_MSG Commit Message: PS3, Line 11:

[Impala-ASF-CR] IMPALA-3912: test random rpc timeout is flaky.

2016-09-01 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3912: test_random_rpc_timeout is flaky. .. Patch Set 3: Code-Review+1 Carry previous +1 -- To view, visit http://gerrit.cloudera.org:8080/4080 To

[Impala-ASF-CR] IMPALA-3912: test random rpc timeout is flaky.

2016-09-01 Thread Juan Yu (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4080 to look at the new patch set (#3). Change subject: IMPALA-3912: test_random_rpc_timeout is flaky. .. IMPALA-3912:

[Impala-ASF-CR] IMPALA-3912: test random rpc timeout is flaky.

2016-09-01 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3912: test_random_rpc_timeout is flaky. .. Patch Set 2: I am not able to reproduce the test failure with this test locally and on a rhel machine. But I tried

[Impala-ASF-CR] IMPALA-3912: test random rpc timeout is flaky.

2016-08-31 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3912: test_random_rpc_timeout is flaky. .. Patch Set 2: Code-Review+1 Carry Sailesh's +1 -- To view, visit http://gerrit.cloudera.org:8080/4080 To

[Impala-ASF-CR] IMPALA-3912: test random rpc timeout is flaky.

2016-08-31 Thread Juan Yu (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4080 to look at the new patch set (#2). Change subject: IMPALA-3912: test_random_rpc_timeout is flaky. .. IMPALA-3912:

[Impala-ASF-CR] IMPALA-3912: test random rpc timeout is flaky.

2016-08-31 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3912: test_random_rpc_timeout is flaky. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4080/1/be/src/service/fragment-mgr.cc File

[Impala-ASF-CR] IMPALA-3912: test random rpc timeout is flaky.

2016-08-22 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3912: test_random_rpc_timeout is flaky. .. Patch Set 1: Couldn't find why the metric "num-fragments-in-flight" not back to 0. One suspect is a query might

[Impala-ASF-CR] IMPALA-3912: test random rpc timeout is flaky.

2016-08-22 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new change for review. http://gerrit.cloudera.org:8080/4080 Change subject: IMPALA-3912: test_random_rpc_timeout is flaky. .. IMPALA-3912: test_random_rpc_timeout is flaky. Change-Id:

[Impala-ASF-CR] IMPALA-3470: DecompressorTest is flaky.

2016-08-15 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3470: DecompressorTest is flaky. .. Patch Set 3: Code-Review+2 Rebase, Carry Dan's +2 -- To view, visit http://gerrit.cloudera.org:8080/3954 To unsubscribe,

[Impala-ASF-CR] IMPALA-3470: DecompressorTest is flaky.

2016-08-12 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3470: DecompressorTest is flaky. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3954/1/be/src/util/decompress-test.cc File

[Impala-ASF-CR] IMPALA-3470: DecompressorTest is flaky.

2016-08-12 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#2). Change subject: IMPALA-3470: DecompressorTest is flaky. .. IMPALA-3470: DecompressorTest is flaky. Make sure the random number is greater than 0 and stream is indeed truncated.

[Impala-ASF-CR] IMPALA-3470: DecompressorTest is flaky.

2016-08-12 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new change for review. http://gerrit.cloudera.org:8080/3954 Change subject: IMPALA-3470: DecompressorTest is flaky. .. IMPALA-3470: DecompressorTest is flaky. Make sure the random number is greater

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

2016-07-08 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 21: (15 comments)

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

2016-07-07 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 20: (3 comments)

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

2016-07-06 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 20: (1 comment)

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

2016-07-06 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 20: (23 comments) http://gerrit.cloudera.org:8080/#/c/3343/20//COMMIT_MSG Commit

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

2016-07-02 Thread Juan Yu (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3343 to look at the new patch set (#19). Change subject: IMPALA-3575: Add retry to backend connection request and rpc timeout

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

2016-07-02 Thread Juan Yu (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3343 to look at the new patch set (#18). Change subject: IMPALA-3575: Add retry to backend connection request and rpc timeout

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

2016-07-02 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 17: (17 comments)

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

2016-06-30 Thread Juan Yu (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3343 to look at the new patch set (#17). Change subject: IMPALA-3575: Add retry to backend connection request and rpc timeout

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

2016-06-30 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 16: (6 comments)

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

2016-06-29 Thread Juan Yu (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3343 to look at the new patch set (#16). Change subject: IMPALA-3575: Add retry to backend connection request and rpc timeout

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

2016-06-29 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 15: (11 comments)

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

2016-06-28 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 15: (1 comment)

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

2016-06-24 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 15: exhaustive test and stress test are green.

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

2016-06-24 Thread Juan Yu (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3343 to look at the new patch set (#15). Change subject: IMPALA-3575: Add retry to backend connection request and rpc timeout

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

2016-06-22 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 14: Code-Review+1 Carry Sailesh's +1 -- To view, visit

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

2016-06-22 Thread Juan Yu (Code Review)
Hello Sailesh Mukil, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3343 to look at the new patch set (#14). Change subject: IMPALA-3575: Add retry to backend connection request and rpc timeout

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

2016-06-22 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 13: (1 comment) I added a section in commit message. Is it enough for explain the

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

2016-06-22 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#13). 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-22 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 12: (3 comments)

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

2016-06-22 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#12). 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-22 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 11: (2 comments)

[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) IMPALA-3575: Add retry to backend connection request and rpc timeout

2016-06-20 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#10). 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-20 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 9: (7 comments) http://gerrit.cloudera.org:8080/#/c/3343/9//COMMIT_MSG Commit

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

2016-06-20 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 9: (1 comment)

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

2016-06-19 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#9). 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-17 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#8). 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-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) 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-3575: Add retry to backend connection request and rpc timeout

2016-06-14 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 5: (4 comments)

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

2016-06-13 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#5). 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-13 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 4: (7 comments)

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

2016-06-09 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 1: (1 comment)

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

2016-06-09 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#4). Change subject: IMPALA-3575: Add retry to backend connection request and rpc timeout .. IMPALA-3575: Add retry to backend connection request and rpc timeout Impala doesn't set socket

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

2016-06-09 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#3). Change subject: IMPALA-3575: Add retry to backend connection request and rpc timeout .. IMPALA-3575: Add retry to backend connection request and rpc timeout Impala doesn't set socket

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

2016-06-09 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new change for review. http://gerrit.cloudera.org:8080/3343 Change subject: IMPALA-3575: Add retry to backend connection request and rpc timeout .. IMPALA-3575: Add retry to backend connection request

[Impala-CR](cdh5-2.5.0_5.7.x) IMPALA-3396: Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure.

2016-04-28 Thread Juan Yu (Code Review)
Juan Yu has submitted this change and it was merged. Change subject: IMPALA-3396: Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure. .. IMPALA-3396: Fix ConcurrentTimerCounter unit test "TimerCounterTest"

[Impala-CR](cdh5-2.5.0_5.7.x) IMPALA-3396: Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure.

2016-04-28 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3396: Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure. .. Patch Set 1: Code-Review+2 Verified+1 Clean cherry-pick, verified by

[Impala-CR](cdh5-2.5.0_5.7.x) IMPALA-2076: Correct execution time tracking for DataStreamSender.

2016-04-28 Thread Juan Yu (Code Review)
Juan Yu has submitted this change and it was merged. Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender. .. IMPALA-2076: Correct execution time tracking for DataStreamSender. DataStreamSender

[Impala-CR](cdh5-2.5.0_5.7.x) IMPALA-2076: Correct execution time tracking for DataStreamSender.

2016-04-28 Thread Juan Yu (Code Review)
Hello Internal Jenkins, Dan Hecht, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/2897 to review the following change. Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender.

[Impala-CR](cdh5-2.5.0_5.7.x) IMPALA-2076: Correct execution time tracking for DataStreamSender.

2016-04-28 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender. .. Patch Set 1: Code-Review+2 Verified+1 Clean cherry pick. verified by

[Impala-CR](cdh5-2.5.0_5.7.x) IMPALA-3396: Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure.

2016-04-28 Thread Juan Yu (Code Review)
Hello Internal Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/2898 to review the following change. Change subject: IMPALA-3396: Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure.

[Impala-CR](cdh5-trunk) IMPALA-2660: Respect auth_to_local configs from hdfs configs

2016-04-22 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2660: Respect auth_to_local configs from hdfs configs .. Patch Set 3: (1 comment)

[Impala-CR](cdh5-trunk) IMPALA-3396: Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure.

2016-04-22 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3396: Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure. .. Patch Set 4: Code-Review+2 Add JIRA number to commit message. carry Tim's +2 -- To

[Impala-CR](cdh5-trunk) IMPALA-2660: Respect auth_to_local configs from hdfs configs

2016-04-22 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2660: Respect auth_to_local configs from hdfs configs .. Patch Set 3: (1 comment)

[Impala-CR](cdh5-trunk) Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure.

2016-04-22 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#3). Change subject: Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure. .. Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure. On ec2 machine, the timer error

[Impala-CR](cdh5-trunk) Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure.

2016-04-22 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure. .. Patch Set 2: (1 comment)

[Impala-CR](cdh5-trunk) Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure.

2016-04-21 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure. .. Patch Set 2: (1 comment)

[Impala-CR](cdh5-trunk) Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure.

2016-04-21 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#2). Change subject: Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure. .. Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure. On ec2 machine, the timer error

[Impala-CR](cdh5-trunk) Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure.

2016-04-21 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new change for review. http://gerrit.cloudera.org:8080/2838 Change subject: Fix ConcurrentTimerCounter unit test "TimerCounterTest" failure. .. Fix ConcurrentTimerCounter unit test "TimerCounterTest"

[Impala-CR](cdh5-trunk) IMPALA-3382: Long warnings cause impala-shell to stall

2016-04-21 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-3382: Long warnings cause impala-shell to stall .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/2821/1//COMMIT_MSG Commit Message: Line 7:

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

2016-04-20 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender. .. Patch Set 8: Code-Review+1 Carry previous +1 -- To view, visit

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

2016-04-20 Thread Juan Yu (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2578 to look at the new patch set (#8). Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender.

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

2016-04-20 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender. .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/2578/6//COMMIT_MSG Commit

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

2016-04-20 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender. .. Patch Set 6: (1 comment)

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

2016-04-20 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender. .. Patch Set 6: (1 comment)

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

2016-04-20 Thread Juan Yu (Code Review)
Hello Henry Robinson, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2578 to look at the new patch set (#7). Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender.

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

2016-04-20 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender. .. Patch Set 6: (2 comments)

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

2016-04-19 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#6). 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-2076: Correct execution time tracking for DataStreamSender.

2016-04-19 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender. .. Patch Set 5: (14 comments)

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

2016-04-15 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#5). 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-2076: Correct execution time tracking for DataStreamSender.

2016-04-15 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender. .. Patch Set 4: (11 comments)

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

2016-04-14 Thread Juan Yu (Code Review)
Juan Yu has uploaded a new patch set (#4). 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-2076: Correct execution time tracking for DataStreamSender.

2016-04-14 Thread Juan Yu (Code Review)
Juan Yu has posted comments on this change. Change subject: IMPALA-2076: Correct execution time tracking for DataStreamSender. .. Patch Set 3: (14 comments)

[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 15: Code-Review+2 Carry Dan's +2 -- To view, visit http://gerrit.cloudera.org:8080/2543 To

[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: (1 comment) http://gerrit.cloudera.org:8080/#/c/2543/14/be/src/util/decompress-test.cc File

[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: rebase, carry Dan's +2 -- To view, visit http://gerrit.cloudera.org:8080/2543 To unsubscribe,

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

2016-04-13 Thread Juan Yu (Code Review)
Hello Skye Wanderman-Milne, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2543 to look at the new patch set (#15). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage

[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

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

2016-04-12 Thread Juan Yu (Code Review)
Hello Skye Wanderman-Milne, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2543 to look at the new patch set (#14). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage

[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-3038: Add multistream gzip/bzip2 test coverage

2016-04-12 Thread Juan Yu (Code Review)
Hello Skye Wanderman-Milne, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2543 to look at the new patch set (#13). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage

[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/2543 To

[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 type for

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

2016-04-07 Thread Juan Yu (Code Review)
Hello Skye Wanderman-Milne, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2543 to look at the new patch set (#12). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage

[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

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

2016-04-05 Thread Juan Yu (Code Review)
Hello Skye Wanderman-Milne, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2543 to look at the new patch set (#11). Change subject: IMPALA-3038: Add multistream gzip/bzip2 test coverage

[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

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

  1   2   >