Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16596 )
Change subject: [partitioning] KUDU-2671: Support for range specific HashSchemas. ...................................................................... [partitioning] KUDU-2671: Support for range specific HashSchemas. This patch updates PartitionSchema::CreatePartitions() to support the ability to add different hash schemas per each range. If no hash schema per range is specified, the table wide hash schema is used. Currently, this only works if no split_rows are specified. Since split_rows only exists for backwards compatibility reasons, this feature will not be supported with split_rows. Returning a message to the user stating to specify both upper and lower bounds either at table creation or alteration time should suffice. Split_rows is also more syntactically ambiguous when specifying bounds. Currently, range_hash_schemas holds the HashBucketSchemas for each range. Its order corresponds to the bounds in range_bounds so that when the bounds are sorted its corresponding hash schemas are sorted as well. Inspiration from Vlad: https://gerrit.cloudera.org/c/15758/ Change-Id: I8725f4bd072a81b05b36dfc7df0c074c172b4ce8 Reviewed-on: http://gerrit.cloudera.org:8080/16596 Reviewed-by: Andrew Wong <[email protected]> Tested-by: Andrew Wong <[email protected]> --- M src/kudu/common/partition-test.cc M src/kudu/common/partition.cc M src/kudu/common/partition.h M src/kudu/common/partition_pruner-test.cc M src/kudu/common/scan_spec-test.cc M src/kudu/integration-tests/ts_tablet_manager-itest.cc M src/kudu/master/catalog_manager.cc M src/kudu/master/sys_catalog.cc M src/kudu/tablet/tablet-harness.h M src/kudu/tserver/tablet_server-test.cc 10 files changed, 455 insertions(+), 126 deletions(-) Approvals: Andrew Wong: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/16596 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I8725f4bd072a81b05b36dfc7df0c074c172b4ce8 Gerrit-Change-Number: 16596 Gerrit-PatchSet: 13 Gerrit-Owner: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Tidy Bot (241)
