You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@parquet.apache.org by "Ryan Blue (JIRA)" <ji...@apache.org> on 2015/04/09 19:16:12 UTC

[jira] [Commented] (PARQUET-248) Simplify ParquetWriters's constructors

    [ https://issues.apache.org/jira/browse/PARQUET-248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14487686#comment-14487686 ] 

Ryan Blue commented on PARQUET-248:
-----------------------------------

I built one of these in the parquet-avro reflect support: https://github.com/apache/incubator-parquet-mr/pull/165

It's built to allow moving it eventually into parquet-hadoop and implementing subclasses for the object models (that's why there is a {{getReadSupport}} method).

> Simplify ParquetWriters's constructors
> --------------------------------------
>
>                 Key: PARQUET-248
>                 URL: https://issues.apache.org/jira/browse/PARQUET-248
>             Project: Parquet
>          Issue Type: Improvement
>    Affects Versions: 1.6.0
>            Reporter: Konstantin Shaposhnikov
>
> ParquetWriter has a lot of constructors. A builder pattern can be used to simplify construction of ParquetWriter objects (similar to ParquetReader, see PARQUET-39).
> ParquetWriter subclasses (like AvroParquetWriter) should be updated to provide reasonable builder() static factory method.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)