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

   ## Which issue does this PR close?
   
   - None, followup to #18152
   
   ## Rationale for this change
   
   `case_when_with_expr` has a code path to handle `null` values early on in 
the evaluation. It determines the presence of nulls using 
`value_array.null_count() > 0`. For nested arrays this may not be correct. 
`logical_null_count()` should be used instead.
   
   ## What changes are included in this PR?
   
   Check `logical_null_count` instead of `null_count`.
   
   ## Are these changes tested?
   
   Additional tests added to ensure the nested array case is tested. The test 
already passed before this change, but was a bit less efficient since the null 
values were tested for equality against each possible when value.
   
   ## 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