alamb commented on PR #15119: URL: https://github.com/apache/datafusion/pull/15119#issuecomment-2713758011
> @alamb I wonder if we can remove the `register_expr_planners` and `expr_planners` from the `FunctionRegistry` trait. I have checked the codebase and they're only used by a test. And it's kind of weird as these methods don’t seem to align with the core responsibility of the trait. It seem reasonable to me -- maybe we could make a PR to deprecate them first ? -- 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
