[Impala-ASF-CR] IMPALA-5532: Stack-allocate compressors in RowBatch (de)serialization

2017-07-20 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/7478 Change subject: IMPALA-5532: Stack-allocate compressors in RowBatch (de)serialization .. IMPALA-5532: Stack-allocate compressors in Row

[Impala-ASF-CR] IMPALA-5627: fix dropped statuses in HDFS writers

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5627: fix dropped statuses in HDFS writers .. Patch Set 8: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7372 To unsubscribe, visi

[Impala-ASF-CR] IMPALA-5627: fix dropped statuses in HDFS writers

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5627: fix dropped statuses in HDFS writers .. IMPALA-5627: fix dropped statuses in HDFS writers The change is mostly mechanical - added S

[Impala-ASF-CR] IMPALA-5315: Cast to timestamp fails for YYYY-M-D format

2017-07-20 Thread Vincent Tran (Code Review)
Vincent Tran has posted comments on this change. Change subject: IMPALA-5315: Cast to timestamp fails for -M-D format .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/7009/5/be/src/runtime/timestamp-parse-ut

[Impala-ASF-CR] IMPALA-5275: Avoid printing status stack trace on hot paths

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths .. IMPALA-5275: Avoid printing status stack trace on hot paths Currently, creation o

[Impala-ASF-CR] IMPALA-5275: Avoid printing status stack trace on hot paths

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7449 To unsubsc

[Impala-ASF-CR] IMPALA-4276: Profile displays non-default query options set by planner

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4276: Profile displays non-default query options set by planner .. Patch Set 7: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/741

[Impala-ASF-CR] IMPALA-4276: Profile displays non-default query options set by planner

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4276: Profile displays non-default query options set by planner .. IMPALA-4276: Profile displays non-default query options set by planner

[Impala-ASF-CR] IMPALA-5167: Reduce the number of Kudu clients created (FE)

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE) .. Patch Set 5: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/9

[Impala-ASF-CR] IMPALA-5688: Reduce run-time of a couple of expr-test heavy-hitters

2017-07-20 Thread anujphadke (Code Review)
anujphadke has posted comments on this change. Change subject: IMPALA-5688: Reduce run-time of a couple of expr-test heavy-hitters .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7474 To unsubs

[Impala-ASF-CR] IMPALA-5689: Avoid inverting non-equi right joins

2017-07-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5689: Avoid inverting non-equi right joins .. Patch Set 3: Code-Review+1 LGTM. Maybe Bharath should take a look too. -- To view, visit http://gerrit.clo

[Impala-ASF-CR] IMPALA-5689: Avoid inverting non-equi right joins

2017-07-20 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#3). Change subject: IMPALA-5689: Avoid inverting non-equi right joins .. IMPALA-5689: Avoid inverting non-equi right joins When checking if a join can be inverted, we forgot to al

[Impala-ASF-CR] IMPALA-5689: Avoid inverting non-equi right joins

2017-07-20 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#3). Change subject: IMPALA-5689: Avoid inverting non-equi right joins .. IMPALA-5689: Avoid inverting non-equi right joins When checking if a join can be inverted, we forgot to al

[Impala-ASF-CR] IMPALA-5689: Avoid inverting non-equi right joins

2017-07-20 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-5689: Avoid inverting non-equi right joins .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7476/2/fe/src/main/java/org/apache/impala/

[Impala-ASF-CR] IMPALA-5498: Support for partial sorts in Kudu INSERTs

2017-07-20 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5498: Support for partial sorts in Kudu INSERTs .. Patch Set 7: Code-Review+2 You probably need to rebase -- To view, visit http://gerrit.cloudera.org:

[Impala-ASF-CR] IMPALA-5689: Avoid inverting non-equi right joins

2017-07-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5689: Avoid inverting non-equi right joins .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7476/2/fe/src/main/java/org/apache/impala/plan

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-20 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 5: Code-Review+1 (1 comment) Carrying +1. http://gerrit.cloudera.or

[Impala-ASF-CR] IMPALA-5689: Avoid inverting non-equi right joins

2017-07-20 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#2). Change subject: IMPALA-5689: Avoid inverting non-equi right joins .. IMPALA-5689: Avoid inverting non-equi right joins When checking if a join can be inverted, we forgot to al

[Impala-ASF-CR] IMPALA-5689: Avoid inverting non-equi right joins

2017-07-20 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#2). Change subject: IMPALA-5689: Avoid inverting non-equi right joins .. IMPALA-5689: Avoid inverting non-equi right joins When checking if a join can be inverted, we forgot to al

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-20 Thread Bharath Vissapragada (Code Review)
Hello Tim Armstrong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7379 to look at the new patch set (#5). Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs

[Impala-ASF-CR] IMPALA-5689: Avoid inverting non-equi right joins

2017-07-20 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-5689: Avoid inverting non-equi right joins .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7476/1/fe/src/main/java/org/apache/impala/

[Impala-ASF-CR] IMPALA-5689: Avoid inverting non-equi right joins

2017-07-20 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#2). Change subject: IMPALA-5689: Avoid inverting non-equi right joins .. IMPALA-5689: Avoid inverting non-equi right joins When checking if a join can be inverted, we forgot to al

[Impala-ASF-CR] IMPALA-5689: Avoid inverting non-equi right joins

2017-07-20 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#2). Change subject: IMPALA-5689: Avoid inverting non-equi right joins .. IMPALA-5689: Avoid inverting non-equi right joins When checking if a join can be inverted, we forgot to al

[Impala-ASF-CR] IMPALA-5275: Avoid printing status stack trace on hot paths

2017-07-20 Thread Mostafa Mokhtar (Code Review)
Mostafa Mokhtar has posted comments on this change. Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7449/3//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-5582: Store sentry privileges in lower case

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5582: Store sentry privileges in lower case .. Patch Set 10: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/900/ --

[Impala-ASF-CR] IMPALA-5676: avoid expensive consistency checks in BTSv2

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5676: avoid expensive consistency checks in BTSv2 .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7459 To unsubscrib

[Impala-ASF-CR] IMPALA-5676: avoid expensive consistency checks in BTSv2

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5676: avoid expensive consistency checks in BTSv2 .. IMPALA-5676: avoid expensive consistency checks in BTSv2 Doing an O(n) consistency c

[Impala-ASF-CR] IMPALA-5627: fix dropped statuses in HDFS writers

2017-07-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5627: fix dropped statuses in HDFS writers .. Patch Set 8: Code-Review+2 Added a missing check. -- To view, visit http://gerrit.cloudera.org:8080/7372 T

[Impala-ASF-CR] IMPALA-5627: fix dropped statuses in HDFS writers

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5627: fix dropped statuses in HDFS writers .. Patch Set 8: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/904/ -- To view, vi

[Impala-ASF-CR] IMPALA-5627: fix dropped statuses in HDFS writers

2017-07-20 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, Lars Volker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7372 to look at the new patch set (#8). Change subject: IMPALA-5627: fix dropped statuses in HDFS writers .

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 4: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/7379/4/fe/

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-20 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/7379/1/fe/src/mai

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-20 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#4). Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. IMPALA-5615: Fix compute incremental stats for general partition exprs The fix fo

[Impala-ASF-CR] IMPALA-5689: Avoid inverting non-equi right joins

2017-07-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5689: Avoid inverting non-equi right joins .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7476/1/fe/src/main/java/org/apache/impala/plan

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-20 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#3). Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. IMPALA-5615: Fix compute incremental stats for general partition exprs The fix fo

[Impala-ASF-CR] IMPALA-5689: Avoid inverting non-equi right joins

2017-07-20 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new change for review. http://gerrit.cloudera.org:8080/7476 Change subject: IMPALA-5689: Avoid inverting non-equi right joins .. IMPALA-5689: Avoid inverting non-equi right joins When checking

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7379/1/fe/src/main/java/

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-20 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7379/1/fe/src/mai

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-20 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new patch set (#2). Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. IMPALA-5615: Fix compute incremental stats for general partition exprs The fix fo

[Impala-ASF-CR] IMPALA-5627: fix dropped statuses in HDFS writers

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5627: fix dropped statuses in HDFS writers .. Patch Set 7: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/898/ -- T

[Impala-ASF-CR] IMPALA-4669: [KUTIL] Add kudu util library to the build.

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4669: [KUTIL] Add kudu_util library to the build. .. Patch Set 25: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/896

[Impala-ASF-CR] IMPALA-5489: Improve Sentry authorization for Kudu tables

2017-07-20 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new patch set (#2). Change subject: IMPALA-5489: Improve Sentry authorization for Kudu tables .. IMPALA-5489: Improve Sentry authorization for Kudu tables IMPALA-4000 added basic authorization suppo

[Impala-ASF-CR] IMPALA-5529: Add additional function signatures for TRUNC()

2017-07-20 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5529: Add additional function signatures for TRUNC() .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7450/2/fe/src/test/java/org

[Impala-ASF-CR] IMPALA-5275: Avoid printing status stack trace on hot paths

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths .. Patch Set 3: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/903/ -- T

[Impala-ASF-CR] IMPALA-4276: Profile displays non-default query options set by planner

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4276: Profile displays non-default query options set by planner .. Patch Set 7: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun

[Impala-ASF-CR] IMPALA-5275: Avoid printing status stack trace on hot paths

2017-07-20 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7449 To unsubscribe

[Impala-ASF-CR] IMPALA-4276: Profile displays non-default query options set by planner

2017-07-20 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4276: Profile displays non-default query options set by planner .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7419 To

[Impala-ASF-CR] IMPALA-5275: Avoid printing status stack trace on hot paths

2017-07-20 Thread Bikramjeet Vig (Code Review)
Bikramjeet Vig has posted comments on this change. Change subject: IMPALA-5275: Avoid printing status stack trace on hot paths .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/7449/2/be/src/common/status.cc Fil

[Impala-ASF-CR] IMPALA-5167: Reduce the number of Kudu clients created (FE)

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE) .. Patch Set 5: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/901/ -- T

[Impala-ASF-CR] IMPALA-5167: Reduce the number of Kudu clients created (FE)

2017-07-20 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE) .. Patch Set 5: Code-Review+2 GVO failed due to IMPALA-5686. Rebased to include the fix for

[Impala-ASF-CR] IMPALA-5167: Reduce the number of Kudu clients created (FE)

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE) .. Patch Set 4: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/8

[Impala-ASF-CR] IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7

[Impala-ASF-CR] IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS .. IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE N

[Impala-ASF-CR] IMPALA-5582: Store sentry privileges in lower case

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5582: Store sentry privileges in lower case .. Patch Set 10: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/900/ -- To view,

[Impala-ASF-CR] IMPALA-5582: Store sentry privileges in lower case

2017-07-20 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5582: Store sentry privileges in lower case .. Patch Set 10: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7332 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5582: Store sentry privileges in lower case

2017-07-20 Thread anujphadke (Code Review)
Hello Impala Public Jenkins, Michael Brown, Bharath Vissapragada, Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7332 to look at the new patch set (#10). Change subject: IMPALA-5582: Store sentry privileges in lower case ..

[Impala-ASF-CR] IMPALA-5676: avoid expensive consistency checks in BTSv2

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5676: avoid expensive consistency checks in BTSv2 .. Patch Set 5: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/899/ -- To v

[Impala-ASF-CR] IMPALA-5676: avoid expensive consistency checks in BTSv2

2017-07-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5676: avoid expensive consistency checks in BTSv2 .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7459 To unsubscribe, vi

[Impala-ASF-CR] IMPALA-5676: avoid expensive consistency checks in BTSv2

2017-07-20 Thread Tim Armstrong (Code Review)
Hello Impala Public Jenkins, Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7459 to look at the new patch set (#5). Change subject: IMPALA-5676: avoid expensive consistency checks in BTSv2 ...

[Impala-ASF-CR] IMPALA-5394: Handle blocked HS2 connections

2017-07-20 Thread John Sherman (Code Review)
John Sherman has posted comments on this change. Change subject: IMPALA-5394: Handle blocked HS2 connections .. Patch Set 3: It looks like it'll be next week (or this weekend) before I'm able to make some time to write/test th

[Impala-ASF-CR] IMPALA-5627: fix dropped statuses in HDFS writers

2017-07-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5627: fix dropped statuses in HDFS writers .. Patch Set 7: Code-Review+2 rebase -- To view, visit http://gerrit.cloudera.org:8080/7372 To unsubscribe, v

[Impala-ASF-CR] IMPALA-5627: fix dropped statuses in HDFS writers

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5627: fix dropped statuses in HDFS writers .. Patch Set 7: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/898/ -- To view, vi

[Impala-ASF-CR] IMPALA-5688: Reduce run-time of a couple of expr-test heavy-hitters

2017-07-20 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/7474 Change subject: IMPALA-5688: Reduce run-time of a couple of expr-test heavy-hitters .. IMPALA-5688: Reduce run-time of a couple of expr

[Impala-ASF-CR] IMPALA-5676: avoid expensive consistency checks in BTSv2

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5676: avoid expensive consistency checks in BTSv2 .. Patch Set 4: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/895/

[Impala-ASF-CR] IMPALA-5627: fix dropped statuses in HDFS writers

2017-07-20 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-5627: fix dropped statuses in HDFS writers .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7372 To unsubscribe, visit http:

[Impala-ASF-CR] IMPALA-5167: Reduce the number of Kudu clients created (FE)

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE) .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/897/ -- T

[Impala-ASF-CR] IMPALA-4669: [KUTIL] Add kudu util library to the build.

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4669: [KUTIL] Add kudu_util library to the build. .. Patch Set 25: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/896/ -- To

[Impala-ASF-CR] IMPALA-5625: write profile when query times out

2017-07-20 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-5625: write profile when query times out .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7376/2/tests/stress/concurrent_select.py File te

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7379/1/fe/src/main/java/

[Impala-ASF-CR] IMPALA-5676: avoid expensive consistency checks in BTSv2

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5676: avoid expensive consistency checks in BTSv2 .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/895/ -- To v

[Impala-ASF-CR] IMPALA-5676: avoid expensive consistency checks in BTSv2

2017-07-20 Thread Tim Armstrong (Code Review)
Hello Michael Ho, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7459 to look at the new patch set (#4). Change subject: IMPALA-5676: avoid expensive consistency checks in BTSv2 .. IMPALA

[Impala-ASF-CR] IMPALA-5676: avoid expensive consistency checks in BTSv2

2017-07-20 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5676: avoid expensive consistency checks in BTSv2 .. Patch Set 4: Code-Review+2 (2 comments) Carry +2 http://gerrit.cloudera.org:8080/#/c/7459/3/be/src/

[Impala-ASF-CR] IMPALA-5529: Add additional function signatures for TRUNC()

2017-07-20 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5529: Add additional function signatures for TRUNC() .. Patch Set 2: (1 comment) Thanks for doing this! Looks pretty good to me. Please make the 1 small

[Impala-ASF-CR] IMPALA-5546: Allow creating unpartitioned Kudu tables

2017-07-20 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables .. Patch Set 1: Nice! I like the warning. How about also a ctas test case? I think it'd also be useful to

[Impala-ASF-CR] IMPALA-5615: Fix compute incremental stats for general partition exprs

2017-07-20 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5615: Fix compute incremental stats for general partition exprs .. Patch Set 1: Any reviewers for this? A good (and simple) one to start with if s

[Impala-ASF-CR] IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-verify-dryr

[Impala-ASF-CR] IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS

2017-07-20 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS .. Patch Set 4: Code-Review+2 Thanks. Rebased. -- To view, visit http://gerrit

[Impala-ASF-CR] IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS

2017-07-20 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS .. Patch Set 3: Code-Review+2 Thanks for reminding me. Looks good. -- To view, visit

[Impala-ASF-CR] IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS

2017-07-20 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5657: Fix a couple of bugs with FunctionCallExpr and IGNORE NULLS .. Patch Set 3: Matt, is this good to go or do you have any further comments? -

[Impala-ASF-CR] IMPALA-4868: Disable flaky TestRequestPoolService test

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4868: Disable flaky TestRequestPoolService test .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7466 To unsubscribe,

[Impala-ASF-CR] IMPALA-4868: Disable flaky TestRequestPoolService test

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-4868: Disable flaky TestRequestPoolService test .. IMPALA-4868: Disable flaky TestRequestPoolService test Disables a test that seemed to

[Impala-ASF-CR] IMPALA-5529: Add additional function signatures for TRUNC()

2017-07-20 Thread Kim Jin Chul (Code Review)
Kim Jin Chul has uploaded a new patch set (#2). Change subject: IMPALA-5529: Add additional function signatures for TRUNC() .. IMPALA-5529: Add additional function signatures for TRUNC() The following signatures to be added: +--

[Impala-ASF-CR] IMPALA-5638: Fix Kudu table set tblproperties inconsistencies

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies .. IMPALA-5638: Fix Kudu table set tblproperties inconsistencies Kudu tables did n

[Impala-ASF-CR] IMPALA-5638: Fix Kudu table set tblproperties inconsistencies

2017-07-20 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5638: Fix Kudu table set tblproperties inconsistencies .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7454 To unsub

[Impala-ASF-CR] IMPALA-5676: avoid expensive consistency checks in BTSv2

2017-07-20 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5676: avoid expensive consistency checks in BTSv2 .. Patch Set 3: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/7459/3/be/src/runtime/buffe