Run RemoteReporter's flush timer in daemon thread.

jaeger-core/src/main/java/com/uber/jaeger/reporters/RemoteReporter.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2016, Uber Technologies, Inc
+ * Copyright (c) 2017, Uber Technologies, Inc
  *
  * Permission is hereby granted, free of charge, to any person obtaining a copy
  * of this software and associated documentation files (the "Software"), to deal
@@ -60,11 +60,11 @@ public RemoteReporter(final Sender sender, int flushInterval, int maxQueueSize,
 
     // start a thread to append spans
     queueProcessor = new QueueProcessor();
-    queueProcessorThread = new Thread(queueProcessor);
+    queueProcessorThread = new Thread(queueProcessor, "jaeger.RemoteReporter-QueueProcessor");
     queueProcessorThread.setDaemon(true);
     queueProcessorThread.start();
 
-    flushTimer = new Timer();
+    flushTimer = new Timer("jaeger.RemoteReporter-FlushTimer", true /* isDaemon */);
     flushTimer.schedule(
         new TimerTask() {
           @Override

jaeger-core/src/test/java/com/uber/jaeger/reporters/RemoteReporterTest.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2016, Uber Technologies, Inc
+ * Copyright (c) 2017, Uber Technologies, Inc
  *
  * Permission is hereby granted, free of charge, to any person obtaining a copy
  * of this software and associated documentation files (the "Software"), to deal
@@ -34,6 +34,8 @@
 import java.util.List;
 
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
 
 public class RemoteReporterTest {
   private Reporter reporter;
@@ -89,4 +91,17 @@ public void testRemoteReporterFlushesOnClose() throws Exception {
     assertEquals(
         100L, metricsReporter.counters.get("jaeger.traces.sampled=y.state=started").longValue());
   }
+
+  @Test
+  public void testRemoteReporterFlushTimerThread() throws Exception {
+    int flushTimerThreadCount = 0;
+    for (Thread thread : Thread.getAllStackTraces().keySet()) {
+      if (!thread.getName().equals("jaeger.RemoteReporter-FlushTimer")) {
+        continue;
+      }
+      ++flushTimerThreadCount;
+      assertTrue(thread.isDaemon());
+    }
+    assertFalse(flushTimerThreadCount == 0);
+  }
 }