]
Juraci Paixão Kröhling commented on WFLY-10904:
-----------------------------------------------
That's a good point. The reason it happens is because we allow applications to provide
their own tracers, which the MP-OT subsystem logs as DEBUG level, whereas the other
message is at the Jaeger level.
If we remove the debug level message, we'll be potentially be removing a valuable
source of information to users who are deploying a custom tracer, which might be also a
Jaeger tracer, with a different configuration.
IMO, we can live with that, as DEBUG does imply (to me) that the same info might be
printed out at different layers.
Lower logging level of messages for OpenTracing
-----------------------------------------------
Key: WFLY-10904
URL:
https://issues.jboss.org/browse/WFLY-10904
Project: WildFly
Issue Type: Bug
Components: MP OpenTracing
Affects Versions: 14.0.0.Beta2
Reporter: Jan Stourac
Assignee: Juraci Paixão Kröhling
Priority: Blocker
Fix For: 14.0.0.CR1
Current logging for OpenTracing is quite verbose logging long messages in INFO level,
e.g.:
{code}
17:38:45,359 INFO [io.jaegertracing.internal.senders.SenderResolver] (ServerService
Thread Pool -- 28) Using sender
UdpSender(udpTransport=ThriftUdpTransport(socket=java.net.DatagramSocket@3e9a2205,
receiveBuf=null, receiveOffSet=-1, receiveLength=0))
17:38:45,395 INFO [io.jaegertracing.Configuration] (ServerService Thread Pool -- 28)
Initialized tracer=JaegerTracer(version=Java-0.30.4, serviceName=deployment.war,
reporter=RemoteReporter(queueProcessor=RemoteReporter.QueueProcessor(open=true),
sender=UdpSender(udpTransport=ThriftUdpTransport(socket=java.net.DatagramSocket@3e9a2205,
receiveBuf=null, receiveOffSet=-1, receiveLength=0)), closeEnqueueTimeout=1000),
sampler=RemoteControlledSampler(maxOperations=2000, serviceName=deployment.war,
manager=HttpSamplingManager(gson={serializeNulls:false,factories:[Factory[typeHierarchy=com.google.gson.JsonElement,adapter=com.google.gson.internal.bind.TypeAdapters$29@5795d61d],
com.google.gson.internal.bind.ObjectTypeAdapter$1@7286f924,
com.google.gson.internal.Excluder@10bd7673,
Factory[type=java.lang.String,adapter=com.google.gson.internal.bind.TypeAdapters$16@56577ab9],
Factory[type=java.lang.Integer+int,adapter=com.google.gson.internal.bind.TypeAdapters$7@2f0b6b50],
Factory[type=java.lang.Boolean+boolean,adapter=com.google.gson.internal.bind.TypeAdapters$3@6a01f84c],
Factory[type=java.lang.Byte+byte,adapter=com.google.gson.internal.bind.TypeAdapters$5@11f41591],
Factory[type=java.lang.Short+short,adapter=com.google.gson.internal.bind.TypeAdapters$6@7c9b37a4],
Factory[type=java.lang.Long+long,adapter=com.google.gson.internal.bind.TypeAdapters$11@8be828c],
Factory[type=java.lang.Double+double,adapter=com.google.gson.Gson$1@6d18d84f],
Factory[type=java.lang.Float+float,adapter=com.google.gson.Gson$2@5c7110e0],
Factory[type=java.lang.Number,adapter=com.google.gson.internal.bind.TypeAdapters$14@79f22add],
Factory[type=java.util.concurrent.atomic.AtomicInteger,adapter=com.google.gson.TypeAdapter$1@5a47d8d],
Factory[type=java.util.concurrent.atomic.AtomicBoolean,adapter=com.google.gson.TypeAdapter$1@2ed507d6],
Factory[type=java.util.concurrent.atomic.AtomicLong,adapter=com.google.gson.TypeAdapter$1@7af7f4fc],
Factory[type=java.util.concurrent.atomic.AtomicLongArray,adapter=com.google.gson.TypeAdapter$1@123b9dbd],
Factory[type=java.util.concurrent.atomic.AtomicIntegerArray,adapter=com.google.gson.TypeAdapter$1@3455c262],
Factory[type=java.lang.Character+char,adapter=com.google.gson.internal.bind.TypeAdapters$15@32b89889],
Factory[type=java.lang.StringBuilder,adapter=com.google.gson.internal.bind.TypeAdapters$19@772dda36],
Factory[type=java.lang.StringBuffer,adapter=com.google.gson.internal.bind.TypeAdapters$20@3935671b],
Factory[type=java.math.BigDecimal,adapter=com.google.gson.internal.bind.TypeAdapters$17@23e9f48d],
Factory[type=java.math.BigInteger,adapter=com.google.gson.internal.bind.TypeAdapters$18@55634ca9],
Factory[type=java.net.URL,adapter=com.google.gson.internal.bind.TypeAdapters$21@4c06d059],
Factory[type=java.net.URI,adapter=com.google.gson.internal.bind.TypeAdapters$22@5e997b7d],
Factory[type=java.util.UUID,adapter=com.google.gson.internal.bind.TypeAdapters$24@3238a1bf],
Factory[type=java.util.Currency,adapter=com.google.gson.TypeAdapter$1@7d71b555],
Factory[type=java.util.Locale,adapter=com.google.gson.internal.bind.TypeAdapters$28@70f36ba],
Factory[typeHierarchy=java.net.InetAddress,adapter=com.google.gson.internal.bind.TypeAdapters$23@12d8e7b5],
Factory[type=java.util.BitSet,adapter=com.google.gson.TypeAdapter$1@659a9908],
com.google.gson.internal.bind.DateTypeAdapter$1@153143f3,
Factory[type=java.util.Calendar+java.util.GregorianCalendar,adapter=com.google.gson.internal.bind.TypeAdapters$27@6bf50d09],
com.google.gson.internal.bind.TimeTypeAdapter$1@3e45ba8d,
com.google.gson.internal.bind.SqlDateTypeAdapter$1@f262694,
com.google.gson.internal.bind.TypeAdapters$26@1017c258,
com.google.gson.internal.bind.ArrayTypeAdapter$1@53c90816,
Factory[type=java.lang.Class,adapter=com.google.gson.TypeAdapter$1@40cb5130],
com.google.gson.internal.bind.CollectionTypeAdapterFactory@7e67cd3b,
com.google.gson.internal.bind.MapTypeAdapterFactory@504cd5ad,
com.google.gson.internal.bind.JsonAdapterAnnotationTypeAdapterFactory@4b5efe0a,
com.google.gson.internal.bind.TypeAdapters$30@51066480,
com.google.gson.internal.bind.ReflectiveTypeAdapterFactory@11c8758e],instanceCreators:{}},
hostPort=localhost:5778), metrics=io.jaegertracing.internal.metrics.Metrics@279f3ba7,
sampler=ProbabilisticSampler(positiveSamplingBoundary=9223372036854776,
negativeSamplingBoundary=-9223372036854776, samplingRate=0.001,
tags={sampler.type=probabilistic, sampler.param=0.001})), ipv4=2130706433,
tags={hostname=rhel7-large-74052, jaeger.version=Java-0.30.4, ip=127.0.0.1},
zipkinSharedRpcSpan=false,
baggageSetter=io.jaegertracing.internal.baggage.BaggageSetter@1b8d3ff0,
expandExceptionLogs=false)
{code}
I think these messages are not useful on INFO level and should be lowered to either DEBUG
or TRACE level.