Merge pull request #1245 from microsoft/trask/fix-shutdown-issue

Fix shutdown issue due to non-daemon thread
microsoft · May 26, 2020 · bd323f0 · bd323f0
2 parents 67ff6be + 8ad7d5d
commit bd323f0
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/core/src/main/java/com/microsoft/applicationinsights/internal/quickpulse/QuickPulse.java b/core/src/main/java/com/microsoft/applicationinsights/internal/quickpulse/QuickPulse.java
@@ -30,6 +30,7 @@
 import com.google.common.base.Preconditions;
 import com.microsoft.applicationinsights.internal.util.DeviceInfo;
 import com.microsoft.applicationinsights.internal.util.LocalStringsUtils;
+import com.microsoft.applicationinsights.internal.util.ThreadPoolUtils;
 import org.apache.http.client.methods.HttpPost;
 
 import com.microsoft.applicationinsights.TelemetryConfiguration;
@@ -60,7 +61,7 @@ public void initialize() {
     public void initialize(final TelemetryConfiguration configuration) {
         Preconditions.checkNotNull(configuration);
         final CountDownLatch latch = new CountDownLatch(1);
-        Executors.newSingleThreadExecutor().execute(new Runnable() {
+        Executors.newSingleThreadExecutor(ThreadPoolUtils.createDaemonThreadFactory(QuickPulse.class)).execute(new Runnable() {
             @Override
             public void run() {
                 initializeSync(latch, configuration);