Dan Burkert has uploaded a new change for review. http://gerrit.cloudera.org:8080/3882
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. An equivalent cleanup for the Java client will be in a follow up commit. Change-Id: Ic731f0abcbb81cd4238cc960b150bf8cd1c1f0ea --- M src/kudu/client/client-test.cc M src/kudu/client/client.cc M src/kudu/client/client.h M src/kudu/client/table_alterer-internal.cc M src/kudu/client/table_alterer-internal.h M src/kudu/client/table_creator-internal.h M src/kudu/common/partition-test.cc M src/kudu/common/partition.cc M src/kudu/common/partition.h M src/kudu/common/row_operations.cc M src/kudu/common/wire_protocol.proto M src/kudu/integration-tests/alter_table-randomized-test.cc M src/kudu/integration-tests/alter_table-test.cc M src/kudu/integration-tests/flex_partitioning-itest.cc M src/kudu/master/catalog_manager.cc M src/kudu/tablet/transactions/write_transaction.cc 16 files changed, 634 insertions(+), 78 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/82/3882/1 -- To view, visit http://gerrit.cloudera.org:8080/3882 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic731f0abcbb81cd4238cc960b150bf8cd1c1f0ea Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <d...@cloudera.com>