gabotechs commented on code in PR #25969: URL: https://github.com/apache/datafusion/pull/25969#discussion_r4184487678
########## docs/source/library-user-guide/upgrading/56.0.0.md: ########## @@ -428,6 +428,28 @@ part of `StatisticsRegistry::default_with_builtin_providers()`. - Remove `DefaultStatisticsProvider` from any custom provider chain; register no terminal provider instead (the walk falls back on its own). +### `StatisticsRegistry::default_with_builtin_providers` is deprecated + +`StatisticsRegistry::default_with_builtin_providers()` is deprecated in favor of +`StatisticsRegistry::with_experimental_providers()`. Most of the providers it Review Comment: > I was considering a two-phased approach, as the Join and Filter providers had improvements we want to keep 👍 My impression is that we could fold those into the built-in statistics of `FilterExec` and `HashJoinExec`, WDYT? (for another PR of course) -- 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]
