aihuaxu opened a new pull request, #512:
URL: https://github.com/apache/parquet-format/pull/512

   <!--
   Thanks for opening a pull request!
   
   If you're new to Parquet-Format, information on how to contribute can be 
found here: https://parquet.apache.org/docs/contribution-guidelines/contributing
   
   Please open a GitHub issue for this pull request: 
https://github.com/apache/parquet-format/issues/new/choose
   and format pull request title as below:
   
       GH-${GITHUB_ISSUE_ID}: ${SUMMARY}
   
   or simply use the title below if it is a minor issue:
   
       MINOR: ${SUMMARY}
   
   -->
   
   ### Rationale for this change
   From the test in https://github.com/apache/arrow-go/pull/455 which validates 
the cross-language variant implementation, we may need to update a few wording 
in the spec to make it clear 
   
   ### What changes are included in this PR?
   1. In variant array, either `value` and `typed_value` fields can be omitted 
but not both in array elements. 
   2. The readers can assume the schema is written properly and will not fail 
the object group is annotated as optional.
   
   ### Do these changes have PoC implementations?
   
   
   <!-- Please uncomment the line below and replace ${GITHUB_ISSUE_ID} with the 
actual Github issue id. -->
   <!-- Closes #${GITHUB_ISSUE_ID} -->
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to