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 (#3).
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 the 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 at https://gerrit.cloudera.org/#/c/17643/).
WIP:
* 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.
* Clarify why current code doesn't support range partitions with
custom hash bucket scheme if no table-wide hash bucket scheme is
declared during table creation: in attemp to insert into
corresponding ranges, the following error surfaces:
Not found: No tablet covering the requested range partition:
NonCoveredRange { lower_bound: (RANGE (key): <range-key-decode-error: Invalid
argument: unable to fully decode range key: \x80\x00\x002>), upper_bound:
(<end>), ttl: 299999ms })
Change-Id: I98fd9754db850dcdd00a00738f470673f42ac5b4
---
M src/kudu/client/CMakeLists.txt
M src/kudu/client/client-test.cc
M src/kudu/client/client.cc
M src/kudu/client/client.h
A src/kudu/client/flex_partitioning_client-test.cc
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
10 files changed, 679 insertions(+), 60 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/57/17657/3
--
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: 3
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]>
