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

   ### Rationale for this change
   https://github.com/apache/iceberg-go/issues/398 discovered that the current 
`NewStructArrayWithFields` fails if any child is marked as non-nullable but has 
nulls (as would be the case in an optional struct array full of nulls with a 
required field). So we need to allow constructing such an array.
   
   ### What changes are included in this PR?
   A new function is created, `NewStructArrayWithFieldsAndNulls` which takes in 
the top level struct null bitmap, the number of nulls, offset columns and list 
of fields.
   
   ### Are these changes tested?
   Yes, a unit test was created for it.
   
   ### Are there any user-facing changes?
   The above case that would error, now will no longer error.
   


-- 
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