Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1890 Allow renaming of primary key column ......................................................................
KUDU-1890 Allow renaming of primary key column Removed the checks preventing renaming of primary key columns, added test cases and updated documents. Testing: Ran a loop of 100 alter_table-randomized-test.cc test successfully which includes modification to rename a primary key and non-primary columns. Addressed code review comments Change-Id: I28a8c52bdb9ac5a3661f9a07c737f7252466d307 Reviewed-on: http://gerrit.cloudera.org:8080/6078 Reviewed-by: Todd Lipcon <[email protected]> Tested-by: Todd Lipcon <[email protected]> --- M docs/developing.adoc M docs/known_issues.adoc M docs/kudu_impala_integration.adoc M docs/schema_design.adoc M java/kudu-client/src/test/java/org/apache/kudu/client/TestAlterTable.java M src/kudu/client/client-test.cc M src/kudu/client/table_alterer-internal.cc M src/kudu/common/schema-test.cc M src/kudu/common/schema.h M src/kudu/integration-tests/alter_table-randomized-test.cc M src/kudu/integration-tests/alter_table-test.cc M src/kudu/master/catalog_manager.cc M src/kudu/tablet/tablet.cc 13 files changed, 219 insertions(+), 48 deletions(-) Approvals: Todd Lipcon: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/6078 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28a8c52bdb9ac5a3661f9a07c737f7252466d307 Gerrit-PatchSet: 14 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Ram Mettu <[email protected]> Gerrit-Reviewer: Ram Mettu <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]>
