Hello Mahesh Reddy, Tidy Bot, Kudu Jenkins, Bankim Bhavsar,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17680
to look at the new patch set (#2).
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/2
--
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: newpatchset
Gerrit-Change-Id: I66ff63488f34173e2f7f48db5aae73b2a612be8c
Gerrit-Change-Number: 17680
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Tidy Bot (241)
