Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18633 )
Change subject: KUDU-2671 introduce RANGE_SPECIFIC_HASH_SCHEMA feature flag ...................................................................... KUDU-2671 introduce RANGE_SPECIFIC_HASH_SCHEMA feature flag This patch introduces a new RANGE_SPECIFIC_HASH_SCHEMA flag for master to signal that a Kudu cluster is able to work with tables having range-specific hash schemas (a.k.a. custom hash schemas per range). In addition, now C++ client requires the new flag to be present at the server side when creating a table having at least one range partition with custom hash schema or when adding a new range partition with custom hash schema. The rationale for introducing the flag is the following: if there were no RANGE_SPECIFIC_HASH_SCHEMA flag and a newer client were not requiring the server to have such a flag, the client would not get an error while trying to perform the following operations against tablet servers of prior versions: * Creating a table having a range partition with custom hash schema * Adding a range partition with custom hash schema to existing table That's because the information on custom hash schemas is provided via newly added flags in corresponding protobuf structures, and the old server would simply ignore the fields, assuming all the ranges to be created have the table-wide hash schema. A follow-up patch will add similar functionality for Kudu Java client. Change-Id: I256d32003e869939e7aa581b21bbe1e77c1e3aba Reviewed-on: http://gerrit.cloudera.org:8080/18633 Reviewed-by: Mahesh Reddy <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Attila Bukor <[email protected]> --- M src/kudu/client/client-internal.cc M src/kudu/client/client-internal.h M src/kudu/client/client.cc M src/kudu/client/flex_partitioning_client-test.cc M src/kudu/client/table_alterer-internal.h M src/kudu/master/master.proto M src/kudu/master/master_service.cc 7 files changed, 175 insertions(+), 10 deletions(-) Approvals: Mahesh Reddy: Looks good to me, but someone else must approve Abhishek Chennaka: Looks good to me, but someone else must approve Alexey Serbin: Verified Attila Bukor: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/18633 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I256d32003e869939e7aa581b21bbe1e77c1e3aba Gerrit-Change-Number: 18633 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]>
