Kontinuation opened a new pull request, #1845:
URL: https://github.com/apache/datafusion-comet/pull/1845

   ## Which issue does this PR close?
   
   Closes #1844 and probably #1823.
   
   ## Rationale for this change
   
   Loading data with custom schema may produce rows containing null struct 
fields. Shuffling such null struct fields throws exceptions mentioned in the 
linked issues.
   
   ## What changes are included in this PR?
   
   Fixes appending null rows to `StructBuilder` by making children field 
builders always in sync with the parent struct builder.
   
   ## How are these changes tested?
   
   1. Add unit tests for `append_field` in native code
   2. Add a Scala unit test modified from the minimal repro of the bug
   


-- 
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...@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to