Merge pull request #18406 from christian-morin/jaeger-reporter-update

Update Jaeger reporter to use logger
quarkusio · Jul 12, 2021 · ac7eced · ac7eced
2 parents 1abc588 + 34a5492
commit ac7eced
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/.../jaeger/runtime/src/main/java/io/quarkus/jaeger/runtime/graal/Target_LoggingReporter.java b/.../jaeger/runtime/src/main/java/io/quarkus/jaeger/runtime/graal/Target_LoggingReporter.java
@@ -1,5 +1,7 @@
 package io.quarkus.jaeger.runtime.graal;
 
+import org.jboss.logging.Logger;
+
 import com.oracle.svm.core.annotate.Substitute;
 import com.oracle.svm.core.annotate.TargetClass;
 
@@ -10,6 +12,8 @@
 @TargetClass(className = "io.jaegertracing.internal.reporters.LoggingReporter")
 final public class Target_LoggingReporter implements Reporter {
 
+    private static final Logger LOG = Logger.getLogger(Target_LoggingReporter.class);
+
     @Substitute
     public Target_LoggingReporter() {
 
@@ -18,7 +22,7 @@ public Target_LoggingReporter() {
     @Substitute
     @Override
     public void report(JaegerSpan span) {
-        System.err.println("--- not logging: " + span);
+        LOG.infof("Span reported: %s", span);
     }
 
     @Substitute