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

   ## Which issue does this PR close?
   - Follow on to https://github.com/apache/datafusion/pull/23299
   - Part of https://github.com/apache/datafusion/issues/19241
   
   ## Rationale for this change
   
   We (mostly @geoffreyclaude ) are in the process of optimizing IN lists with 
specialized implementations for various different data types. 
   
   @kosiew  suggested in 
https://github.com/apache/datafusion/pull/23299#issuecomment-4876519158 that we 
add SQL level coverage for the IN list optimizations to both cover the dispatch 
logic as well as ensure everything works end to end. 
   
   > One small follow-up idea: it could be useful to add a public-path 
regression through InListExpr::try_new_from_array, or a SQL-level test, for 
Int8 and Int16 signed values.
   
   ## What changes are included in this PR?
   1. Add `in_list.slt` file to specifically target the IN lists
   2. Only add test coverage for integer types 
   
   ## Are these changes tested?
   
   Only tests 
   
   ## 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