IntenseFinance/nosqlbench
3
0
Fork 0
mirror of https://github.com/nosqlbench/nosqlbench.git synced 2025-01-26 15:36:33 -06:00
Code Issues Packages Projects Releases Wiki Activity

replace missing cancelation for shutdown hook

Browse Source
This commit is contained in:
Jonathan Shook 2023-01-18 14:28:17 -06:00
parent 3cd4a42e64
commit 5914549124
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
engine-core/src/main/java/io/nosqlbench/engine/core/lifecycle/scenario/Scenario.java
View File

@ -278,6 +278,7 @@ public class Scenario implements Callable<ExecutionMetricsResult> {
Runtime.getRuntime().removeShutdownHook(scenarioShutdownHook);
scenarioShutdownHook.run();
scenarioShutdownHook = null;
}
public void notifyException(Thread t, Throwable e) {
@ -339,7 +340,7 @@ public class Scenario implements Callable<ExecutionMetricsResult> {
}
}
public void finish() {
public synchronized void finish() {
logger.debug("finishing scenario");
endedAtMillis = System.currentTimeMillis(); //TODO: Make only one endedAtMillis assignment
if (this.state == State.Running) {
@ -351,6 +352,8 @@ public class Scenario implements Callable<ExecutionMetricsResult> {
// that the scenario was ended before the hook was uninstalled normally.
this.state = State.Interrupted;
logger.warn("Scenario was interrupted by process exit, shutting down");
} else {
logger.info("Scenario completed successfully, with " + scenarioController.getActivityExecutorMap().size() + " logical activities.");
}
logger.info(() -> "scenario state: " + this.state);