andygrove opened a new issue, #2133:
URL: https://github.com/apache/datafusion-comet/issues/2133

   ### Describe the bug
   
   In `QueryPlanSerde`, there is specific handling of some string expressions 
for `FilterExec`.
   
   ```scala
             // Some native expressions do not support operating on 
dictionary-encoded arrays, so
             // wrap the child in a CopyExec to unpack dictionaries first.
             def wrapChildInCopyExec(condition: Expression): Boolean = {
               condition.exists(expr => {
                 expr.isInstanceOf[StartsWith] || expr.isInstanceOf[EndsWith] 
|| expr
                   .isInstanceOf[Contains]
               })
             }
   ```
   
   However, it is possible for these expressions to exist in other operators, 
such as `ProjectExec`, and it does not look like we add `CopyExec` in those 
cases.
   
   ### Steps to reproduce
   
   _No response_
   
   ### Expected behavior
   
   _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: github-unsubscr...@datafusion.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to