Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/20219 )

Change subject: KUDU-1945 Update default range partition key
......................................................................

KUDU-1945 Update default range partition key

The default range partition key includes all the columns of the
primary key. We should not include auto incrementing column as
this is not expected to be a part of the partition key by design.

Change-Id: I89818ceb261064369a63712f6c093f41e57ca5cc
Reviewed-on: http://gerrit.cloudera.org:8080/20219
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <[email protected]>
---
M src/kudu/common/partition-test.cc
M src/kudu/common/partition.cc
2 files changed, 23 insertions(+), 2 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/20219
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I89818ceb261064369a63712f6c093f41e57ca5cc
Gerrit-Change-Number: 20219
Gerrit-PatchSet: 5
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>

Reply via email to