zeroshade opened a new pull request, #311:
URL: https://github.com/apache/arrow-go/pull/311

   ### Rationale for this change
   While working on iceberg-go I discovered a bug in how we converted Arrow 
<--> Parquet schemas when dealing with the nullability of the element for a 
list field.
   
   ### What changes are included in this PR?
   Fixing the proper handling for nullability instead of blanket setting it to 
nullable for elements of lists. Thus correctly propagating the nullability.
   
   ### Are these changes tested?
   Yes, unit tests are added.
   
   ### Are there any user-facing changes?
   Parquet schemas that are being created from Arrow Schemas will now properly 
respect the required/optional for elements of lists.
   


-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to