You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@cloudera.org> on 2016/05/24 20:14:08 UTC

[kudu-CR] python: add support for specifying partitioning

Hello Dan Burkert, Wes McKinney,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/3196

to review the following change.

Change subject: python: add support for specifying partitioning
......................................................................

python: add support for specifying partitioning

This adds support for range and hash partitioning when creating
a table from the Python client.

Unfortunately, range partitioning is less-than-useful right now,
since we don't yet have a way to specify the split rows themselves.
I elected to defer that work since it's a bit tricky -- the PartialRow
constructor currently requires a Table object, and the Table object
isn't available until the table has been created.

Change-Id: Ie92c897c559fb3070240c51ceb03fe7c2ccd17ba
---
M python/kudu/client.pyx
M python/kudu/libkudu_client.pxd
M python/kudu/tests/test_client.py
3 files changed, 105 insertions(+), 5 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie92c897c559fb3070240c51ceb03fe7c2ccd17ba
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Dan Burkert <da...@cloudera.com>
Gerrit-Reviewer: Wes McKinney <we...@cloudera.com>