Use daemon threads for SentryExecutorService

CHANGELOG.md

@@ -8,6 +8,8 @@
 
 ### Fixes
 
+- Use daemon threads for SentryExecutorService ([#2747](https://github.com/getsentry/sentry-java/pull/2747))
+  - We started using `SentryExecutorService` in `6.19.0` which caused the application to hang on shutdown unless `Sentry.close()` was called. By using daemon threads we no longer block shutdown.
 - Don't warn R8 on missing `ComposeViewHierarchyExporter` class ([#2743](https://github.com/getsentry/sentry-java/pull/2743))
 
 ## 6.20.0

sentry/src/main/java/io/sentry/SentryExecutorService.java

@@ -4,6 +4,7 @@
 import java.util.concurrent.Executors;
 import java.util.concurrent.Future;
 import java.util.concurrent.ScheduledExecutorService;
+import java.util.concurrent.ThreadFactory;
 import java.util.concurrent.TimeUnit;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.TestOnly;
@@ -18,7 +19,7 @@ final class SentryExecutorService implements ISentryExecutorService {
   }
 
   SentryExecutorService() {
-    this(Executors.newSingleThreadScheduledExecutor());
+    this(Executors.newSingleThreadScheduledExecutor(new SentryExecutorServiceThreadFactory()));
   }
 
   @Override
@@ -59,4 +60,15 @@ public boolean isClosed() {
       return executorService.isShutdown();
     }
   }
+
+  private static final class SentryExecutorServiceThreadFactory implements ThreadFactory {
+    private int cnt;
+
+    @Override
+    public @NotNull Thread newThread(final @NotNull Runnable r) {
+      final Thread ret = new Thread(r, "SentryExecutorServiceThreadFactory-" + cnt++);
+      ret.setDaemon(true);
+      return ret;
+    }
+  }
 }