[Impala-ASF-CR] IMPALA-4270: Gracefully fail unsupported queries with mt dop > 0.

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4270: Gracefully fail unsupported queries with mt_dop > 0. .. Patch Set 5: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-4294: Make check-schema-diff.sh executable from anywhere.

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4294: Make check-schema-diff.sh executable from anywhere. .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4726 To

[Impala-ASF-CR] IMPALA-4294: Make check-schema-diff.sh executable from anywhere.

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4294: Make check-schema-diff.sh executable from anywhere. .. IMPALA-4294: Make check-schema-diff.sh executable from anywhere. Fixes a

[Impala-ASF-CR] IMPALA-3348: Avoid per-slot check vector size in KuduScanner

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3348: Avoid per-slot check vector size in KuduScanner .. IMPALA-3348: Avoid per-slot check vector size in KuduScanner Fixes a small perf

[Impala-ASF-CR] IMPALA-3348: Avoid per-slot check vector size in KuduScanner

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3348: Avoid per-slot check vector size in KuduScanner .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4688 To

[Impala-ASF-CR] IMPALA-3002/IMPALA-1473: Cardinality observability cleanup

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3002/IMPALA-1473: Cardinality observability cleanup .. IMPALA-3002/IMPALA-1473: Cardinality observability cleanup IMPALA-3002: The shell

[Impala-ASF-CR] IMPALA-3002/IMPALA-1473: Cardinality observability cleanup

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3002/IMPALA-1473: Cardinality observability cleanup .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4679 To unsubscribe,

[Impala-ASF-CR] IMPALA-4102: Remote Kudu reads should be reported

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4102: Remote Kudu reads should be reported .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4687 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4102: Remote Kudu reads should be reported

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4102: Remote Kudu reads should be reported .. IMPALA-4102: Remote Kudu reads should be reported Adds a profile counter for the number of kudu

[Impala-ASF-CR] IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables

2016-10-14 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables .. Patch Set 5: (41 comments) Sending out the first wave.

[Impala-ASF-CR] IMPALA-2905: Handle coordinator fragment lifecycle like all others

2016-10-14 Thread Henry Robinson (Code Review)
Hello Marcel Kornacker, Internal Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4402 to look at the new patch set (#19). Change subject: IMPALA-2905: Handle coordinator fragment lifecycle like all others

[Impala-ASF-CR] IMPALA-4294: Make check-schema-diff.sh executable from anywhere.

2016-10-14 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4294: Make check-schema-diff.sh executable from anywhere. .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4726 To

[Impala-ASF-CR] IMPALA-4270: Gracefully fail unsupported queries with mt dop > 0.

2016-10-14 Thread Alex Behm (Code Review)
Hello Marcel Kornacker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4677 to look at the new patch set (#5). Change subject: IMPALA-4270: Gracefully fail unsupported queries with mt_dop > 0.

[Impala-ASF-CR] IMPALA-4270: Gracefully fail unsupported queries with mt dop > 0.

2016-10-14 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4270: Gracefully fail unsupported queries with mt_dop > 0. .. Patch Set 4: (3 comments)

[Impala-ASF-CR] IMPALA-4294: Make check-schema-diff.sh executable from anywhere.

2016-10-14 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. Change subject: IMPALA-4294: Make check-schema-diff.sh executable from anywhere. .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4726/2/buildall.sh File

[Impala-ASF-CR] IMPALA-4294: Make check-schema-diff.sh executable from anywhere.

2016-10-14 Thread David Knupp (Code Review)
David Knupp has uploaded a new patch set (#3). Change subject: IMPALA-4294: Make check-schema-diff.sh executable from anywhere. .. IMPALA-4294: Make check-schema-diff.sh executable from anywhere. Fixes a regression in the data

[Impala-ASF-CR] IMPALA-2905: Handle coordinator fragment lifecycle like all others

2016-10-14 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: IMPALA-2905: Handle coordinator fragment lifecycle like all others .. Patch Set 18: I think this change has exposed a race in error log reporting that's very

[Impala-ASF-CR] IMPALA-4134: Use AUTO FLUSH BACKGROUND for Kudu sink

2016-10-14 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4134: Use AUTO_FLUSH_BACKGROUND for Kudu sink .. Patch Set 1: Hold off on reviewing this. I'm going to make more changes. -- To view, visit

[Impala-ASF-CR] IMPALA-3644 Make predicate order deterministic

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3644 Make predicate order deterministic .. IMPALA-3644 Make predicate order deterministic This adds a tie-break to make sure that we sort

[Impala-ASF-CR] IMPALA-4294: Execute data load steps from within git repo.

2016-10-14 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4294: Execute data load steps from within git repo. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4726/2/buildall.sh File

[Impala-ASF-CR] IMPALA-4294: Execute data load steps from within git repo.

2016-10-14 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. Change subject: IMPALA-4294: Execute data load steps from within git repo. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4726/2/buildall.sh File buildall.sh:

[Impala-ASF-CR] IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables

2016-10-14 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has uploaded a new patch set (#6). Change subject: IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables .. IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables With this commit we

[Impala-ASF-CR] IMPALA-4134: Use AUTO FLUSH BACKGROUND for Kudu sink

2016-10-14 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4134: Use AUTO_FLUSH_BACKGROUND for Kudu sink .. Patch Set 3: (3 comments) Thanks. I made a bunch of changes and ended up making it into a new commit.

[Impala-ASF-CR] IMPALA-2905: Handle coordinator fragment lifecycle like all others

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-2905: Handle coordinator fragment lifecycle like all others .. Patch Set 18: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-3348: Avoid per-slot check vector size in KuduScanner

2016-10-14 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3348: Avoid per-slot check vector size in KuduScanner .. Patch Set 2: Code-Review+2 rebase and carrying +2 -- To view, visit

[Impala-ASF-CR] IMPALA-4294: Execute data load steps from within git repo.

2016-10-14 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. Change subject: IMPALA-4294: Execute data load steps from within git repo. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4726/2/buildall.sh File buildall.sh:

[Impala-ASF-CR] IMPALA-4294: Execute data load steps from within git repo.

2016-10-14 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4294: Execute data load steps from within git repo. .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4726/2/buildall.sh File

[Impala-ASF-CR] IMPALA-3348: Avoid per-slot check vector size in KuduScanner

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3348: Avoid per-slot check vector size in KuduScanner .. Patch Set 1: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-4270: Gracefully fail unsupported queries with mt dop > 0.

2016-10-14 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4270: Gracefully fail unsupported queries with mt_dop > 0. .. Patch Set 4: Code-Review+2 (3 comments)

[Impala-ASF-CR] IMPALA-3348: Avoid per-slot check vector size in KuduScanner

2016-10-14 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3348: Avoid per-slot check vector size in KuduScanner .. Patch Set 1: Code-Review+2 Trivial change, I feel comfortable giving +2 -- To view, visit

[Impala-ASF-CR] IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables

2016-10-14 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3719: Simplify CREATE TABLE statements with Kudu tables .. Patch Set 5: (1 comment)

[Impala-ASF-CR] IMPALA-3739: Enable stress tests on Kudu

2016-10-14 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-3739: Enable stress tests on Kudu .. Patch Set 6: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/4327 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-3002/IMPALA-1473: Cardinality observability cleanup

2016-10-14 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3002/IMPALA-1473: Cardinality observability cleanup .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4679 To

[Impala-ASF-CR] IMPALA-3002/IMPALA-1473: Cardinality observability cleanup

2016-10-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3002/IMPALA-1473: Cardinality observability cleanup .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/4679/2//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-3002/IMPALA-1473: Cardinality observability cleanup

2016-10-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-3002/IMPALA-1473: Cardinality observability cleanup .. IMPALA-3002/IMPALA-1473: Cardinality observability cleanup IMPALA-3002: The shell prints an

[Impala-ASF-CR] IMPALA-4123: Fast bit unpacking

2016-10-14 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-4123: Fast bit unpacking .. Patch Set 11: (5 comments) http://gerrit.cloudera.org:8080/#/c/4494/11/be/src/util/bit-packing-test.cc File

[Impala-ASF-CR] IMPALA-3644 Make predicate order deterministic

2016-10-14 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3644 Make predicate order deterministic .. Patch Set 5: Code-Review+2 Carry Alex's +2 -- To view, visit http://gerrit.cloudera.org:8080/4671 To

[Impala-ASF-CR] IMPALA-3644 Make predicate order deterministic

2016-10-14 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3644 Make predicate order deterministic .. Patch Set 5: Code-Review+2 carrying alex's +2 -- To view, visit http://gerrit.cloudera.org:8080/4671 To

[Impala-ASF-CR] IMPALA-2905: Handle coordinator fragment lifecycle like all others

2016-10-14 Thread Henry Robinson (Code Review)
Hello Marcel Kornacker, Internal Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4402 to look at the new patch set (#18). Change subject: IMPALA-2905: Handle coordinator fragment lifecycle like all others

[Impala-ASF-CR] IMPALA-2905: Handle coordinator fragment lifecycle like all others

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-2905: Handle coordinator fragment lifecycle like all others .. Patch Set 17: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-3644 Make predicate order deterministic

2016-10-14 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: IMPALA-3644 Make predicate order deterministic .. Patch Set 4: (1 comment) Thanks for the review. Can you +2 again?

[Impala-ASF-CR] IMPALA-3644 Make predicate order deterministic

2016-10-14 Thread Lars Volker (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4671 to look at the new patch set (#5). Change subject: IMPALA-3644 Make predicate order deterministic .. IMPALA-3644 Make

[Impala-ASF-CR] IMPALA-4188: Leopard: support external Docker volumes

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4188: Leopard: support external Docker volumes .. Patch Set 8: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4678 To unsubscribe,

[Impala-ASF-CR] IMPALA-2905: Handle coordinator fragment lifecycle like all others

2016-10-14 Thread Henry Robinson (Code Review)
Hello Marcel Kornacker, Internal Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4402 to look at the new patch set (#17). Change subject: IMPALA-2905: Handle coordinator fragment lifecycle like all others

[Impala-ASF-CR] IMPALA-3971, IMPALA-3229: Bootstrap an Impala dev environment

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3971, IMPALA-3229: Bootstrap an Impala dev environment .. IMPALA-3971, IMPALA-3229: Bootstrap an Impala dev environment This script

[Impala-ASF-CR] IMPALA-3971, IMPALA-3229: Bootstrap an Impala dev environment

2016-10-14 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3971, IMPALA-3229: Bootstrap an Impala dev environment .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/4674 To