Remove trace logger in Transformer on default path when a class doesn't need to be transformed. (#136152)

This trace log badly interfered with entitlements in tests. When a class (eg Regex, used by the test mock logger in the case of JULBridgeTests) is loaded as a result of logging, tracing logging that in the Transformer causes a recursive call to the logging appender.

Fixes #132280

Author: mosche

libs/entitlement/src/main/java/org/elasticsearch/entitlement/instrumentation/Transformer.java

@@ -62,9 +62,7 @@ public byte[] transform(
                 // effectively the same as returning null anyways, so we instead log it here completely
                 return null;
             }
-        } else {
-            logger.trace("Not transforming " + className);
-            return null;
         }
+        return null;
     }
 }

muted-tests.yml

@@ -369,9 +369,6 @@ tests:
 - class: org.elasticsearch.xpack.sql.qa.mixed_node.SqlCompatIT
   method: testNullsOrderWithMissingOrderSupportQueryingNewNode
   issue: https://github.com/elastic/elasticsearch/issues/132249
-- class: org.elasticsearch.common.logging.JULBridgeTests
-  method: testThrowable
-  issue: https://github.com/elastic/elasticsearch/issues/132280
 - class: org.elasticsearch.xpack.ml.integration.AutodetectMemoryLimitIT
   method: testManyDistinctOverFields
   issue: https://github.com/elastic/elasticsearch/issues/132308