youngyjd opened a new pull request, #8954:
URL: https://github.com/apache/gravitino/pull/8954

   ### What changes were proposed in this pull request?
   
   Refactored the FileSystem caching strategy in BaseGVFSOperations to cache by 
storage backend instead of by fileset.
   
   Main Changes:
   - New cache key: Changed from Pair<NameIdentifier, String> (fileset 
identifier + location name) to FileSystemCacheKey(scheme, authority, 
UserGroupInformation)
   - New FileSystemCacheKey inner class: Static inner class with proper 
equals()/hashCode() implementation based on filesystem scheme, authority, and 
user
   - Restructured caching logic: Moved FileSystem caching from 
getActualFileSystemByLocationName() to getActualFileSystemByPath(), separating 
fileset metadata resolution from filesystem instantiation
   
   ### Why are the changes needed?
   
   More granular caching at the actual filesystem level rather than virtual 
fileset level. Now getActualFileSystemByPath() load FS from cache.
   
   Fix: #8953 
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. This is an internal caching optimization. The external API and behavior 
remain unchanged.
   
   ### How was this patch tested?
   
   Unit tests
   


-- 
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]

Reply via email to