neilconway opened a new pull request, #25544:
URL: https://github.com/apache/datafusion/pull/25544

   ## Which issue does this PR close?
   
   - N/A
   
   ## Rationale for this change
   
   The Spark-compatible `shuffle` implementation unnecessarily copied the first 
child element as a placeholder for each null list. This is wrong for typed-NULL 
lists with no children, and was redundant work anyway: NULL lists can be 
adequately represented by a repeated offset entry, along with setting the 
appropriate bit in the validity buffer.
   
   ## What changes are included in this PR?
   
   * Fix bug
   * Add unit test
   
   ## What is the testing strategy for this PR?
   
   Existing tests pass, new test added.
   
   ## Are there any user-facing changes?
   
   No.
   


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