Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17769 )
Change subject: KUDU-2671 key encoding for custom hash bucket schemas ...................................................................... KUDU-2671 key encoding for custom hash bucket schemas This patch updates the PartitionSchema class to properly encode range keys in the presence of range partitions with custom hash bucket schemas in a Kudu table. I added a few new test scenarios and updated the existing ones accordingly to make it possible to send write operations into appropriate tablets. In addition, I'm sneaking in minor style-related updates in * src/kudu/tablet/tablet.cc * src/kudu/master/catalog_manager.cc As one can see, the newly added method GetHashBucketSchemasForKey() is used in PartitionSchema also in case when Partition object is at hand. A follow-up changelist is going to change that, so there will be no need to perform a dictionary lookup once the Partition class contains information about its hash bucket schema. This is a follow-up to 586b7913258df2d0ee75470ddfb2b88d472ba235. Change-Id: I81aa1c10998d88a1bd5314fc3132037b1c8bbe4b Reviewed-on: http://gerrit.cloudera.org:8080/17769 Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[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/meta_cache.cc M src/kudu/client/write_op.h M src/kudu/common/key_encoder.cc M src/kudu/common/key_encoder.h M src/kudu/common/partition-test.cc M src/kudu/common/partition.cc M src/kudu/common/partition.h M src/kudu/master/catalog_manager.cc M src/kudu/tablet/tablet.cc 12 files changed, 340 insertions(+), 107 deletions(-) Approvals: Kudu Jenkins: Verified Andrew Wong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/17769 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I81aa1c10998d88a1bd5314fc3132037b1c8bbe4b Gerrit-Change-Number: 17769 Gerrit-PatchSet: 6 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]>
