Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Grant Henke,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15758
to look at the new patch set (#7).
Change subject: [partitioning] KUDU-2671 [part 1] Support for different hash
partitioning per range partition.
......................................................................
[partitioning] KUDU-2671 [part 1] Support for different hash partitioning per
range partition.
Updated algorithm in Partition::CreatePartitions() to support different
hash partitioning per range partition. Current range bounds
represented as std::pair, but we need to add third parameter. For that
it would be nice to replace std::pair with std::tuple. But replacing
std::pair with std::tuple would require to change a lot of code. Thus,
I added another parameter to Partition::CreatePartitions(), which
describes hash partitiong schema for every range bound and must have
same size as range bounds.
Change-Id: Ie843f37392240c3a47a589658d9702c2049ee011
---
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/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
8 files changed, 194 insertions(+), 49 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/58/15758/7
--
To view, visit http://gerrit.cloudera.org:8080/15758
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie843f37392240c3a47a589658d9702c2049ee011
Gerrit-Change-Number: 15758
Gerrit-PatchSet: 7
Gerrit-Owner: Volodymyr Verovkin <verjov...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Volodymyr Verovkin <verjov...@cloudera.com>
