adriangb commented on PR #19505: URL: https://github.com/apache/datafusion/pull/19505#issuecomment-3696779084
> Looking at the old API, it seems like you could previously override per-expression level nullability and other properties Yes but I don't think that was ever used? I get the feeling (based on actual usage) that the idea was more to allow creating a context with or without a schema, with or without a config, etc. But that is better suited to this builder style API than a trait. I see that `trait SimplifyInfo` was originally added in #1774. @jimexist I know it's been years since then but do you have any thoughts on it? I don't see any discussion on why it is a trait, etc. -- 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]
