alamb commented on issue #19217: URL: https://github.com/apache/datafusion/issues/19217#issuecomment-3655552269
> Does it make sense to only create and store the `FileStatisticsCache` in the `CacheManager` and remove the creation of `FileStatisticsCache` from the `ListingTable` completely and soley reference it with `with_cache`? I think it's easier to reason about it. Yes, this would be my preference too It is also consistent with what the rest of the system does -- 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]
