ianmcook opened a new issue, #37859:
URL: https://github.com/apache/arrow/issues/37859

   ### Describe the bug, including details regarding any error messages, 
version, and platform.
   
   There is a TODO here:
   
https://github.com/apache/arrow/blob/3c013da56fd55122072ffaca3e23afb12c290075/cpp/src/arrow/engine/substrait/expression_internal.cc#L1284
   
   Now that https://github.com/apache/arrow/pull/36739 is merged, can we use 
the new [`INCONCLUSIVE` null matching 
option](https://arrow.apache.org/docs/dev/cpp/api/compute.html#_CPPv4N5arrow7compute16SetLookupOptions20NullMatchingBehavior12INCONCLUSIVEE)
 to translate Substrait [`SingularOrList` 
expressions](https://github.com/substrait-io/substrait/blob/651829aabde4c9562b291f8bb444aa0303c482cd/proto/substrait/algebra.proto#L1017)
 to/from Acero `is_in` function options with consistent null handling behavior?
   
   Can we also use this new option to ensure that [Substrait's `index_in` 
function](https://github.com/substrait-io/substrait/blob/651829aabde4c9562b291f8bb444aa0303c482cd/extensions/functions_set.yaml#L5)
 is translated to/from Acero's `index_in` function with the correct options set 
to give consistent behavior?
   
   Some other related issues:
   - https://github.com/apache/arrow/issues/36535
   - https://github.com/substrait-io/substrait/issues/517
   
   
   ### Component(s)
   
   C++


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

Reply via email to