Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21495 )
Change subject: KUDU-3577 fix altering tables with custom hash schemas ...................................................................... KUDU-3577 fix altering tables with custom hash schemas Since partition boundaries for ranges with custom hash schemas are represented via RowOperationsPB (see RangeWithHashSchemaPB::range_bounds field in src/kudu/common/common.proto), addressing this design defect requires re-encoding the information as a part of PartitionSchemaPB stored in the system catalog upon particular modifications of the table's schema. This patch does exactly so, and also adds corresponding test scenario which would fail without the fix. A proper solution would be using primary-key-only projection of a table's schema to encode the information on range boundaries, but it's necessary to provide backwards compatibility with already released Kudu clients. See KUDU-3577 for details. Change-Id: I21a775538063768b986edd2b6bc25d03360b5216 Reviewed-on: http://gerrit.cloudera.org:8080/21486 Tested-by: Alexey Serbin <ale...@apache.org> Reviewed-by: Mahesh Reddy <mre...@cloudera.com> Reviewed-by: Abhishek Chennaka <achenn...@cloudera.com> (cherry picked from commit d254964e6037f6ae0c9459d99cffa13303596f07) Reviewed-on: http://gerrit.cloudera.org:8080/21495 --- M src/kudu/integration-tests/alter_table-test.cc M src/kudu/master/catalog_manager.cc M src/kudu/master/catalog_manager.h 3 files changed, 172 insertions(+), 3 deletions(-) Approvals: Mahesh Reddy: Looks good to me, but someone else must approve Abhishek Chennaka: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/21495 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: I21a775538063768b986edd2b6bc25d03360b5216 Gerrit-Change-Number: 21495 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Ashwani Raina <ara...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <mre...@cloudera.com>
