andygrove commented on PR #2251: URL: https://github.com/apache/datafusion-comet/pull/2251#issuecomment-3234967366
> lgtm. I only wonder if this will end up making a bunch of S3 calls every time ScanRule is called (but perhaps for the time being this is ok until we address this better). That is a great point. Let me see if there is a way to cache this so we don't keep re-validating the same config every time the rule runs. -- 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
