Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/18765


Change subject: [client] KUDU-2671 move KuduRangePartition out of 
KuduTableCreator
......................................................................

[client] KUDU-2671 move KuduRangePartition out of KuduTableCreator

This patch moves the KuduRangePartition class out of the
KuduTableCreator class.  The reasoning behind this change is two-fold:
  * make the API of KuduTableAlterer more consistent with the added
    AddRangePartition(KuduRangePartition* partition) method
  * avoid issues with embedded classes while cythonizing C++ client API
    while adding support for range-specific hash schemas

This change might break ABI compatibility, but we haven't announced
support for range-specific hash schemas in Kudu 1.16 since it was not
ready yet.  Any Kudu C++ client application that started using that
experimental API (which is very unlikely) should be recompiled with
updated headers and the kudu_client library.

Change-Id: I5af14e7e802baca496e13e05860d66685914dd29
---
M src/kudu/client/client.cc
M src/kudu/client/client.h
M src/kudu/client/flex_partitioning_client-test.cc
M src/kudu/client/scan_token-test.cc
M src/kudu/client/table_alterer-internal.h
M src/kudu/client/table_creator-internal.cc
M src/kudu/client/table_creator-internal.h
7 files changed, 103 insertions(+), 110 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/65/18765/1
--
To view, visit http://gerrit.cloudera.org:8080/18765
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5af14e7e802baca496e13e05860d66685914dd29
Gerrit-Change-Number: 18765
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to