asolimando commented on PR #21815: URL: https://github.com/apache/datafusion/pull/21815#issuecomment-4523382389
@alamb @xudong963, there was another conflict so I had to force push again, since there were a couple of artifact from the previous rebase, I did a new one and reworked the commit by "theme", as it was getting hard to manage. Current commits: - `6055a43`: deprecate `partition_statistics`, add `statistics_with_args` with `StatisticsArgs` + shared `StatsCache`, lazy computation, fix `SampleExec` - `396efd4`: benchmarks for 4 plan shapes: coalesce chain, cross-join tree, filter chain, mixed chain (this one is new, I added it when switched to pure lazy stats without pre-computation as I wanted to make sure all was good in that case too) - `6916e14`: upgrade guide (55.0.0.md) - `7eb0484`: rename statistics_context -> statistics, use `impl AsRef` (per @alamb) - `d25e1ad`: remove compute_statistics free function, callers manage `StatisticsArgs` directly (per @alamb) -- 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]
