Alright, let's cache 'em

prdoyle · prdoyle · commit 536fd4ff2ba0 · 2025-03-14T13:13:07.000-04:00
diff --git a/libs/entitlement/src/main/java/org/elasticsearch/entitlement/runtime/policy/PolicyManager.java b/libs/entitlement/src/main/java/org/elasticsearch/entitlement/runtime/policy/PolicyManager.java
@@ -100,10 +100,12 @@ public <E extends Entitlement> Stream<E> getEntitlements(Class<E> entitlementCla
         }
 
         public Logger logger() {
-            return LogManager.getLogger(loggerName);
+            return MODULE_LOGGERS.computeIfAbsent(loggerName, LogManager::getLogger);
         }
     }
 
+    private static final ConcurrentHashMap<String, Logger> MODULE_LOGGERS = new ConcurrentHashMap<>();
+
     private FileAccessTree getDefaultFileAccess(String componentName, Path componentPath) {
         return FileAccessTree.of(componentName, UNKNOWN_COMPONENT_NAME, FilesEntitlement.EMPTY, pathLookup, componentPath, List.of());
     }

Original file line number	Diff line number	Diff line change
`@@ -100,10 +100,12 @@ public <E extends Entitlement> Stream<E> getEntitlements(Class<E> entitlementCla`
`100`	`100`	`}`
`101`	`101`
`102`	`102`	`public Logger logger() {`
`103`		`- return LogManager.getLogger(loggerName);`
	`103`	`+ return MODULE_LOGGERS.computeIfAbsent(loggerName, LogManager::getLogger);`
`104`	`104`	`}`
`105`	`105`	`}`
`106`	`106`
	`107`	`+ private static final ConcurrentHashMap<String, Logger> MODULE_LOGGERS = new ConcurrentHashMap<>();`
	`108`	`+`
`107`	`109`	`private FileAccessTree getDefaultFileAccess(String componentName, Path componentPath) {`
`108`	`110`	`return FileAccessTree.of(componentName, UNKNOWN_COMPONENT_NAME, FilesEntitlement.EMPTY, pathLookup, componentPath, List.of());`
`109`	`111`	`}`