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 (#3). Change subject: [partitioning] KUDU-2671 Support for different hash partitioning per range partition. ...................................................................... [partitioning] KUDU-2671 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, 166 insertions(+), 48 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/58/15758/3 -- 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: 3 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)