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

   ## Which issue does this PR close?
   
   - Closes #25223.
   
   ## Rationale for this change
   
   Arrow allows elements of a `ListArray` to be logically NULL (according to 
the validity buffer), but to have non-empty offset ranges. Such elements should 
be considered NULL.
   
   `array_repeat` only considered the offsets and ignored the validity buffer, 
which resulted in mishandling such NULL values.
   
   ## What changes are included in this PR?
   
   * Fix `array_repeat` bug
   * Add unit test
   * Add SLT
   
   ## What is the testing strategy for this PR?
   
   Existing tests pass; new tests added. We add both a unit test and an SLT 
because the SLT depends on `nullif` creating such a NULL value, which might 
change in the future.
   
   ## Are there any user-facing changes?
   
   Yes, bug fixed.
   


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