[
https://issues.apache.org/jira/browse/KUDU-1689?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15566256#comment-15566256
]
Jordan Birdsell edited comment on KUDU-1689 at 10/11/16 6:56 PM:
-----------------------------------------------------------------
[~tlipcon] Yea, thats a fair. My initial line of thinking was to take this
approach to make it more similar to schema, scanner and scan token. One thing
that i liked about the builder approach is that we could cleanly re-couple
partition setting and schema setting (needed for KUDU-1648). Otherwise we may
have to use something like tuples for the range partition implementation and
any errors w/ the PartialRow and the actual schema would be deferred until
table creation time. I suppose we could also require the schema as an arg to
Partition, but that starts getting ugly.
was (Author: jtbirdsell):
Yea, thats a fair. My initial line of thinking was to take this approach to
make it more similar to schema, scanner and scan token. One thing that i liked
about the builder approach is that we could cleanly re-couple partition setting
and schema setting (needed for KUDU-1648). Otherwise we may have to use
something like tuples for the range partition implementation and any errors w/
the PartialRow and the actual schema would be deferred until table creation
time. I suppose we could also require the schema as an arg to Partition, but
that starts getting ugly.
> [Python] - Expose Table Creator API
> -----------------------------------
>
> Key: KUDU-1689
> URL: https://issues.apache.org/jira/browse/KUDU-1689
> Project: Kudu
> Issue Type: New Feature
> Components: python
> Affects Versions: 1.0.0
> Reporter: Jordan Birdsell
> Assignee: Jordan Birdsell
> Priority: Minor
>
> Currently the python client doesn't expose the TableCreator API. To keep the
> table creation process extensible, this should be exposed so that we avoid
> constantly adding parameters to the create_table method.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)