wip

Signed-off-by: Attila Mészáros <[email protected]>

Author: csviri

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/informer/TemporaryResourceCache.java

@@ -105,22 +105,23 @@ public boolean onAddOrUpdateEvent(T resource) {
 
   private boolean onEvent(T resource, boolean unknownState) {
     var resourceId = ResourceID.fromResource(resource);
-    // todo move to trace these log messages
-    log.debug(
-        "Processing event for resource id: {} version: {} ",
-        resourceId,
-        resource.getMetadata().getResourceVersion());
+    if (log.isDebugEnabled()) {
+      log.debug(
+          "Processing event for resource id: {} version: {} ",
+          resourceId,
+          resource.getMetadata().getResourceVersion());
+    }
     ReentrantLock lock = activelyModifying.get(resourceId);
     if (lock != null) {
-      log.debug("Lock for event filtering resource id: {}", resourceId);
+      log.trace("Lock for event filtering resource id: {}", resourceId);
       // note that this is a special case of lock striping; event handling happens
       // always on the same thread of the informer we lock only if the update is happening
       // for the same resource (not any resource), and if the event comes from the current update
       // this should be locked for a very short time, since that update request already send at this
       // point.
       lock.lock(); // wait for the modification to finish
       lock.unlock(); // simply unlock as the event is guaranteed after the modification
-      log.debug("Unlock for event resource id: {}", resourceId);
+      log.trace("Unlock for event resource id: {}", resourceId);
     }
     boolean[] filter = new boolean[1];
     synchronized (this) {