Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17680
Change subject: [common] extra test PartitionSchema and a small clean-up ...................................................................... [common] extra test PartitionSchema and a small clean-up While working on adding support for custom hash bucket schemas in range partitions for the Kudu C++ client, I found that a test scenario for custom range partitions with non-existent table-wide hash bucket schema was missing. This patch add a simple one to cover such a case. In addition, I updated the code in partition.cc to use std::move semantics in PartitionSchema::GenerateHashPartitions() and other related methods. I also did other style-related changes. Change-Id: I66ff63488f34173e2f7f48db5aae73b2a612be8c --- M src/kudu/common/partition-test.cc M src/kudu/common/partition.cc M src/kudu/common/partition.h 3 files changed, 337 insertions(+), 203 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/80/17680/1 -- To view, visit http://gerrit.cloudera.org:8080/17680 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I66ff63488f34173e2f7f48db5aae73b2a612be8c Gerrit-Change-Number: 17680 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
