[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5382 To unsubscribe, vi

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE .. Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE This commit also removes the now

[Impala-ASF-CR] IMPALA-3126: Conservative assignment of inner-join On-clause predicates.

2016-12-06 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-3126: Conservative assignment of inner-join On-clause predicates. .. Patch Set 3: Code-Review+2 (4 comments) http://gerrit.cloudera.org:8080/#/c/4982

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. Patch Set 5: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5386 To unsubsc

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. IMPALA-4592: Improve error msg for non-deterministic predicates. Impala cannot co

[Impala-ASF-CR] IMPALA-4586: don't constant fold in backend

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4586: don't constant fold in backend .. Patch Set 3: (10 comments) http://gerrit.cloudera.org:8080/#/c/5391/3//COMMIT_MSG Commit Message: Line 15: constant

[Impala-ASF-CR] IMPALA-4613: Make sure timers are finished before sending report profile

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4613: Make sure timers are finished before sending report profile .. Patch Set 2: Verified-1 Build failed: http://sandbox.jenkins.cloudera.com/job/im

[Impala-ASF-CR] IMPALA-4613: Make sure timers are finished before sending report profile

2016-12-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-4613: Make sure timers are finished before sending report profile .. Patch Set 2: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-

[Impala-ASF-CR] IMPALA-4477: Bump Kudu version to latest master (60aa54e)

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4477: Bump Kudu version to latest master (60aa54e) .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5369 To unsubscribe, v

[Impala-ASF-CR] IMPALA-4477: Bump Kudu version to latest master (60aa54e)

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4477: Bump Kudu version to latest master (60aa54e) .. IMPALA-4477: Bump Kudu version to latest master (60aa54e) Bumps the toolchain version to

[Impala-ASF-CR] IMPALA-3788: Add flag for Kudu read-your-writes

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-3788: Add flag for Kudu read-your-writes .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5288 To unsubscribe, visit http:

[Impala-ASF-CR] IMPALA-3788: Add flag for Kudu read-your-writes

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3788: Add flag for Kudu read-your-writes .. IMPALA-3788: Add flag for Kudu read-your-writes The previous attempt to support for Kudu 'read-you

[Impala-ASF-CR] IMPALA-4613: Make sure timers are finished before sending report profile

2016-12-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4613: Make sure timers are finished before sending report profile .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5394 T

[Impala-ASF-CR] IMPALA-2864: Ensure that client connections are closed after a failed Open()

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-2864: Ensure that client connections are closed after a failed Open() .. IMPALA-2864: Ensure that client connections are closed after a failed

[Impala-ASF-CR] IMPALA-2864: Ensure that client connections are closed after a failed Open()

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-2864: Ensure that client connections are closed after a failed Open() .. Patch Set 3: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/53

[Impala-ASF-CR] IMPALA-4613: Make sure timers are finished before sending report profile

2016-12-06 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new patch set (#2). Change subject: IMPALA-4613: Make sure timers are finished before sending report profile .. IMPALA-4613: Make sure timers are finished before sending report profile FragmentComp

[Impala-ASF-CR] IMPALA-4613: Make sure timers are finished before sending report profile

2016-12-06 Thread Henry Robinson (Code Review)
Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/5394 Change subject: IMPALA-4613: Make sure timers are finished before sending report profile .. IMPALA-4613: Make sure timers are finished

[Impala-ASF-CR] IMPALA-4566: Set Kudu vlog level independently of Impala vlog

2016-12-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/5393 Change subject: IMPALA-4566: Set Kudu vlog level independently of Impala vlog .. IMPALA-4566: Set Kudu vlog level independently of Impal

[Impala-ASF-CR] IMPALA-4566: Kudu client glog contention can cause timeouts

2016-12-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4566: Kudu client glog contention can cause timeouts .. Patch Set 2: FYI I've sent Todd an e-mail (he's in Singapore) asking why he thinks this isn't wo

[Impala-ASF-CR] IMPALA-2057: Better error message for incorrect avro decimal column declaration

2016-12-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-2057: Better error message for incorrect avro decimal column declaration .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-12-06 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has uploaded a new patch set (#8). Change subject: IMPALA-4014: Introduce query-wide execution state. .. IMPALA-4014: Introduce query-wide execution state. This introduces a global structure to coordinate execut

[Impala-ASF-CR] IMPALA-4595: Ignore discarded functions after linking

2016-12-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4595: Ignore discarded functions after linking .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5384/1/be/src/codegen/llvm-codegen.cc File

[Impala-ASF-CR] IMPALA-4609: prefix thread counters in fragment profile

2016-12-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/5392 Change subject: IMPALA-4609: prefix thread counters in fragment profile .. IMPALA-4609: prefix thread counters in fragment profile This

[Impala-ASF-CR] IMPALA-4014: Introduce query-wide execution state.

2016-12-06 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4014: Introduce query-wide execution state. .. Patch Set 7: (15 comments) http://gerrit.cloudera.org:8080/#/c/4418/7/be/src/runtime/coordinator.cc Fil

[Impala-ASF-CR] IMPALA-3314: Fix Avro schema loading for partitioned tables.

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3314: Fix Avro schema loading for partitioned tables. .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5388 To unsubscribe, vi

[Impala-ASF-CR] IMPALA-4477: Bump Kudu version to latest master (60aa54e)

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4477: Bump Kudu version to latest master (60aa54e) .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5369 To unsubscribe, v

[Impala-ASF-CR] IMPALA-4586: don't constant fold in backend

2016-12-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4586: don't constant fold in backend .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5391/3//COMMIT_MSG Commit Message: Line 14: offers

[Impala-ASF-CR] IMPALA-4586: don't constant fold in backend

2016-12-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new change for review. http://gerrit.cloudera.org:8080/5391 Change subject: IMPALA-4586: don't constant fold in backend .. IMPALA-4586: don't constant fold in backend Before this patch, the backen

[Impala-ASF-CR] IMPALA-4403: Implement SHOW RANGE PARTITIONS for Kudu tables

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4403: Implement SHOW RANGE PARTITIONS for Kudu tables .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/5390/1/common/thrift/Frontend.thrift F

[Impala-ASF-CR] IMPALA-4467: Add support for DML statements in stress test

2016-12-06 Thread Michael Brown (Code Review)
Michael Brown has posted comments on this change. Change subject: IMPALA-4467: Add support for DML statements in stress test .. Patch Set 2: (26 comments) http://gerrit.cloudera.org:8080/#/c/5093/2//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-4594: WriteSlot and CodegenWriteSlot handle escaped NULL slots differently

2016-12-06 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4594: WriteSlot and CodegenWriteSlot handle escaped NULL slots differently .. Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/5377/2//COMM

[Impala-ASF-CR] IMPALA-4594: WriteSlot and CodegenWriteSlot handle escaped NULL slots differently

2016-12-06 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has uploaded a new patch set (#3). Change subject: IMPALA-4594: WriteSlot and CodegenWriteSlot handle escaped NULL slots differently .. IMPALA-4594: WriteSlot and CodegenWriteSlot handle escaped NULL slots differe

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE .. Patch Set 3: I submitted this for a gvo: http://sandbox.jenkins.cloudera.com/job/impala-external-ge

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE .. Patch Set 3: Code-Review+2 carrying Dimitris' +2 -- To view, visit http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-4585: Allow the $DATABASE template in the CATCH section

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4585: Allow the $DATABASE template in the CATCH section .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5372 To unsubscribe,

[Impala-ASF-CR] IMPALA-4585: Allow the $DATABASE template in the CATCH section

2016-12-06 Thread Taras Bobrovytsky (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5372 to look at the new patch set (#3). Change subject: IMPALA-4585: Allow the $DATABASE template in the CATCH section .. I

[Impala-ASF-CR] IMPALA-4403: Implement SHOW RANGE PARTITIONS for Kudu tables

2016-12-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4403: Implement SHOW RANGE PARTITIONS for Kudu tables .. Patch Set 1: Thanks, Lars! So firstly, this functionality I think is pretty important so worth

[Impala-ASF-CR] IMPALA-4585: Replace HDFS FILENAME with actual paths

2016-12-06 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-4585: Replace __HDFS_FILENAME__ with actual paths .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5372/2//COMMIT_MSG Commit Message:

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Dan Burkert (Code Review)
Hello Dimitris Tsirogiannis, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5382 to look at the new patch set (#3). Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE ...

[Impala-ASF-CR] IMPALA-4585: Replace HDFS FILENAME with actual paths

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4585: Replace __HDFS_FILENAME__ with actual paths .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/5372/2//COMMIT_MSG Commit Mes

[Impala-ASF-CR] IMPALA-4585: Replace HDFS FILENAME with actual paths

2016-12-06 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#2). Change subject: IMPALA-4585: Replace __HDFS_FILENAME__ with actual paths .. IMPALA-4585: Replace __HDFS_FILENAME__ with actual paths In a recent change (IMPALA-4363) we introd

[Impala-ASF-CR] IMPALA-4585: Replace HDFS FILENAME with actual paths

2016-12-06 Thread Taras Bobrovytsky (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5372 to look at the new patch set (#2). Change subject: IMPALA-4585: Replace __HDFS_FILENAME__ with actual paths .. IMPALA-

[Impala-ASF-CR] IMPALA-4585: Replace HDFS FILENAME actual paths

2016-12-06 Thread Taras Bobrovytsky (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5372 to look at the new patch set (#2). Change subject: IMPALA-4585: Replace __HDFS_FILENAME__ actual paths .. IMPALA-4585:

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5386 To unsubscribe

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5386/4/fe/src/main/java/org/apache/

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Alex Behm (Code Review)
Hello Dimitris Tsirogiannis, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5386 to look at the new patch set (#5). Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. ...

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. Patch Set 4: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/5386/4/fe

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new patch set (#4). Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. IMPALA-4592: Improve error msg for non-deterministic predicates. Impala cannot correctly evaluate o

[Impala-ASF-CR] IMPALA-4403: Implement SHOW RANGE PARTITIONS for Kudu tables

2016-12-06 Thread Lars Volker (Code Review)
Lars Volker has uploaded a new change for review. http://gerrit.cloudera.org:8080/5390 Change subject: IMPALA-4403: Implement SHOW RANGE PARTITIONS for Kudu tables .. IMPALA-4403: Implement SHOW RANGE PARTITIONS for Kudu table

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5386/3/fe/src/main/java/org/apache/

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5386/3/fe/src/main/java/org/apache/

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5386/3/fe/src/main/java

[Impala-ASF-CR] IMPALA-3524: Don't process spilled partitions with 0 probe rows

2016-12-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-3524: Don't process spilled partitions with 0 probe rows .. IMPALA-3524: Don't process spilled partitions with 0 probe rows In the partitioned hash

[Impala-ASF-CR] IMPALA-3524: Don't process spilled partitions with 0 probe rows

2016-12-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/5389 Change subject: IMPALA-3524: Don't process spilled partitions with 0 probe rows .. IMPALA-3524: Don't process spilled partitions

[Impala-ASF-CR] IMPALA-4595: Ignore discarded functions after linking

2016-12-06 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4595: Ignore discarded functions after linking .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5384/1/be/src/codegen/llvm-codegen.cc File be

[Impala-ASF-CR] IMPALA-4595: Ignore discarded functions after linking

2016-12-06 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4595: Ignore discarded functions after linking .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5384/1/be/src/codegen/llvm-codegen.cc File be

[Impala-ASF-CR] IMPALA-4594: WriteSlot and CodegenWriteSlot handle escaped NULL slots differently

2016-12-06 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4594: WriteSlot and CodegenWriteSlot handle escaped NULL slots differently .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/5377/2//COMMIT_

[Impala-ASF-CR] IMPALA-4595: Ignore discarded functions after linking

2016-12-06 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4595: Ignore discarded functions after linking .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5384/1/be/src/codegen/llvm-codegen.cc File be/

[Impala-ASF-CR] IMPALA-3314: Fix Avro schema loading for partitioned tables.

2016-12-06 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has uploaded a new change for review. http://gerrit.cloudera.org:8080/5388 Change subject: IMPALA-3314: Fix Avro schema loading for partitioned tables. .. IMPALA-3314: Fix Avro schema loading for partition

[Impala-ASF-CR] IMPALA-3788: Add flag for Kudu read-your-writes

2016-12-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3788: Add flag for Kudu read-your-writes .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5288 To unsubscribe, visit http

[Impala-ASF-CR] IMPALA-3788: Add flag for Kudu read-your-writes

2016-12-06 Thread Matthew Jacobs (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5288 to look at the new patch set (#4). Change subject: IMPALA-3788: Add flag for Kudu read-your-writes .. IMPALA-3788: Add

[Impala-ASF-CR] IMPALA-3788: Add flag for Kudu read-your-writes

2016-12-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-3788: Add flag for Kudu read-your-writes .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5288/3/be/src/exec/kudu-scanner.cc File be/src/

[Impala-ASF-CR] IMPALA-3788: Add flag for Kudu read-your-writes

2016-12-06 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3788: Add flag for Kudu read-your-writes .. Patch Set 3: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/5288/3/be/src/exec/kudu-scanner.cc Fil

[Impala-ASF-CR] IMPALA-4477: Bump Kudu version to latest master (60aa54e)

2016-12-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4477: Bump Kudu version to latest master (60aa54e) .. Patch Set 2: Code-Review+2 rebase, minor conflict in test -- To view, visit http://gerrit.clouder

[Impala-ASF-CR] IMPALA-4477: Bump Kudu version to latest master (60aa54e)

2016-12-06 Thread Matthew Jacobs (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5369 to look at the new patch set (#2). Change subject: IMPALA-4477: Bump Kudu version to latest master (60aa54e) .. IMPALA

[Impala-ASF-CR] IMPALA-4594: WriteSlot and CodegenWriteSlot handle escaped NULL slots differently

2016-12-06 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-4594: WriteSlot and CodegenWriteSlot handle escaped NULL slots differently .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/5377/2/be/src/e

[Impala-ASF-CR] Add Apache license header to files in doc directory

2016-12-06 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: Add Apache license header to files in doc directory .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5232/2//COMMIT_MSG Commit Message: PS2, Line 7:

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new patch set (#3). Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. IMPALA-4592: Improve error msg for non-deterministic predicates. Impala cannot correctly evaluate o

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5386/1//COMMIT_MSG Commit Message:

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread David Knupp (Code Review)
David Knupp has posted comments on this change. Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5386/1//COMMIT_MSG Commit Message

[Impala-ASF-CR] IMPALA-4355: random query generator: modify statement execution flow to support DML

2016-12-06 Thread Michael Brown (Code Review)
Michael Brown has uploaded a new change for review. http://gerrit.cloudera.org:8080/5387 Change subject: IMPALA-4355: random query generator: modify statement execution flow to support DML .. IMPALA-4355: random query generat

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new patch set (#2). Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. IMPALA-4592: Improve error msg for non-deterministic predicates. Impala cannot correctly evaluate o

[Impala-ASF-CR] IMPALA-4592: Improve error msg for non-deterministic predicates.

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new change for review. http://gerrit.cloudera.org:8080/5386 Change subject: IMPALA-4592: Improve error msg for non-deterministic predicates. .. IMPALA-4592: Improve error msg for non-deterministic pred

[Impala-ASF-CR] Add Apache license header to files in doc directory

2016-12-06 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: Add Apache license header to files in doc directory .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5232/2//COMMIT_MSG Commit Message: PS2, Line

[Impala-ASF-CR] IMPALA-3398: Rework Impala documentation to be non-Cloudera-specific

2016-12-06 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3398: Rework Impala documentation to be non-Cloudera-specific .. Patch Set 5: I have substantially reduced the documentation changes after our out-of-band d

[Impala-ASF-CR] IMPALA-3398: Rework Impala documentation to be non-Cloudera-specific

2016-12-06 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#5). Change subject: IMPALA-3398: Rework Impala documentation to be non-Cloudera-specific .. IMPALA-3398: Rework Impala documentation to be non-Cloudera-specific Removes some of the Cloud

[Impala-ASF-CR] IMPALA-2864: Ensure that client connections are closed after a failed Open()

2016-12-06 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-2864: Ensure that client connections are closed after a failed Open() .. Patch Set 3: Code-Review+2 (2 comments) Rebase. Carry +2. http://gerrit.cloude

[Impala-ASF-CR] IMPALA-2864: Ensure that client connections are closed after a failed Open()

2016-12-06 Thread Sailesh Mukil (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5385 to look at the new patch set (#3). Change subject: IMPALA-2864: Ensure that client connections are closed after a failed Open() ..

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE .. Patch Set 2: FYI I submitted a private jenkins test run for this change: http://sandbox.jenkins.clo

[Impala-ASF-CR] IMPALA-4574: Do not treat UUID() like a constant expr.

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has posted comments on this change. Change subject: IMPALA-4574: Do not treat UUID() like a constant expr. .. Patch Set 6: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5324 To unsubscribe, visi

[Impala-ASF-CR] IMPALA-4574: Do not treat UUID() like a constant expr.

2016-12-06 Thread Internal Jenkins (Code Review)
Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-4574: Do not treat UUID() like a constant expr. .. IMPALA-4574: Do not treat UUID() like a constant expr. A recent change (IMPALA-1788) lead U

[Impala-ASF-CR] IMPALA-3126: Conservative assignment of inner-join On-clause predicates.

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has uploaded a new patch set (#3). Change subject: IMPALA-3126: Conservative assignment of inner-join On-clause predicates. .. IMPALA-3126: Conservative assignment of inner-join On-clause predicates. Implements the fo

[Impala-ASF-CR] IMPALA-3126: Conservative assignment of inner-join On-clause predicates.

2016-12-06 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3126: Conservative assignment of inner-join On-clause predicates. .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/4982/2/testdata/workloads

[Impala-ASF-CR] IMPALA-4477: Bump Kudu version to latest master (60aa54e)

2016-12-06 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4477: Bump Kudu version to latest master (60aa54e) .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5369/1/testdata/workloads/functional-query

[Impala-ASF-CR] IMPALA-4477: Bump Kudu version to latest master (60aa54e)

2016-12-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: IMPALA-4477: Bump Kudu version to latest master (60aa54e) .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/5369/1/testdata/workloads/functional-

[Impala-ASF-CR] IMPALA-4595: Ignore discarded functions after linking

2016-12-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-4595: Ignore discarded functions after linking .. Patch Set 1: (1 comment) Code change looks good http://gerrit.cloudera.org:8080/#/c/5384/1/be/src/test

[Impala-ASF-CR] IMPALA-2864: Ensure that client connections are closed after a failed Open()

2016-12-06 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-2864: Ensure that client connections are closed after a failed Open() .. Patch Set 2: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/5385/2/

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5382 To unsubsc

[Impala-ASF-CR] IMPALA-3398: Rework Impala documentation to be non-Cloudera-specific

2016-12-06 Thread Jim Apple (Code Review)
Jim Apple has posted comments on this change. Change subject: IMPALA-3398: Rework Impala documentation to be non-Cloudera-specific .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5239/3/docs/impala_html.ditava

[Impala-ASF-CR] IMPALA-3398: Rework Impala documentation to be non-Cloudera-specific

2016-12-06 Thread Jim Apple (Code Review)
Jim Apple has uploaded a new patch set (#4). Change subject: IMPALA-3398: Rework Impala documentation to be non-Cloudera-specific .. IMPALA-3398: Rework Impala documentation to be non-Cloudera-specific This fix is a prototype f

[Impala-ASF-CR] IMPALA-3398: Rework Impala documentation to be non-Cloudera-specific

2016-12-06 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: IMPALA-3398: Rework Impala documentation to be non-Cloudera-specific .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5239/3/docs/impala_html.dit

[Impala-ASF-CR] IMPALA-2864: Ensure that client connections are closed after a failed Open()

2016-12-06 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has uploaded a new patch set (#2). Change subject: IMPALA-2864: Ensure that client connections are closed after a failed Open() .. IMPALA-2864: Ensure that client connections are closed after a failed Open() When

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE .. Patch Set 2: PS2 build: http://sandbox.jenkins.cloudera.com/job/impala-umbrella-build-and-test/6174/co

[Impala-ASF-CR] IMPALA-2864: Ensure that client connections are closed after a failed Open()

2016-12-06 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has uploaded a new change for review. http://gerrit.cloudera.org:8080/5385 Change subject: IMPALA-2864: Ensure that client connections are closed after a failed Open() .. IMPALA-2864: Ensure that client connecti

[Impala-ASF-CR] IMPALA-3788: Add flag for Kudu read-your-writes

2016-12-06 Thread Matthew Jacobs (Code Review)
Hello Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5288 to look at the new patch set (#3). Change subject: IMPALA-3788: Add flag for Kudu read-your-writes .. IMPALA-3788: Add

[Impala-ASF-CR] IMPALA-4595: Ignore discarded functions after linking

2016-12-06 Thread Michael Ho (Code Review)
Michael Ho has uploaded a new change for review. http://gerrit.cloudera.org:8080/5384 Change subject: IMPALA-4595: Ignore discarded functions after linking .. IMPALA-4595: Ignore discarded functions after linking For LLVM IR

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/5382/1/docs/topics/impala_create_table.x

[Impala-ASF-CR] Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE

2016-12-06 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new patch set (#2). Change subject: Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE .. Replace INTO N BUCKETS with PARTITIONS N in CREATE TABLE This commit also removes the now unused `DISTRIB

[Impala-ASF-CR] IMPALA-4477: Bump Kudu version to latest master (60aa54e)

2016-12-06 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4477: Bump Kudu version to latest master (60aa54e) .. Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/5369/1/testdata/workloads/fu

[Impala-ASF-CR] IMPALA-3788: Add flag for Kudu read-your-writes

2016-12-06 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-3788: Add flag for Kudu read-your-writes .. Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/5288/2/be/src/exec/kudu-scanner.cc Fil

  1   2   >