Re: [PR] [#6566] improvement(core): Add the cache mechanism for metalake and use cache to load `in-use` information. [gravitino]

[via GitHub]

jerryshao commented on code in PR #6569:
URL: https://github.com/apache/gravitino/pull/6569#discussion_r1977300386


##########
core/src/main/java/org/apache/gravitino/catalog/CatalogManager.java:
##########
@@ -848,12 +848,17 @@ private static boolean catalogInUse(EntityStore store, 
NameIdentifier ident)
 
   private static boolean getCatalogInUseValue(EntityStore store, 
NameIdentifier catalogIdent) {
     try {
-      CatalogEntity catalogEntity =
-          store.get(catalogIdent, EntityType.CATALOG, CatalogEntity.class);
+      CatalogWrapper wrapper = catalogCache.getIfPresent(catalogIdent);
+      CatalogEntity catalogEntity;
+      if (wrapper != null) {
+        catalogEntity = wrapper.catalog.entity();
+      } else {
+        catalogEntity = store.get(catalogIdent, EntityType.CATALOG, 
CatalogEntity.class);
+      }
+
       return (boolean)
           BASIC_CATALOG_PROPERTIES_METADATA.getOrDefault(
               catalogEntity.getProperties(), PROPERTY_IN_USE);
-

Review Comment:
   This is a unrelated change, please revert 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: commits-unsubscr...@gravitino.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org