Merge pull request #48349 from gsmet/opentelemetry-fix

Use JBoss Logging in LogsExporterCDIProvider

Author: brunobat

extensions/opentelemetry/runtime/src/main/java/io/quarkus/opentelemetry/runtime/logs/spi/LogsExporterCDIProvider.java

@@ -2,12 +2,12 @@
 
 import static io.quarkus.opentelemetry.runtime.config.build.ExporterType.Constants.CDI_VALUE;
 
-import java.util.logging.Logger;
-
 import jakarta.enterprise.inject.Any;
 import jakarta.enterprise.inject.Instance;
 import jakarta.enterprise.inject.spi.CDI;
 
+import org.jboss.logging.Logger;
+
 import io.opentelemetry.sdk.autoconfigure.spi.ConfigProperties;
 import io.opentelemetry.sdk.autoconfigure.spi.logs.ConfigurableLogRecordExporterProvider;
 import io.opentelemetry.sdk.logs.export.LogRecordExporter;
@@ -20,14 +20,16 @@ public class LogsExporterCDIProvider implements ConfigurableLogRecordExporterPro
     @Override
     public LogRecordExporter createExporter(ConfigProperties configProperties) {
         Instance<LogRecordExporter> exporters = CDI.current().select(LogRecordExporter.class, Any.Literal.INSTANCE);
-        log.fine("available exporters: " + exporters.stream()
-                .map(e -> e.getClass().getName())
-                .reduce((a, b) -> a + ", " + b)
-                .orElse("none"));
+        if (log.isDebugEnabled()) {
+            log.debug("available exporters: " + exporters.stream()
+                    .map(e -> e.getClass().getName())
+                    .reduce((a, b) -> a + ", " + b)
+                    .orElse("none"));
+        }
         if (exporters.isUnsatisfied()) {
             return NoopLogRecordExporter.INSTANCE;
         } else {
-            log.fine("using exporter: " + exporters.get().getClass().getName());
+            log.debugf("using exporter: %s", exporters.get().getClass().getName());
             return exporters.get();
         }
     }