asolimando commented on PR #21122: URL: https://github.com/apache/datafusion/pull/21122#issuecomment-4726842675
> I think this has to be reworked on top of #21815 - [#21122 (comment)](https://github.com/apache/datafusion/pull/21122#discussion_r3269082502) Yes, what I plan to tackle, in order, is the following: 1. #21815 2. #22958 3. [#21122 (comment)](https://github.com/apache/datafusion/pull/21122#discussion_r3269082502) a. Integrate `StatisticsRegistry` (from #21483) along the line suggested by @alamb (default but no-op if no providers are registered) 4. Extension to PhysicalExpr (new trait methods, default implementation returns None, non-breaking), along the line of https://github.com/apache/datafusion/pull/21122#issuecomment-4567251239 5. [#21122 (comment)](https://github.com/apache/datafusion/pull/21122#discussion_r3269082502) a. Rework `ExpressionAnalyzer` to make it the default, no-op if no providers are registered (again Andrew's suggestion) We could mark the current PR as draft, I will see if it makes more sense to open a new one (for 5.) or rebase this one not to lose previous context. WDYT? -- 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]
