abhita commented on issue #18405: URL: https://github.com/apache/datafusion/issues/18405#issuecomment-3482453102
@nuno-faria Right. I guess I might have added something wrong here instead of `FileMetadataCache`. Something like a `CustomMetadataCache` holding Cache and an eviction Policy. Let me correct the Issue Description. I did not get the statement below. Trying to understand what would be the downsides for doing so > Custom implementations (e.g., storing the cache in an in-memory database) would not work, as they would be forced to use the DashMap. Just thinking out loud, if we think of modifying `DefaultFilesMetadataCache` to receive some eviction policy, doesn't having cache in-memory look out to be a cleaner way to decouple storage and Eviction-strategies? Given that any Eviction Policy would require it's own data-structure to manipulate data around and for the use-case of concurrent access -> `DashMap`? -- 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]
