honeymoose/OpenSearch
1
0
Fork 0
mirror of https://github.com/honeymoose/OpenSearch.git synced 2025-03-04 01:49:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

If starting watcher fails, watcher should set itself back to the stopped state.

Browse Source 
At the moment if the starting fails watcher will hang in the `starting` phase. This is bad because any subsequent start will be ignored, because the the watcher state isn't `stopped`.

Original commit: elastic/x-pack-elasticsearch@5cbc1d2a13
This commit is contained in:
Martijn van Groningen 2015-06-24 09:39:46 +02:00
parent b8e80773d9
commit 67fdad6357
4 changed files with 37 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/main/java/org/elasticsearch/watcher/WatcherService.java
View File

@ -52,15 +52,20 @@ public class WatcherService extends AbstractComponent {
public void start(ClusterState clusterState) {
if (state.compareAndSet(WatcherState.STOPPED, WatcherState.STARTING)) {
logger.info("starting watch service...");
watchLockService.start();
try {
logger.info("starting watch service...");
watchLockService.start();
// Try to load watch store before the execution service, b/c action depends on watch store
watchStore.start(clusterState);
executionService.start(clusterState);
triggerService.start(watchStore.watches().values());
state.set(WatcherState.STARTED);
logger.info("watch service has started");
// Try to load watch store before the execution service, b/c action depends on watch store
watchStore.start(clusterState);
executionService.start(clusterState);
triggerService.start(watchStore.watches().values());
state.set(WatcherState.STARTED);
logger.info("watch service has started");
} catch (Exception e) {
state.set(WatcherState.STOPPED);
throw e;
}
} else {
logger.debug("not starting watcher, because its state is [{}] while [{}] is expected", state, WatcherState.STOPPED);
}

19
src/main/java/org/elasticsearch/watcher/execution/ExecutionService.java
View File

@ -80,13 +80,18 @@ public class ExecutionService extends AbstractComponent {
assert executor.queue().isEmpty() : "queue should be empty, but contains " + executor.queue().size() + " elements.";
if (started.compareAndSet(false, true)) {
logger.debug("starting execution service");
historyStore.start();
triggeredWatchStore.start();
currentExecutions = new CurrentExecutions();
Collection<TriggeredWatch> triggeredWatches = triggeredWatchStore.loadTriggeredWatches(state);
executeTriggeredWatches(triggeredWatches);
logger.debug("started execution service");
try {
logger.debug("starting execution service");
historyStore.start();
triggeredWatchStore.start();
currentExecutions = new CurrentExecutions();
Collection<TriggeredWatch> triggeredWatches = triggeredWatchStore.loadTriggeredWatches(state);
executeTriggeredWatches(triggeredWatches);
logger.debug("started execution service");
} catch (Exception e) {
started.set(false);
throw e;
}
}
}

7
src/main/java/org/elasticsearch/watcher/execution/TriggeredWatchStore.java
View File

@ -93,7 +93,12 @@ public class TriggeredWatchStore extends AbstractComponent {
public void start() {
if (started.compareAndSet(false, true)) {
templateUtils.putTemplate(INDEX_TEMPLATE_NAME, customIndexSettings);
try {
templateUtils.putTemplate(INDEX_TEMPLATE_NAME, customIndexSettings);
} catch (Exception e) {
started.set(false);
throw e;
}
}
}

7
src/main/java/org/elasticsearch/watcher/history/HistoryStore.java
View File

@ -117,7 +117,12 @@ public class HistoryStore extends AbstractComponent implements NodeSettingsServi
public void start() {
if (started.compareAndSet(false, true)) {
templateUtils.putTemplate(INDEX_TEMPLATE_NAME, customIndexSettings);
try {
templateUtils.putTemplate(INDEX_TEMPLATE_NAME, customIndexSettings);
} catch (Exception e) {
started.set(false);
throw e;
}
}
}