xloya commented on code in PR #4320:
URL: https://github.com/apache/gravitino/pull/4320#discussion_r1768310821
##########
clients/filesystem-hadoop3/src/main/java/org/apache/gravitino/filesystem/hadoop/GravitinoVirtualFileSystem.java:
##########
@@ -65,8 +69,10 @@ public class GravitinoVirtualFileSystem extends FileSystem {
private URI uri;
private GravitinoClient client;
private String metalakeName;
- private Cache<NameIdentifier, Pair<Fileset, FileSystem>> filesetCache;
- private ScheduledThreadPoolExecutor scheduler;
+ private Cache<NameIdentifier, FilesetCatalog> catalogCache;
Review Comment:
Currently, to obtain the actual file location, we need to `loadCatalog`
first, and then call `asFilesetCatalog().getFileLocation()`. If the Catalog is
not cached here, two RPCs are required for each file operation. Considering
that changes to the Catalog are not very frequent, these requests may be
unnecessary.
--
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]
