[
https://issues.apache.org/jira/browse/PARQUET-278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14548336#comment-14548336
]
Tianshuo Deng commented on PARQUET-278:
---------------------------------------
Not sure what you mean by "using constructors doesn't help ensure types are
valid". With the fluentAPI, you do the check in the builder, with constructors,
you put the check in the constructor. I don't see in the case of constructing a
MessageType, what can be done in the builder can not be done in the constructor.
The goal is to construct a tree(nested) MessageType object, IMO, the things
that matters are:
1. Prefer API to reflect the structure of the schema being constructed
2. The simpler the better
No strong opinion on doing it on builders or constructors though, but prefer
simplicity.
> enforce non empty group on MessageType level
> --------------------------------------------
>
> Key: PARQUET-278
> URL: https://issues.apache.org/jira/browse/PARQUET-278
> Project: Parquet
> Issue Type: Improvement
> Reporter: Tianshuo Deng
>
> As columnar format, parquet currently does not support empty struct/group
> without leaves. We should throw when constructing an empty GroupType to give
> a clear message.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
