Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18663 )
Change subject: [c++ client] KUDU-2671 Custom hash schema alter table support ...................................................................... [c++ client] KUDU-2671 Custom hash schema alter table support This patch adds public methods to C++ client to alter a table and add a new range partition with custom hash sub-partitioning. We make use of the KuduTableCreator::KuduRangePartition() for this purpose. The necessary changes are done in table_alterer-internal classes and methods to use the above mentioned KuduRangePartition() to store the table bounds information as well as custom hash schema information. Necessary tests are included in this patch which include adding and dropping the ranges with custom hash schema by altering the table. We also read and write the data into these partitions. The pending work in this patch is to rebase on top of https://gerrit.cloudera.org/#/c/17879/ and include test cases with scans with predicates on these partitions. Change-Id: Id4b1e306cca096d9479f06669cc22cc40d77fb42 Reviewed-on: http://gerrit.cloudera.org:8080/18663 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/client/client.cc M src/kudu/client/client.h M src/kudu/client/flex_partitioning_client-test.cc M src/kudu/client/table_alterer-internal.cc M src/kudu/client/table_alterer-internal.h 5 files changed, 289 insertions(+), 29 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/18663 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id4b1e306cca096d9479f06669cc22cc40d77fb42 Gerrit-Change-Number: 18663 Gerrit-PatchSet: 6 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]> Gerrit-Reviewer: Tidy Bot (241)
