Alexey Serbin has uploaded this change for review. (
http://gerrit.cloudera.org:8080/18045
Change subject: WIP KUDU-2671 disallow the variation of per-range hash
dimensions
......................................................................
WIP KUDU-2671 disallow the variation of per-range hash dimensions
This patch adds a restriction on the number of hash dimensions for
per-range hash schemas: it should be the same for all the ranges.
It's allowed to change the set of columns for the hash bucketing
and other parameters, but the number of hash dimensions should stay
the same.
As a part of this changelist, a few test scenarios are now disabled:
those are to be re-enabled once the rest of the code in system catalog
and the client metacache is able to handle varying hash dimensions.
I updated the comparison operator for PartitionKey: since the number
of hash dimensions isn't varying across per-range hash schemas, it's
no longer necessary to concatenate the hash and the range parts to
provide the legacy ordering of partition keys for some edge cases.
I guess the operator might change if switching to just having a single
string under the hood.
WIP
* collect initial feedback
* probably, switch to keeping just one string under PartitionKey
and return strings views from the range_key() and hash_key() methods
* add proper commit description
* provide the context about this change
* add a test for the Java client as well
Change-Id: Ic884fa556462b85c64d77385a521d9077d33c7c1
---
M src/kudu/client/flex_partitioning_client-test.cc
M src/kudu/common/partition.h
M src/kudu/common/partition_pruner-test.cc
M src/kudu/master/catalog_manager.cc
4 files changed, 332 insertions(+), 34 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/45/18045/1
--
To view, visit http://gerrit.cloudera.org:8080/18045
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic884fa556462b85c64d77385a521d9077d33c7c1
Gerrit-Change-Number: 18045
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
