pepijnve opened a new issue, #18499:
URL: https://github.com/apache/datafusion/issues/18499

   ### Is your feature request related to a problem or challenge?
   
   Recent changes in the implementation of `CaseExpr` have replaced the usages 
of `PhysicalExpr::evaluate_selection` with custom filtering and selective 
evaluation logic. This was done to avoid overhead introduced by the generic 
approach `evaluate_selection` takes where it first filters the incoming record 
batch using the selection vector, calls plain `evaluate` with the filtered 
record batch, and then expands the result back to the length of the original 
record batch.
   
   Because of all these changes, `PhysicalExpr::evaluate_selection` is no 
longer used at all in DataFusion itself. Since it's no longer used, and its use 
can be a cause of performance overhead, it might be better to deprecate (or 
even remove) it.
   
   ### Describe the solution you'd like
   
   _No response_
   
   ### Describe alternatives you've considered
   
   _No response_
   
   ### Additional context
   
   _No response_


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