jkosh44 commented on PR #14289: URL: https://github.com/apache/datafusion/pull/14289#issuecomment-2613713471
> An alternative approach would be to add a function like the following to the `ScalarUDFImpl` trait > >```Rust > /// Returns true if the function should return NULL when any of the arguments are NULL, false > /// otherwise. > fn propagates_nulls(&self) -> bool; >``` > > Then we could handle this for all functions in the same place at some higher level. Maybe somewhere like `make_scalar_function`. Though, that would skip some of the error checking that happens inside of the function implementation which wouldn't be great. -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
