Todd Lipcon has submitted this change and it was merged.

Change subject: Reorganize range partition client API

Reorganize range partition client API

This commit redesigns the client APIs dealing with adding and dropping range
partitions. The goal is to make them more consistent and easier to understand.
The biggest public facing change is that all methods which deal with range
partitions are now named that, and the 'bounds' nomenclature has mostly been
dropped. All of these methods are new since 0.9.1, so this shouldn't be a
breaking change (wire compatibility is not broken).

Additionally, range partitions can now be created, added, and dropped with
exclusive lower bounds and inclusive upper bounds. Originally I thought this
would be a pretty minor feature, but I think it's going to end up being useful
when creating higher level SQL bindings.

Finally, the error messages when altering tables have been significantly
improved with new pretty printing of the offending range partition. Before this
it would print the serialized range partition keys, which are pretty much
impossible to figure out.

An equivalent cleanup for the Java client will be in a follow up commit.

Change-Id: Ic731f0abcbb81cd4238cc960b150bf8cd1c1f0ea
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <>
M java/kudu-client/src/test/java/org/apache/kudu/client/
M src/kudu/client/
M src/kudu/client/
M src/kudu/client/client.h
M src/kudu/client/
M src/kudu/client/table_alterer-internal.h
M src/kudu/client/table_creator-internal.h
M src/kudu/common/
M src/kudu/common/
M src/kudu/common/partition.h
M src/kudu/common/
M src/kudu/common/
M src/kudu/common/wire_protocol.proto
M src/kudu/integration-tests/
M src/kudu/integration-tests/
M src/kudu/integration-tests/
M src/kudu/master/
M src/kudu/master/
M src/kudu/tablet/transactions/
M src/kudu/tserver/
20 files changed, 838 insertions(+), 190 deletions(-)

  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified

To view, visit
To unsubscribe, visit

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic731f0abcbb81cd4238cc960b150bf8cd1c1f0ea
Gerrit-PatchSet: 6
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <>
Gerrit-Reviewer: Adar Dembo <>
Gerrit-Reviewer: Dan Burkert <>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <>

Reply via email to