Hello Mahesh Reddy, Kudu Jenkins, Andrew Wong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17769
to look at the new patch set (#2).
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 sneaking in minor style and formatting 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
---
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.cc
M src/kudu/common/partition.h
M src/kudu/master/catalog_manager.cc
M src/kudu/tablet/tablet.cc
11 files changed, 249 insertions(+), 100 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/69/17769/2
--
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: newpatchset
Gerrit-Change-Id: I81aa1c10998d88a1bd5314fc3132037b1c8bbe4b
Gerrit-Change-Number: 17769
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
