Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20234 )
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]> (cherry picked from commit 4d0a654024981b78e36912aaf7838705e4ca0e78) Reviewed-on: http://gerrit.cloudera.org:8080/20234 Reviewed-by: Yingchun Lai <[email protected]> Tested-by: Yingchun Lai <[email protected]> --- M src/kudu/common/partition-test.cc M src/kudu/common/partition.cc 2 files changed, 23 insertions(+), 2 deletions(-) Approvals: Yingchun Lai: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/20234 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: I89818ceb261064369a63712f6c093f41e57ca5cc Gerrit-Change-Number: 20234 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]>
