Jefffrey commented on PR #19213: URL: https://github.com/apache/datafusion/pull/19213#issuecomment-3635519486
> I think changing here is too of a breaking change. WDYT @Jefffrey I think this makes sense to have, as I see the multiple PRs being raised to fix individual (Spark) functions and can see not having this as default makes it not very ergonomic for downstream functions; that said, we'd probably need some more investigation to make sure affected functions (i.e. all of them...) will play well with this change. Perhaps we can raise a separate issue for this entirely (e.g. "Change default nullability of UDFs") so can have more visibility on the discussion? -- 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]
