Hello Mahesh Reddy, Kudu Jenkins, Andrew Wong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17657
to look at the new patch set (#2).
Change subject: WIP [client] KUDU-2671 flexible partitioning during table
creation
......................................................................
WIP [client] KUDU-2671 flexible partitioning during table creation
This changelist introduces C++ client API to support flexible table
partitioning while creating a table. In other words, with this patch
it's now possible to create a Kudu table with custom hash bucket
schemas per range partition using Kudu C++ client.
Verification via KuduPartitioner or via PartitionSchema returned by
KuduTable::partition_schema() isn't added yet because it depends on
the proper functioning PartitionPruner, which needs to be updated
correspondingly to understand range partitions with custom hash
schemas (posted as WIP patch at https://gerrit.cloudera.org/#/c/17643/)
WIP:
* Add more test scenarios: multi-level, mixed, negative, end-to-end
(insert rows into the created ranges, check every range gets
rows as expected), etc.
* Probably, separate the newly added test scenarios into a new test:
client-test contains too much scenarios at this point.
* Clarify on whether the current server-side implementation of the
flexible range partitioning supports flat, not hash-partitioned
ranges even if there is table-wide hash bucket partitioning.
* address TODOs, brush up the patch
Change-Id: I98fd9754db850dcdd00a00738f470673f42ac5b4
---
M src/kudu/client/client-test.cc
M src/kudu/client/client.cc
M src/kudu/client/client.h
M src/kudu/client/partitioner-internal.cc
M src/kudu/client/table_creator-internal.cc
M src/kudu/client/table_creator-internal.h
M src/kudu/common/partition.cc
M src/kudu/master/master.proto
8 files changed, 501 insertions(+), 56 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/57/17657/2
--
To view, visit http://gerrit.cloudera.org:8080/17657
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I98fd9754db850dcdd00a00738f470673f42ac5b4
Gerrit-Change-Number: 17657
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
