[native-toolchain-CR] Bump Kudu version to a71ecfd

2017-09-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to a71ecfd .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7975 To unsubscribe, visit http

[Impala-ASF-CR] IMPALA-5871: KuduPartitionExpr incorrectly handles its child types

2017-08-31 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5871: KuduPartitionExpr incorrectly handles its child types .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/7922/1

[Impala-ASF-CR] IMPALA-5871: KuduPartitionExpr incorrectly handles its child types

2017-08-31 Thread Thomas Tauber-Marshall (Code Review)
a.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I44cf31e46a77f3e7c92cf6b9112653808a001705 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Matthew Jacobs

[Impala-ASF-CR] IMPALA-5871: KuduPartitionExpr incorrectly handles its child types

2017-08-31 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7922 Change subject: IMPALA-5871: KuduPartitionExpr incorrectly handles its child types .. IMPALA-5871: KuduPartitionExpr

[Impala-ASF-CR] Bump Kudu version to 1c70e5d

2017-08-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7903 Change subject: Bump Kudu version to 1c70e5d .. Bump Kudu version to 1c70e5d Change-Id

[Impala-ASF-CR] IMPALA-5863: Include-what-you-use for Kudu client

2017-08-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5863: Include-what-you-use for Kudu client .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7872/2//COMMIT_MSG Commit Message

[Impala-ASF-CR] IMPALA-5863: Include-what-you-use for Kudu client

2017-08-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-5863: Include-what-you-use for Kudu client .. IMPALA-5863: Include-what-you-use for Kudu client A recent commit in Kudu removed some unnecessary

[Impala-ASF-CR] Include-what-you-use for Kudu client

2017-08-29 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: Include-what-you-use for Kudu client .. Include-what-you-use for Kudu client A recent commit in Kudu removed some unnecessary includes from Kudu client

[Impala-ASF-CR] Include-what-you-use for Kudu client

2017-08-29 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Include-what-you-use for Kudu client .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7872/1/be/src/exprs/kudu-partition-expr.h File be

[Impala-ASF-CR] Bump Kudu version to 1c70e5d

2017-08-29 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7872 Change subject: Bump Kudu version to 1c70e5d .. Bump Kudu version to 1c70e5d This required adding a few extra includes due to

[native-toolchain-CR] Bump Kudu version to 1c70e5d

2017-08-28 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has submitted this change and it was merged. Change subject: Bump Kudu version to 1c70e5d .. Bump Kudu version to 1c70e5d Change-Id: Iba57fbe903a2aea0e34851f1a0611691a9bb319e --- M buildall.sh 1 file

[native-toolchain-CR] Bump Kudu version to 1c70e5d

2017-08-28 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 1c70e5d .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7855 To unsubscribe, visit http

[native-toolchain-CR] Bump Kudu version to 1c70e5d

2017-08-28 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 1c70e5d .. Patch Set 2: > > (1 comment) > > Ah, that's on Cloudera's github mirror. Let's choose th

[native-toolchain-CR] Bump Kudu version to 1c70e5d

2017-08-28 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: Bump Kudu version to 1c70e5d .. Bump Kudu version to 1c70e5d Change-Id: Iba57fbe903a2aea0e34851f1a0611691a9bb319e --- M buildall.sh 1 file changed, 1

[native-toolchain-CR] Bump Kudu version to 1c70e5d

2017-08-28 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 1c70e5d .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7855/1//COMMIT_MSG Commit Message: PS1, Line 7: 1c70e5d

[native-toolchain-CR] Bump Kudu version to 22a19d9

2017-08-28 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7855 Change subject: Bump Kudu version to 22a19d9 .. Bump Kudu version to 22a19d9 Change-Id

[Impala-ASF-CR] IMPALA-5799: Kudu DML can crash if schema has changed

2017-08-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5799: Kudu DML can crash if schema has changed .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/7688/3/be/src/exec/kudu-table

[Impala-ASF-CR] IMPALA-5799: Kudu DML can crash if schema has changed

2017-08-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#4). Change subject: IMPALA-5799: Kudu DML can crash if schema has changed .. IMPALA-5799: Kudu DML can crash if schema has changed We check that the number/types of columns

[Impala-ASF-CR] IMPALA-5799: Kudu DML can crash if schema has changed

2017-08-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5799: Kudu DML can crash if schema has changed .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7688/3/be/src/exec/kudu-table

[Impala-ASF-CR] IMPALA-5798: ASAN use-after-poison in Parquet decoder

2017-08-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5798: ASAN use-after-poison in Parquet decoder .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7769/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-5798: ASAN use-after-poison in Parquet decoder

2017-08-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-5798: ASAN use-after-poison in Parquet decoder .. IMPALA-5798: ASAN use-after-poison in Parquet decoder In ParquetLevelDecoder::Init() for RLE

[Impala-ASF-CR] IMPALA-5749: coordinator race hits DCHECK 'num remaining backends > 0'

2017-08-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5749: coordinator race hits DCHECK 'num_remaining_backends_ > 0' .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/

[Impala-ASF-CR] IMPALA-5749: coordinator race hits DCHECK 'num remaining backends > 0'

2017-08-21 Thread Thomas Tauber-Marshall (Code Review)
t http://gerrit.cloudera.org:8080/7577 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I1528661e5df6d9732ebfeb414576c82ec5c92241 Gerrit-PatchSet: 5 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Ge

[Impala-ASF-CR] IMPALA-5798: ASAN use-after-poison in Parquet decoder

2017-08-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7769 Change subject: IMPALA-5798: ASAN use-after-poison in Parquet decoder .. IMPALA-5798: ASAN use-after-poison in Parquet decoder

[Impala-ASF-CR] Bump Kudu version to 44a820b

2017-08-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 44a820b .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7767 To unsubscribe, visit http

[native-toolchain-CR] Bump Kudu version to 44a820b

2017-08-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 44a820b .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7755 To unsubscribe, visit http

[Impala-ASF-CR] IMPALA-5799: Kudu DML can crash if schema has changed

2017-08-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5799: Kudu DML can crash if schema has changed .. Patch Set 3: (6 comments) http://gerrit.cloudera.org:8080/#/c/7688/2//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-5799: Kudu DML can crash if schema has changed

2017-08-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-5799: Kudu DML can crash if schema has changed .. IMPALA-5799: Kudu DML can crash if schema has changed We check that the number/types of columns

[Impala-ASF-CR] IMPALA-5799: Kudu DML can crash if schema has changed

2017-08-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-5799: Kudu DML can crash if schema has changed .. IMPALA-5799: Kudu DML can crash if schema has changed We check that the number/types of columns

[Impala-ASF-CR] IMPALA-5799: Kudu DML can crash if schema has changed

2017-08-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5799: Kudu DML can crash if schema has changed .. Patch Set 2: (9 comments) http://gerrit.cloudera.org:8080/#/c/7688/1//COMMIT_MSG Commit

[Impala-ASF-CR] IMPALA-5749: coordinator race hits DCHECK 'num remaining backends > 0'

2017-08-17 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5749: coordinator race hits DCHECK 'num_remaining_backends_ > 0' .. Patch Set 4: Code-Review+1 Carrying forward +1 -- To vie

[Impala-ASF-CR] IMPALA-5749: coordinator race hits DCHECK 'num remaining backends > 0'

2017-08-17 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5749: coordinator race hits DCHECK 'num_remaining_backends_ > 0' .. Patch Set 4: (4 comments) > (3 comments) > > Tha

[Impala-ASF-CR] IMPALA-5749: coordinator race hits DCHECK 'num remaining backends > 0'

2017-08-17 Thread Thomas Tauber-Marshall (Code Review)
t http://gerrit.cloudera.org:8080/7577 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I1528661e5df6d9732ebfeb414576c82ec5c92241 Gerrit-PatchSet: 4 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall

[Impala-ASF-CR] IMPALA-5796: CTAS for Kudu fails with expr rewrite

2017-08-17 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5796: CTAS for Kudu fails with expr rewrite .. Patch Set 3: Code-Review+2 (1 comment) Carrying forward +2 http://gerrit.cloudera.org:8080/#/c

[Impala-ASF-CR] IMPALA-5796: CTAS for Kudu fails with expr rewrite

2017-08-17 Thread Thomas Tauber-Marshall (Code Review)
80/7666 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia67bb33736b5a843663b226cdd0fa5bd839cbea1 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerri

[native-toolchain-CR] Bump Kudu version to b198ed8

2017-08-16 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to b198ed8 .. Patch Set 1: > corresponding Impala change? This was included in the toolchain bump Lars did: ht

[Impala-ASF-CR] IMPALA-5796: CTAS for Kudu fails with expr rewrite

2017-08-16 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5796: CTAS for Kudu fails with expr rewrite .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/7666/1/fe/src/main/java/org/apache

[Impala-ASF-CR] IMPALA-5796: CTAS for Kudu fails with expr rewrite

2017-08-16 Thread Thomas Tauber-Marshall (Code Review)
ibe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia67bb33736b5a843663b226cdd0fa5bd839cbea1 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Matthew Jacobs

[Impala-ASF-CR] IMPALA-5799: Kudu DML can crash if schema has changed

2017-08-16 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7688 Change subject: IMPALA-5799: Kudu DML can crash if schema has changed .. IMPALA-5799: Kudu DML can crash if schema has changed

[Impala-ASF-CR] IMPALA-5477: Fix minidump-2-core tool

2017-08-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5477: Fix minidump-2-core tool .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7677 To unsubscribe, visit http

[Impala-ASF-CR] IMPALA-5787: Dropped status in KuduTableSink::Send()

2017-08-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5787: Dropped status in KuduTableSink::Send() .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7667/1/be/src/exec/kudu-table

[Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE

2017-08-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-5787: Dropped status in KuduTableSink::Send()

2017-08-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7667 Change subject: IMPALA-5787: Dropped status in KuduTableSink::Send() .. IMPALA-5787: Dropped status in KuduTableSink::Send

[Impala-ASF-CR] IMPALA-5708: Test failure with invalid exec summary

2017-08-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has submitted this change and it was merged. Change subject: IMPALA-5708: Test failure with invalid exec summary .. IMPALA-5708: Test failure with invalid exec summary For some queries, the exec summary

[Impala-ASF-CR] IMPALA-5708: Test failure with invalid exec summary

2017-08-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5708: Test failure with invalid exec summary .. Patch Set 3: Code-Review+2 Carrying forward -- To view, visit http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-5796: CTAS for Kudu fails with expr rewrite

2017-08-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7666 Change subject: IMPALA-5796: CTAS for Kudu fails with expr rewrite .. IMPALA-5796: CTAS for Kudu fails with expr rewrite When

[native-toolchain-CR] Bump Kudu version to b198ed8

2017-08-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has submitted this change and it was merged. Change subject: Bump Kudu version to b198ed8 .. Bump Kudu version to b198ed8 Change-Id: Ia62deb0e61f06de4ce4e95476b85988e1e2754bb --- M buildall.sh 1 file

[native-toolchain-CR] Bump Kudu version to b198ed8

2017-08-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to b198ed8 .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7665 To unsubscribe, visit http

[native-toolchain-CR] Bump Kudu version to b198ed8

2017-08-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7665 Change subject: Bump Kudu version to b198ed8 .. Bump Kudu version to b198ed8 Change-Id

[Impala-ASF-CR] IMPALA-5749: coordinator race hits DCHECK 'num remaining backends > 0'

2017-08-11 Thread Thomas Tauber-Marshall (Code Review)
udera.org:8080/7577 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I1528661e5df6d9732ebfeb414576c82ec5c92241 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: He

[Impala-ASF-CR] IMPALA-5681: release reservation from blocking operators

2017-08-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5681: release reservation from blocking operators .. Patch Set 3: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/7619/3

[Impala-ASF-CR] IMPALA-5708: Test failure with invalid exec summary

2017-08-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5708: Test failure with invalid exec summary .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7627/1/tests/beeswax

[Impala-ASF-CR] IMPALA-5708: Test failure with invalid exec summary

2017-08-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-5708: Test failure with invalid exec summary .. IMPALA-5708: Test failure with invalid exec summary For some queries, the exec summary will not be

[Impala-ASF-CR] IMPALA-5708: Test failure with invalid exec summary

2017-08-09 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7627 Change subject: IMPALA-5708: Test failure with invalid exec summary .. IMPALA-5708: Test failure with invalid exec summary For

[Impala-ASF-CR] Bump Kudu version to 943b1ae

2017-08-09 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 943b1ae .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7626 To unsubscribe, visit http

[native-toolchain-CR] Bump Kudu version to 943b1ae

2017-08-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 943b1ae .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7601 To unsubscribe, visit http

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

2017-08-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables .. Patch Set 4: Code-Review+2 (1 comment) Carrying forward. http://gerrit.cloudera.org:8080/#/c

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

2017-08-07 Thread Thomas Tauber-Marshall (Code Review)
udera.org:8080/7446 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I281f173dbec1484eb13434d53ea581a0f245358a Gerrit-PatchSet: 4 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Matthew Jacobs Gerrit-Reviewer: Thomas Tauber-Marshall

[Impala-ASF-CR] IMPALA-5749: coordinator race hits DCHECK 'num remaining backends > 0'

2017-08-07 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5749: coordinator race hits DCHECK 'num_remaining_backends_ > 0' .. Patch Set 2: Code-Review+1 Carrying forward +1 -- To vie

[Impala-ASF-CR] IMPALA-5749: coordinator race hits DCHECK 'num remaining backends > 0'

2017-08-04 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5749: coordinator race hits DCHECK 'num_remaining_backends_ > 0' .. Patch Set 2: (1 comment) > > Does this trigger only

[Impala-ASF-CR] IMPALA-5749: coordinator race hits DCHECK 'num remaining backends > 0'

2017-08-04 Thread Thomas Tauber-Marshall (Code Review)
MessageType: newpatchset Gerrit-Change-Id: I1528661e5df6d9732ebfeb414576c82ec5c92241 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Henry Robinson Gerrit-Reviewer: Michael Ho Gerrit-Reviewer: Sailesh Mukil

[Impala-ASF-CR] IMPALA-5725: coalesce() with outer join incorrectly rewritten

2017-08-04 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5725: coalesce() with outer join incorrectly rewritten .. Patch Set 2: Code-Review+2 GVO failed due to IMPALA-5760 -- To view, visit http

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

2017-08-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7446/2/testdata/workloads

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

2017-08-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables .. IMPALA-5546: Allow creating unpartitioned Kudu tables This patch makes it possible to create

[Impala-ASF-CR] IMPALA-5749: coordinator race hits DCHECK 'num remaining backends > 0'

2017-08-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7577 Change subject: IMPALA-5749: coordinator race hits DCHECK 'num_remaining_backends_ > 0' .. IMPALA-5749: coordina

[Impala-ASF-CR] IMPALA-5725: coalesce() with outer join incorrectly rewritten

2017-08-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5725: coalesce() with outer join incorrectly rewritten .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/7567/1/fe/src/main/java

[Impala-ASF-CR] IMPALA-5725: coalesce() with outer join incorrectly rewritten

2017-08-02 Thread Thomas Tauber-Marshall (Code Review)
ubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I1ca6df949f9d416ab207016236dbcb5886295337 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Matthew Jacobs

[Impala-ASF-CR] IMPALA-3548: Prune runtime filters based on query options in the FE

2017-08-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3548: Prune runtime filters based on query options in the FE .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/7564/1/fe/src

[Impala-ASF-CR] IMPALA-5725: coalesce() with outer join incorrectly rewritten

2017-08-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7567 Change subject: IMPALA-5725: coalesce() with outer join incorrectly rewritten .. IMPALA-5725: coalesce() with outer join

[Impala-ASF-CR] Bump Kudu version to 1c0276a

2017-08-02 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7565 Change subject: Bump Kudu version to 1c0276a .. Bump Kudu version to 1c0276a Change-Id

[Impala-ASF-CR] IMPALA-5616: Add --enable minidumps startup flag

2017-08-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5616: Add --enable_minidumps startup flag .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7541 To unsubscribe

[Impala-ASF-CR] IMPALA-5616: Add --enable minidumps startup flag

2017-08-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5616: Add --enable_minidumps startup flag .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/7541/1//COMMIT_MSG Commit Message

[native-toolchain-CR] Bump Kudu version to 1c0276a

2017-08-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has submitted this change and it was merged. Change subject: Bump Kudu version to 1c0276a .. Bump Kudu version to 1c0276a Change-Id: If9437a4731ed721a4d53669340e01892b60e80de --- M buildall.sh 1 file

[native-toolchain-CR] Bump Kudu version to 1c0276a

2017-08-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 1c0276a .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7550 To unsubscribe, visit http

[native-toolchain-CR] Bump Kudu version to 1c0276a

2017-08-01 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7550 Change subject: Bump Kudu version to 1c0276a .. Bump Kudu version to 1c0276a Change-Id

[Impala-ASF-CR] IMPALA-4674: Part 2: port backend exec to BufferPool

2017-07-26 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4674: Part 2: port backend exec to BufferPool .. Patch Set 37: Code-Review+1 partial sort stuff looks good -- To view, visit http

[Impala-ASF-CR] Bump Kudu version to 27854fd

2017-07-24 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to 27854fd .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7486 To unsubscribe, visit http

[native-toolchain-CR] Remove Kudu build workaround for llvm stdlib path

2017-07-24 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Remove Kudu build workaround for llvm stdlib path .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7485 To unsubscribe

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

2017-07-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables .. Patch Set 2: > Nice! I like the warning. How about also a ctas test case? > > I t

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

2017-07-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables .. IMPALA-5546: Allow creating unpartitioned Kudu tables This patch makes it possible to create

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

2017-07-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5498: Support for partial sorts in Kudu INSERTs .. Patch Set 8: Code-Review+2 Rebased -- To view, visit http://gerrit.cloudera.org:8080/7267

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

2017-07-21 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: GVO failure was unrelated. I filed: IMPALA-5692 -- To view, visit http

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

2017-07-21 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 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7450/3/fe/src/test/java

[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

[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-19 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 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6898/4/testdata/workloads

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

2017-07-19 Thread Thomas Tauber-Marshall (Code Review)
:8080/6898 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I9b0b346f37ee43f7f0eefe34a093eddbbdcf2a5e Gerrit-PatchSet: 4 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Impala Publi

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

2017-07-19 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 3: So the GVO failed because there's a custom cluster test that runs i

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

2017-07-19 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 1: (1 comment) > (4 comments) > > Hi, > > Would you p

[Impala-ASF-CR] Bump Kudu version to f144d57

2017-07-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to f144d57 .. Patch Set 1: Code-Review+2 The GVO failure is definitely unrelated, as the test has nothing to do with Kudu. I filed

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

2017-07-18 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 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6898/2/fe/src/main/java

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

2017-07-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE) .. IMPALA-5167: Reduce the number of Kudu clients created (FE) Creating Kudu clients is very

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

2017-07-18 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 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6898/1/fe/src/main/java

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

2017-07-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE) .. IMPALA-5167: Reduce the number of Kudu clients created (FE) Creating Kudu clients is very

[Impala-ASF-CR] Bump Kudu version to f144d57

2017-07-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7451 Change subject: Bump Kudu version to f144d57 .. Bump Kudu version to f144d57 Change-Id

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

2017-07-17 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7446 Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables .. IMPALA-5546: Allow creating unpartitioned Kudu tables

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

2017-07-17 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5498: Support for partial sorts in Kudu INSERTs .. Patch Set 6: (14 comments) http://gerrit.cloudera.org:8080/#/c/7267/6//COMMIT_MSG Commit

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

2017-07-17 Thread Thomas Tauber-Marshall (Code Review)
ettings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ieec2a15a0cc5240b1c13682067ab64670d1e0a38 Gerrit-PatchSet: 7 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Matthew Jacobs Gerrit-Reviewer: Mostafa Mokhtar

[native-toolchain-CR] Bump Kudu version to f144d57

2017-07-17 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has submitted this change and it was merged. Change subject: Bump Kudu version to f144d57 .. Bump Kudu version to f144d57 Change-Id: Ic9986f8dfaecef00e87e1cef93fc5e5e9dc8c810 --- M buildall.sh 1 file

[native-toolchain-CR] Bump Kudu version to f144d57

2017-07-17 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to f144d57 .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7443 To unsubscribe, visit http

<    1   2   3   4   5   6   7   >