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

   Cleanup a few places where the code returned a null array but it would be a 
bit cleaner and faster to return a typed scalar null instead.
   
   ## Which issue does this PR close?
   
   Does not close an issue; this cleanup was mentioned in the code review for 
#20361
   
   ## Rationale for this change
   
   Returning a typed scalar null should be preferred to returning a null array: 
it still has type information, and avoids materializing an all-null array. The 
downstream consumer can always materialize the equivalent array if they want to.
   
   ## What changes are included in this PR?
   
   Cleanup five instances of this pattern.
   
   ## Are these changes tested?
   
   Yes. No new test cases possible/warranted.
   
   ## 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