jayzhan211 commented on PR #13683: URL: https://github.com/apache/datafusion/pull/13683#issuecomment-2525402657
> Interesting thing is that DuckDB says they based it on the PrestoDB behavior: [duckdb/duckdb#3065](https://github.com/duckdb/duckdb/pull/3065) > > But a quick look at the PrestoDB implementation suggests they use the behavior I'm suggesting, if one null is found the result is null: https://github.com/prestodb/presto/blob/b68af58583b8d58992b6ab0804d8d3618f7f402b/presto-main/src/main/java/com/facebook/presto/operator/scalar/ArrayContains.java#L52 > > So I wonder if DuckDBs null handling was actually intentionally or just an oversight by whoever initially implemented it Interesting, in this case I think changing the result is not a bad idea. -- 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]
