From af1d4ad79cf109788ba42e10790387097cbd3493 Mon Sep 17 00:00:00 2001
From: Martijn van Groningen <martijn.v.groningen@gmail.com>
Date: Wed, 13 May 2015 10:44:15 +0200
Subject: [PATCH] Use ThreadPool.Names.SAME executor in stats and service apis.

Original commit: elastic/x-pack-elasticsearch@03c52668e0c5a7e02bd8059bb35efaf7baf54cfa
---
 .../actions/service/TransportWatcherServiceAction.java         | 3 ++-
 .../transport/actions/stats/TransportWatcherStatsAction.java   | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/elasticsearch/watcher/transport/actions/service/TransportWatcherServiceAction.java b/src/main/java/org/elasticsearch/watcher/transport/actions/service/TransportWatcherServiceAction.java
index c5911aeec8a..b01c01deba4 100644
--- a/src/main/java/org/elasticsearch/watcher/transport/actions/service/TransportWatcherServiceAction.java
+++ b/src/main/java/org/elasticsearch/watcher/transport/actions/service/TransportWatcherServiceAction.java
@@ -35,7 +35,8 @@ public class TransportWatcherServiceAction extends WatcherTransportAction<Watche
 
     @Override
     protected String executor() {
-        return ThreadPool.Names.MANAGEMENT;
+        // We should always be able to stop or restart the watcher service, even if the a TP is exhausted, so don't fork into another thread:
+        return ThreadPool.Names.SAME;
     }
 
     @Override
diff --git a/src/main/java/org/elasticsearch/watcher/transport/actions/stats/TransportWatcherStatsAction.java b/src/main/java/org/elasticsearch/watcher/transport/actions/stats/TransportWatcherStatsAction.java
index 1d9ffdf14b3..206a47dfb0a 100644
--- a/src/main/java/org/elasticsearch/watcher/transport/actions/stats/TransportWatcherStatsAction.java
+++ b/src/main/java/org/elasticsearch/watcher/transport/actions/stats/TransportWatcherStatsAction.java
@@ -42,7 +42,8 @@ public class TransportWatcherStatsAction extends WatcherTransportAction<WatcherS
 
     @Override
     protected String executor() {
-        return ThreadPool.Names.MANAGEMENT;
+        // cheap operation, no need to fork into another thread
+        return ThreadPool.Names.SAME;
     }
 
     @Override