jayzhan211 commented on PR #14668: URL: https://github.com/apache/datafusion/pull/14668#issuecomment-2686798534
> Implementing a function that should be called also for null input requires declaring it, by wrapping relevant arguments in Option<...> If we can handle `Option<T>` as `null` and we can define whatever we want for null case then I think it is good enough to land as the first step. We need Option both in `inputs` and `return type` ``` fn concat(a: Option<T>, b: Option<T>) -> Result<Option<T>> ``` -- 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
