larry98 commented on PR #43256: URL: https://github.com/apache/arrow/pull/43256#issuecomment-2231801059
> I saw that in #43187 you mentioned: > > > Our current approach adds a new field to SetLookupOptions which allows the user to declare whether the value set is pre-sorted and deduplicated. > > I think this makes sense as it makes the `is_in` expression more self-explaining. Why is it not in this PR? (Instead, an extra argument `is_in_value_set_sorted` is added.) I was unsure if people would have liked that approach so I went with something simpler. But it sounds like we're OK with it, so I'll update this PR with the SetLookupOptions change. -- 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]
