Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20525 )
Change subject: KUDU-3515: fix incompatibility introduced with KUDU-2671 ...................................................................... KUDU-3515: fix incompatibility introduced with KUDU-2671 This patch addresses the issue reported in KUDU-3515. The issue has been originally reported at the #kudu-general Slack channel [1]. The root cause of the issue is an incompatibility in the serialized representation of tablets' partition keys introduced by changelist [2]. The essence of the fix is to convert the serialized partition keys representing unbounded ends of the tables' ranges from the legacy to the new format on-the-fly while loading that information from the system catalog upon bootstrapping a leader master. This patch contains unit test scenarios for the function that performs the conversion from the legacy to the new format. Also, I verified the fix works as expected for an existing Kudu 1.16.0 cluster with a bunch of tables after upgrading the binaries to Kudu 1.17.0 bits. This is a follow-up to 8df970f7a6520bb0dc0f9cc89ad7f62ab349e84d. [1] https://getkudu.slack.com/archives/C0CPXJ3CH/p1695107377230829 [2] https://github.com/apache/kudu/commit/8df970f7a652 Change-Id: I45df424770a09cf7c94f5e1d390757f29f9fb3f4 Reviewed-on: http://gerrit.cloudera.org:8080/20525 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Mahesh Reddy <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> --- M src/kudu/common/partition.cc M src/kudu/common/partition.h M src/kudu/master/CMakeLists.txt M src/kudu/master/catalog_manager.cc M src/kudu/master/sys_catalog-test.cc A src/kudu/master/tablet_loader.cc A src/kudu/master/tablet_loader.h 7 files changed, 543 insertions(+), 71 deletions(-) Approvals: Alexey Serbin: Verified Mahesh Reddy: Looks good to me, but someone else must approve Abhishek Chennaka: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/20525 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I45df424770a09cf7c94f5e1d390757f29f9fb3f4 Gerrit-Change-Number: 20525 Gerrit-PatchSet: 4 Gerrit-Owner: Alexey Serbin <[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]> Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Yingchun Lai <[email protected]>
