[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

[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)

[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

[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

[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)

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

2017-07-05 Thread Thomas Tauber-Marshall (Code Review)
Hello Matthew Jacobs, Tim Armstrong, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7346 to look at the new patch set (#5). Change subject: IMPALA-5611: KuduPartitionExpr holds onto memory unnecessarily

[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, do you see

[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)

[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)

[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

[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)

[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, eg. in

[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

[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

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

2017-06-20 Thread Thomas Tauber-Marshall (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6955 to look at the new patch set (#10). Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. IMPALA-4622: Add

[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

[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

[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)

[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)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6955 to look at the new patch set (#9). Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. IMPALA-4622: Add ALTER

[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)

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

2017-06-16 Thread Thomas Tauber-Marshall (Code Review)
Hello Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6955 to look at the new patch set (#8). Change subject: IMPALA-4622: Add ALTER COLUMN statement. .. IMPALA-4622: Add ALTER

[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-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)

[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 value

[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)

[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 > of

[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 value

[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 value

[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)

[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

[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-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)

[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 > systems this

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

2017-06-05 Thread Thomas Tauber-Marshall (Code Review)
Hello Lars Volker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7072 to look at the new patch set (#2). Change subject: IMPALA-5426: Update Hive schema script to 1.1.0 .. IMPALA-5426:

[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)
Hello Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6975 to look at the new patch set (#2). Change subject: Bumped Kudu version to 795c435 .. Bumped Kudu version to

[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)

[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 value

[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

[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 getting a

[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 this new

[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 value

[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)

[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 locally

[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)

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

2017-05-19 Thread Thomas Tauber-Marshall (Code Review)
Hello Michael Ho, Matthew Jacobs, Dan Hecht, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6923 to look at the new patch set (#3). Change subject: IMPALA-5340: Query profile displays stale query state

[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 timing

[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)

[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)

[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 like

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

2017-05-16 Thread Thomas Tauber-Marshall (Code Review)
Hello Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6792 to look at the new patch set (#6). Change subject: IMPALA-5167: Reduce the number of Kudu clients created (BE) ..

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

2017-05-12 Thread Thomas Tauber-Marshall (Code Review)
Hello Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6792 to look at the new patch set (#5). Change subject: IMPALA-5167: Reduce the number of Kudu clients created ..

[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 left on

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

2017-05-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6859 Change subject: IMPALA-5305: test_observability.py failing on s3, localFS and Isilon .. IMPALA-5305: test_observability.py

[Impala-ASF-CR] IMPALA-5137: Support TIMESTAMPs in Kudu range predicate DDL

2017-05-11 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5137: Support TIMESTAMPs in Kudu range predicate DDL .. Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/6849/1//COMMIT_MSG Commit

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

2017-05-10 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 .. Patch Set 4: Code-Review+1 (2 comments) > How about a rebase? Done

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

2017-05-10 Thread Thomas Tauber-Marshall (Code Review)
Hello Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6792 to look at the new patch set (#4). Change subject: IMPALA-5167: Reduce the number of Kudu clients created ..

[Impala-ASF-CR] IMPALA-5294: Kudu INSERT partitioning fails with constants

2017-05-10 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5294: Kudu INSERT partitioning fails with constants .. Patch Set 3: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-5294: Kudu INSERT partitioning fails with constants

2017-05-10 Thread Thomas Tauber-Marshall (Code Review)
Hello Marcel Kornacker, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6828 to look at the new patch set (#3). Change subject: IMPALA-5294: Kudu INSERT partitioning fails with constants

[Impala-ASF-CR] IMPALA-4499: Table name missing from exec summary

2017-05-10 Thread Thomas Tauber-Marshall (Code Review)
Hello Matthew Jacobs, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6832 to look at the new patch set (#2). Change subject: IMPALA-4499: Table name missing from exec summary ..

[Impala-ASF-CR] IMPALA-4499: Table name missing from exec summary

2017-05-10 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-4499: Table name missing from exec summary .. Patch Set 2: Code-Review+1 (1 comment)

[Impala-ASF-CR] IMPALA-5294: Kudu INSERT partitioning fails with constants

2017-05-09 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has posted comments on this change. Change subject: IMPALA-5294: Kudu INSERT partitioning fails with constants .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-5294: Kudu INSERT partitioning fails with constants

2017-05-09 Thread Thomas Tauber-Marshall (Code Review)
Thomas Tauber-Marshall has uploaded a new patch set (#2). Change subject: IMPALA-5294: Kudu INSERT partitioning fails with constants .. IMPALA-5294: Kudu INSERT partitioning fails with constants An INSERT into a Kudu table with

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

2017-05-09 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 .. IMPALA-5167: Reduce the number of Kudu clients created Creating Kudu clients is very expensive

<    1   2   3   4   5   6   >