Ted-Jiang commented on PR #7620: URL: https://github.com/apache/arrow-datafusion/pull/7620#issuecomment-1732498465
> I was thinking a little about the cache APIs here -- at the moment these are all related to `ListingTable` but are attached to the `SessionState`. I wonder if we should make a containing structure like `ListingTableCache` or something to contain them 🤔 . > > This is not a suggested change for this PR, just something I was thinking about. Thanks for your kindly review! Yes the cache method here are all in `ListingTable`, I will check refine it later. As this is cache , it need share the cache result in some place, i think there is only `SessionState` has the state can share in some level like(session), is there any other way to store the result ? -- 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]
