From: Sébastien Lesaint <sebastien.lesaint@sonarsource.com>
Date: Thu, 15 Oct 2015 15:33:17 +0000 (+0200)
Subject: add name to thread used by NotificationService
X-Git-Tag: 5.2-RC3~18
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=1fb6d54e82deefb1e356e1974275c8816a676a7f;p=sonarqube.git

add name to thread used by NotificationService

threads created by SQ should always be named to easy investigations
---

diff --git a/server/sonar-server/src/main/java/org/sonar/server/notification/NotificationService.java b/server/sonar-server/src/main/java/org/sonar/server/notification/NotificationService.java
index 185a54e54c4..66af432960c 100644
--- a/server/sonar-server/src/main/java/org/sonar/server/notification/NotificationService.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/notification/NotificationService.java
@@ -24,6 +24,7 @@ import com.google.common.collect.HashMultimap;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.Multimap;
 import com.google.common.collect.SetMultimap;
+import com.google.common.util.concurrent.ThreadFactoryBuilder;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.List;
@@ -60,6 +61,7 @@ import org.sonar.db.DbClient;
 })
 @ServerSide
 public class NotificationService implements Startable {
+  private static final String THREAD_NAME_PREFIX = "sq-notification-service-";
 
   private static final Logger LOG = Loggers.get(NotificationService.class);
 
@@ -93,7 +95,12 @@ public class NotificationService implements Startable {
 
   @Override
   public void start() {
-    executorService = Executors.newSingleThreadScheduledExecutor();
+    executorService =
+      Executors.newSingleThreadScheduledExecutor(
+        new ThreadFactoryBuilder()
+          .setNameFormat(THREAD_NAME_PREFIX + "%d")
+          .setPriority(Thread.MIN_PRIORITY)
+          .build());
     executorService.scheduleWithFixedDelay(new Runnable() {
       @Override
       public void run() {