Wes McKinney has posted comments on this change.

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


Patch Set 1:

This seems reasonable to me. If you anticipate partitioning options growing in 
complexity, then packaging this up in a configurable object makes sense (versus 
adding more and more parameters to the create_table method).

-- 
To view, visit http://gerrit.cloudera.org:8080/3196
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie92c897c559fb3070240c51ceb03fe7c2ccd17ba
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Dan Burkert <d...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Wes McKinney <w...@cloudera.com>
Gerrit-HasComments: No

Reply via email to