Disable generic metastore cache for Glue with internal caching

Author: dain

plugin/trino-hive/src/main/java/io/trino/plugin/hive/metastore/HiveMetastoreFactory.java

@@ -21,6 +21,11 @@
 
 public interface HiveMetastoreFactory
 {
+    default boolean hasBuiltInCaching()
+    {
+        return false;
+    }
+
     boolean isImpersonationEnabled();
 
     /**

plugin/trino-hive/src/main/java/io/trino/plugin/hive/metastore/cache/SharedHiveMetastoreCache.java

@@ -119,7 +119,7 @@ public boolean isEnabled()
 
     public HiveMetastoreFactory createCachingHiveMetastoreFactory(HiveMetastoreFactory metastoreFactory)
     {
-        if (!enabled) {
+        if (!enabled || metastoreFactory.hasBuiltInCaching()) {
             return metastoreFactory;
         }
 

plugin/trino-hive/src/main/java/io/trino/plugin/hive/metastore/glue/GlueHiveMetastoreFactory.java

@@ -34,6 +34,12 @@ public GlueHiveMetastoreFactory(GlueHiveMetastore metastore, Tracer tracer)
         this.metastore = new TracingHiveMetastore(tracer, metastore);
     }
 
+    @Override
+    public boolean hasBuiltInCaching()
+    {
+        return true;
+    }
+
     @Override
     public boolean isImpersonationEnabled()
     {