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

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

[Impala-ASF-CR] IMPALA-5586: Null-aware anti-join can take a long time to cancel

2017-07-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5586: Null-aware anti-join can take a long time to cancel .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7393

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

2017-07-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5498: Support for partial sorts .. Patch Set 6: (1 comment) > Can you extend the Sort metrics in the query profile to include > Avg, M

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

2017-07-14 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#6). Change subject: IMPALA-5498: Support for partial sorts .. IMPALA-5498: Support for partial sorts Impala currently supports total sorts (the entire set of data is sorted

[Impala-ASF-CR] IMPALA-5585: Fix expr-test to call last day() tests

2017-07-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5585: Fix expr-test to call last_day() tests .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7403 To

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

2017-07-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5498: Support for partial sorts .. Patch Set 5: (5 comments) http://gerrit.cloudera.org:8080/#/c/7267/4/be/src/exec/partial-sort-node.cc File

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

2017-07-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#5). Change subject: IMPALA-5498: Support for partial sorts .. IMPALA-5498: Support for partial sorts Impala currently supports total sorts (the entire set of data is sorted

[Impala-ASF-CR] IMPALA-5586: Null-aware anti-join can take a long time to cancel

2017-07-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5586: Null-aware anti-join can take a long time to cancel .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/7393/2//COMMIT_MSG

[Impala-ASF-CR] Bump Kudu version to 1070e76

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

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

2017-07-10 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5498: Support for partial sorts .. Patch Set 4: (11 comments) http://gerrit.cloudera.org:8080/#/c/7267/3/be/src/exec/partial-sort-node.cc File

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

2017-07-10 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#4). Change subject: IMPALA-5498: Support for partial sorts .. IMPALA-5498: Support for partial sorts Impala currently supports total sorts (the entire set of data is sorted

[native-toolchain-CR] Bump Kudu version to 1070e76

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

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

2017-07-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5498: Support for partial sorts .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/7267/2/common/thrift/PlanNodes.thrift File

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

2017-07-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-5498: Support for partial sorts .. IMPALA-5498: Support for partial sorts Impala currently supports total sorts (the entire set of data is sorted

[Impala-ASF-CR] IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily

2017-07-05 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/7346/4/be/src/runtime

[Impala-ASF-CR] IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily

2017-07-05 Thread Thomas Tauber-Marshall (Code Review)
loudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia661eb8bed114070728a1497ccf7ed6893237e5e Gerrit-PatchSet: 5 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Matthew Jacobs Gerrit-Reviewer: M

[Impala-ASF-CR] IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily

2017-06-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#4). Change subject: IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily .. IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily IMPALA-3742 introduced

[Impala-ASF-CR] IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily

2017-06-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily .. Patch Set 3: (2 comments) > (2 comments) > > nice! After this change, d

[Impala-ASF-CR] IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily

2017-06-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/7346/2/be/src/runtime

[Impala-ASF-CR] IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily

2017-06-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily .. IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily IMPALA-3742 introduced

[Impala-ASF-CR] IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily

2017-06-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7346/1/be/src/exprs

[Impala-ASF-CR] IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily

2017-06-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily .. IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily IMPALA-3742 introduced

[Impala-ASF-CR] IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily

2017-06-30 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7346 Change subject: IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily .. IMPALA-5611: KuduPartitionExpr holds onto

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

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

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

2017-06-29 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bump Kudu version to bbed78c .. Patch Set 1: http://unittest.jenkins.cloudera.com/job/verify-impala-toolchain-package-build/416/ -- To view, visit

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

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

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

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

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

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

[Impala-ASF-CR] IMPALA-5554: sorter DCHECK on null column

2017-06-26 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5554: sorter DCHECK on null column .. Patch Set 1: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/7295/1//COMMIT_MSG Commit

[Impala-ASF-CR] DRAFT - IMPALA-5498: Support for partial sorts

2017-06-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: DRAFT - IMPALA-5498: Support for partial sorts .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/7267/1/be/src/exec/partial-sort

[Impala-ASF-CR] DRAFT - IMPALA-5498: Support for partial sorts

2017-06-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: DRAFT - IMPALA-5498: Support for partial sorts .. DRAFT - IMPALA-5498: Support for partial sorts Impala currently supports total sorts (the entire set of

[Impala-ASF-CR] DRAFT - IMPALA-5498: Support for partial sorts

2017-06-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: DRAFT - IMPALA-5498: Support for partial sorts .. Patch Set 1: (2 comments) I haven't looked much at the Planner changes needed, e

[Impala-ASF-CR] DRAFT - IMPALA-5498: Support for partial sorts

2017-06-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7267 Change subject: DRAFT - IMPALA-5498: Support for partial sorts .. DRAFT - IMPALA-5498: Support for partial sorts Impala

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-06-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. Patch Set 10: Code-Review+2 Rebased, carrying forward -- To view, visit http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-5540: Revert Sentry version back to 5.13

2017-06-21 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7247 Change subject: IMPALA-5540: Revert Sentry version back to 5.13 .. IMPALA-5540: Revert Sentry version back to 5.13 Sentry has

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

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

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-06-20 Thread Thomas Tauber-Marshall (Code Review)
Gerrit-MessageType: newpatchset Gerrit-Change-Id: Id2e8bd65342b79644a0fdcd925e6f17797e89ad6 Gerrit-PatchSet: 10 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Matthew Jacobs Gerrit-Reviewer: Thomas Tauber-Marshall

[native-toolchain-CR] Bump Kudu vesion to c0798a9

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

[native-toolchain-CR] Bump Kudu vesion to c0798a9

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

[Impala-ASF-CR] IMPALA-5530: fix compilation against Sentry

2017-06-19 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5530: fix compilation against Sentry .. Patch Set 5: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/7222 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-5530: fix compilation against Sentry

2017-06-19 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5530: fix compilation against Sentry .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7222/2/fe/src/main/java/org/apache/impala

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-06-19 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. Patch Set 9: (2 comments) http://gerrit.cloudera.org:8080/#/c/6955/8/common/thrift/JniCatalog.thrift File

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-06-19 Thread Thomas Tauber-Marshall (Code Review)
Gerrit-MessageType: newpatchset Gerrit-Change-Id: Id2e8bd65342b79644a0fdcd925e6f17797e89ad6 Gerrit-PatchSet: 9 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Matthew Jacobs Gerrit-Reviewer: Thomas Tauber-Marshall

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-06-16 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/6955/7/fe/src/main/java/org/apache/impala

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-06-16 Thread Thomas Tauber-Marshall (Code Review)
Gerrit-MessageType: newpatchset Gerrit-Change-Id: Id2e8bd65342b79644a0fdcd925e6f17797e89ad6 Gerrit-PatchSet: 8 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Matthew Jacobs Gerrit-Reviewer: Thomas Tauber-Marshall

[Impala-ASF-CR] IMPALA-5286/IMPALA-5283: Kudu column name case cleanup

2017-06-16 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5286/IMPALA-5283: Kudu column name case cleanup .. Patch Set 5: Code-Review+2 Rebased -- To view, visit http://gerrit.cloudera.org:8080/6902

[Impala-ASF-CR] IMPALA-5286/IMPALA-5283: Kudu column name case cleanup

2017-06-16 Thread Thomas Tauber-Marshall (Code Review)
0/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I14aba88510012174716691b9946e1c7d54d01b44 Gerrit-PatchSet: 5 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Matthew Jacobs Gerrit-Reviewer: Thomas Tauber-Marshall

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-06-16 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. Patch Set 6: (5 comments) http://gerrit.cloudera.org:8080/#/c/6955/5/fe/src/main/java/org/apache/impala

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-06-16 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#6). Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. IMPALA-4622: Add ALTER COLUMN statement. Kudu recently added the ability to alter a column's default

[Impala-ASF-CR] IMPALA-5286/IMPALA-5283: Kudu column name case cleanup

2017-06-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5286/IMPALA-5283: Kudu column name case cleanup .. Patch Set 3: (10 comments) http://gerrit.cloudera.org:8080/#/c/6902/2/common/thrift

[Impala-ASF-CR] IMPALA-5286/IMPALA-5283: Kudu column name case cleanup

2017-06-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#4). Change subject: IMPALA-5286/IMPALA-5283: Kudu column name case cleanup .. IMPALA-5286/IMPALA-5283: Kudu column name case cleanup Impala is case insensitive for column

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-06-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. Patch Set 5: (18 comments) > We should add an e2e test that exhaustively tests all combinations &

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-06-15 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#5). Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. IMPALA-4622: Add ALTER COLUMN statement. Kudu recently added the ability to alter a column's default

[Impala-ASF-CR] IMPALA-5286/IMPALA-5283: Kudu column name case cleanup

2017-06-13 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-5286/IMPALA-5283: Kudu column name case cleanup .. IMPALA-5286/IMPALA-5283: Kudu column name case cleanup Impala is case insensitive for column

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-06-12 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#4). Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. IMPALA-4622: Add ALTER COLUMN statement. Kudu recently added the ability to alter a column's default

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-06-12 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/6955/3/fe/src/main/java/org/apache/impala

[Impala-ASF-CR] IMPALA-3040 addendum: use specific build type timeout for slow builds

2017-06-12 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3040 addendum: use specific_build_type_timeout for slow builds .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080

[Impala-ASF-CR] IMPALA-5453: test create table like file fails on enum.parquet

2017-06-09 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7139 Change subject: IMPALA-5453: test_create_table_like_file fails on enum.parquet .. IMPALA-5453: test_create_table_like_file

[Impala-ASF-CR] IMPALA-3040 addendum: use specific build type timeout for slow builds

2017-06-08 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-3040 addendum: use specific_build_type_timeout for slow builds .. Patch Set 1: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-06-06 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6955/3/fe/src/main/java/org/apache/impala

[Impala-ASF-CR] IMPALA-5428: update external hadoop ecosystem versions

2017-06-05 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5428: update external hadoop ecosystem versions .. Patch Set 2: Code-Review+2 gvo failed because it needed to be rebased -- To view, visit

[Impala-ASF-CR] IMPALA-5428: update external hadoop ecosystem versions

2017-06-05 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5428: update external hadoop ecosystem versions .. Patch Set 1: Lars - sorry to not be clearer here. Yes, this patch will have to go in after

[Impala-ASF-CR] IMPALA-5426: Update Hive schema script to 1.1.0

2017-06-05 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5426: Update Hive schema script to 1.1.0 .. Patch Set 1: > Thank you for fixing this. Is the hive schema file available on all > system

[Impala-ASF-CR] IMPALA-5426: Update Hive schema script to 1.1.0

2017-06-05 Thread Thomas Tauber-Marshall (Code Review)
Gerrit-MessageType: newpatchset Gerrit-Change-Id: I9ea3269c1f95f76d8c02b76a5dea3ca3aa324b70 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Matthew Jacobs

[Impala-ASF-CR] IMPALA-5428: update external hadoop ecosystem versions

2017-06-04 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7076 Change subject: IMPALA-5428: update external hadoop ecosystem versions .. IMPALA-5428: update external hadoop ecosystem

[Impala-ASF-CR] IMPALA-5426: Update Hive schema script to 1.1.0

2017-06-03 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/7072 Change subject: IMPALA-5426: Update Hive schema script to 1.1.0 .. IMPALA-5426: Update Hive schema script to 1.1.0 A recent

[Impala-ASF-CR] IMPALA-5286: Kudu column name case cleanup

2017-05-31 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-5286: Kudu column name case cleanup .. IMPALA-5286: Kudu column name case cleanup Impala is case insensitive for column names and generally deals

[Impala-ASF-CR] Bumped Kudu version to 795c435

2017-05-25 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bumped Kudu version to 795c435 .. Patch Set 2: http://unittest.jenkins.cloudera.com/job/verify-impala-toolchain-package-build/394/ -- To view, visit

[Impala-ASF-CR] Bumped Kudu version to 795c435

2017-05-25 Thread Thomas Tauber-Marshall (Code Review)
/6975 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I925f5d746dcb497e17ab6f85217da9fcee1d4c00 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Impala P

[Impala-ASF-CR] IMPALA-5354: INSERT hints for Kudu tables

2017-05-25 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5354: INSERT hints for Kudu tables .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/6980/1/fe/src/main/java/org/apache/impala

[Impala-ASF-CR] IMPALA-5354: INSERT hints for Kudu tables

2017-05-25 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-5354: INSERT hints for Kudu tables .. IMPALA-5354: INSERT hints for Kudu tables A previous change, IMPALA-3742, added an exchange node and sort

[Impala-ASF-CR] IMPALA-5354: INSERT hints for Kudu tables

2017-05-24 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6980 Change subject: IMPALA-5354: INSERT hints for Kudu tables .. IMPALA-5354: INSERT hints for Kudu tables A previous change

[native-toolchain-CR] Bumped Kudu version to 795c435.

2017-05-24 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has submitted this change and it was merged. Change subject: Bumped Kudu version to 795c435. .. Bumped Kudu version to 795c435. This is needed for IMPALA-5167. Change-Id

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-05-24 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. Patch Set 3: So I simplified this by representing it as a CHANGE operation in the catalog. I think that it

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-05-24 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#3). Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. IMPALA-4622: Add ALTER COLUMN statement. Kudu recently added the ability to alter a column's default

[Impala-ASF-CR] Bumped Kudu version to 795c435

2017-05-24 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6975 Change subject: Bumped Kudu version to 795c435 .. Bumped Kudu version to 795c435 This is needed for IMPALA-5167. Change-Id

[native-toolchain-CR] Bumped Kudu version to 795c435.

2017-05-24 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bumped Kudu version to 795c435. .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6971 To unsubscribe, visit http

[native-toolchain-CR] Bumped Kudu version to 795c435.

2017-05-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6971 Change subject: Bumped Kudu version to 795c435. .. Bumped Kudu version to 795c435. This is needed for IMPALA-5167. Change-Id

[native-toolchain-CR] Bumped Kudu version to 795c435.

2017-05-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: Bumped Kudu version to 795c435. .. Patch Set 1: http://unittest.jenkins.cloudera.com/job/verify-impala-toolchain-package-build/393/ -- To view, visit

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

2017-05-23 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 (BE) .. Patch Set 6: > anything holding this up? if not, can you submit? working on gettin

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-05-23 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. Patch Set 2: > To be a bit more prescriptive than my last comment: > > Can you see how much of

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-05-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. IMPALA-4622: Add ALTER COLUMN statement. Kudu recently added the ability to alter a column's default

[Impala-ASF-CR] IMPALA-5338: Fix Kudu timestamp column default values

2017-05-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5338: Fix Kudu timestamp column default values .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/6936/1/be/src/runtime/timestamp

[Impala-ASF-CR] IMPALA-4622: Add ALTER COLUMN statement.

2017-05-22 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6955 Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. IMPALA-4622: Add ALTER COLUMN statement. Kudu recently added the

[Impala-ASF-CR] IMPALA-5340: Query profile displays stale query state

2017-05-19 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5340: Query profile displays stale query state .. Patch Set 3: Code-Review+2 I should also add - I haven't tested this with CM, just lo

[Impala-ASF-CR] IMPALA-5340: Query profile displays stale query state

2017-05-19 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5340: Query profile displays stale query state .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6923/3/tests/query_test

[Impala-ASF-CR] IMPALA-5340: Query profile displays stale query state

2017-05-19 Thread Thomas Tauber-Marshall (Code Review)
rit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Dan Hecht Gerrit-Reviewer: Matthew Jacobs Gerrit-Reviewer: Michael Ho Gerrit-Reviewer: Thomas Tauber-Marshall

[Impala-ASF-CR] IMPALA-5340: Query profile displays stale query state

2017-05-19 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5340: Query profile displays stale query state .. Patch Set 2: (1 comment) > Any good way to regression test this? Obviously its a tim

[Impala-ASF-CR] IMPALA-5340: Query profile displays stale query state

2017-05-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5340: Query profile displays stale query state .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6923/1/be/src/service/client

[Impala-ASF-CR] IMPALA-5340: Query profile displays stale query state

2017-05-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-5340: Query profile displays stale query state .. IMPALA-5340: Query profile displays stale query state Previously, updates to the query state in

[Impala-ASF-CR] IMPALA-5339: Fix analysis with sort.columns and expr rewrites

2017-05-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5339: Fix analysis with sort.columns and expr rewrites .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6921 To

[Impala-ASF-CR] IMPALA-5340: Query profile displays stale query state

2017-05-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6923 Change subject: IMPALA-5340: Query profile displays stale query state .. IMPALA-5340: Query profile displays stale query state

[Impala-ASF-CR] IMPALA-5339: Fix analysis with sort.columns and expr rewrites

2017-05-18 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5339: Fix analysis with sort.columns and expr rewrites .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6921/1//COMMIT_MSG

[Impala-ASF-CR] IMPALA-5286: Query fails due to Kudu column name case

2017-05-17 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5286: Query fails due to Kudu column name case .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6902/1/fe/src/main/java/org

[Impala-ASF-CR] IMPALA-5286: Query fails due to Kudu column name case

2017-05-16 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6902 Change subject: IMPALA-5286: Query fails due to Kudu column name case .. IMPALA-5286: Query fails due to Kudu column name case

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

2017-05-16 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6898 Change subject: IMPALA-5167: Reduce the number of Kudu clients created (FE) .. IMPALA-5167: Reduce the number of Kudu clients

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

2017-05-16 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 (BE) .. Patch Set 6: (3 comments) > So I just spoke with the Kudu folks and it sounds l

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

2017-05-16 Thread Thomas Tauber-Marshall (Code Review)
ubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I6b0c12a256c33e8ef32315b3736cae2dea2ae705 Gerrit-PatchSet: 6 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Marcel Kornacker Gerrit-Reviewer: Matthew Jac

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

2017-05-12 Thread Thomas Tauber-Marshall (Code Review)
/gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I6b0c12a256c33e8ef32315b3736cae2dea2ae705 Gerrit-PatchSet: 5 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Marcel Kornacker Gerrit-Reviewer: Matthew Jac

[Impala-ASF-CR] IMPALA-5305: test observability.py failing on s3, localFS and Isilon

2017-05-12 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5305: test_observability.py failing on s3, localFS and Isilon .. Patch Set 1: Code-Review+2 GVO failed with unrelated error "No space le

<    1   2   3   4   5   6   7   >