alchemist51 commented on issue #18362: URL: https://github.com/apache/datafusion/issues/18362#issuecomment-3517942739
While working on it, we found another issue with MetadataCache object being maintained outside of the Datafusion. ``` fn remove(&mut self, k: &K) -> Option<V>; ``` This function in CacheAccessor needs a `mut` type of self hence we always need to store the MetadataCache is something like this to get the mut type: ``` Arc<Mutex<DefaultMetadataCache> ``` Now the current interfaces does not accept the Arc<Mutex<dyn FileMetaDataCache>> which makes it a bit tricky. I think we should provide another API in cacheAccessor to remove without mut. @alamb @nuno-faria do you have any thought on how to tackle this in general or why the remove is kepy mut while every other API doesn't need it? -- 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]
