search

opentracing / opentracing-java

OpenTracing API for Java. 🛑 This library is DEPRECATED! https://github.com/opentracing/specification/issues/163
http://opentracing.io
Apache License 2.0
1.68k stars 344 forks source link

Do not make isRegistered() synchronized. #333

Closed carlosalberto closed 5 years ago

carlosalberto commented 5 years ago

This one does not really need to be synchronized. The field itself is volatile and that is enough.

Probably being too pedantic on this, but it feels this is the correct thing to do ;)

@yurishkuro @sjoerdtalsma

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 77.301% when pulling 5e23e0d1099083e1c37a92998b9ce3ef01d1bb71 on carlosalberto:global_tracer_registered_non_synch into 81762085f2e07ffe616c230ada8ad80a1532cc9c on opentracing:v0.32.0.