seddonm1 commented on pull request #9038: URL: https://github.com/apache/arrow/pull/9038#issuecomment-752776874
> 1. Yes I think there should be a different/more efficient implementation that handles the "scalar" case, where the scalar in this case is the list with values. Agree. I can have a look as part of this PR. > 2. I believe the current behavior related to NULL is correct, just as field=NULL will also be false. I have tested in Postgres and it will return `NULL` if value is `NULL` unlike this implementation which returns `false`. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
