Revert "Unregister shutdown hook when tracer is closed (jaegertracing…

…#679)"

This reverts commit 4bde959.

Signed-off-by: Tomas Hofman <thofman@redhat.com>

jaeger-core/src/main/java/io/jaegertracing/internal/JaegerTracer.java

@@ -79,7 +79,6 @@ public class JaegerTracer implements Tracer, Closeable {
   @ToString.Exclude private final BaggageSetter baggageSetter;
   @ToString.Exclude private final JaegerObjectFactory objectFactory;
   @ToString.Exclude private final int ipv4; // human readable representation is present within the tag map
-  @ToString.Exclude private Thread shutdownHook;
 
   protected JaegerTracer(JaegerTracer.Builder builder) {
     this.serviceName = builder.serviceName;
@@ -130,13 +129,12 @@ protected JaegerTracer(JaegerTracer.Builder builder) {
       log.info("No shutdown hook registered: Please call close() manually on application shutdown.");
     } else {
       // register this tracer with a shutdown hook, to flush the spans before the VM shuts down
-      shutdownHook = new Thread() {
+      Runtime.getRuntime().addShutdownHook(new Thread() {
         @Override
         public void run() {
           JaegerTracer.this.close();
         }
-      };
-      Runtime.getRuntime().addShutdownHook(shutdownHook);
+      });
     }
   }
 
@@ -226,9 +224,6 @@ public <T> JaegerSpanContext extract(Format<T> format, T carrier) {
   public void close() {
     reporter.close();
     sampler.close();
-    if (shutdownHook != null) {
-      Runtime.getRuntime().removeShutdownHook(shutdownHook);
-    }
   }
 
   public class SpanBuilder implements Tracer.SpanBuilder {