Don't log stack trace for expected exceptions (#9279)

open-telemetry · Aug 22, 2023 · 04436a1 · 04436a1
1 parent b8753fb
commit 04436a1
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/...1/library/src/main/java/io/opentelemetry/instrumentation/awssdk/v1_11/PluginImplUtil.java b/...1/library/src/main/java/io/opentelemetry/instrumentation/awssdk/v1_11/PluginImplUtil.java
@@ -45,10 +45,12 @@ static boolean isImplPresent(String implSimpleClassName) {
       // dependency is not on the class path).
       logger.log(
           Level.FINE,
-          e,
           () ->
-              implFullClassName
-                  + " not present. "
+              "Failed to load "
+                  + implFullClassName
+                  + " ("
+                  + e.getClass().getName()
+                  + "). "
                   + "Most likely, corresponding SDK component is either not on classpath or incompatible.");
       return false;
     }

diff --git a/....2/library/src/main/java/io/opentelemetry/instrumentation/awssdk/v2_2/PluginImplUtil.java b/....2/library/src/main/java/io/opentelemetry/instrumentation/awssdk/v2_2/PluginImplUtil.java
@@ -45,10 +45,12 @@ static boolean isImplPresent(String implSimpleClassName) {
       // dependency is not on the class path).
       logger.log(
           Level.FINE,
-          e,
           () ->
-              implFullClassName
-                  + " not present. "
+              "Failed to load "
+                  + implFullClassName
+                  + " ("
+                  + e.getClass().getName()
+                  + "). "
                   + "Most likely, corresponding SDK component is either not on classpath or incompatible.");
       return false;
     }