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

ASF GitHub Bot commented on PARQUET-138:
----------------------------------------

rmettu1-rms commented on pull request #623: Merge PARQUET-138: Allow merging 
more restrictive field in less restrictive field
URL: https://github.com/apache/parquet-mr/pull/623
 
 
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


> Parquet should allow a merge between required and optional schemas
> ------------------------------------------------------------------
>
>                 Key: PARQUET-138
>                 URL: https://issues.apache.org/jira/browse/PARQUET-138
>             Project: Parquet
>          Issue Type: Bug
>          Components: parquet-mr
>    Affects Versions: 1.6.0
>            Reporter: Robert Justice
>            Assignee: Nicolas Trinquier
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.11.0
>
>
> In discussion with Ryan, he felt we should be able to merge from required 
> binary to optional binary and the resulting schema would be optional
> https://github.com/Parquet/parquet-mr/blob/master/parquet-column/src/test/java/parquet/schema/TestMessageType.java
> {code:java}
> try {
>       t3.union(t4);
>       fail("moving from optional to required");
>     } catch (IncompatibleSchemaModificationException e) {
>       assertEquals("repetition constraint is more restrictive: can not merge 
> type required binary a into optional binary a", e.getMessage());
>     }
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to