commit 54de9a67633dc82a059d2a6db21f98d9ac8a1827
author Xikui Wang <xkkwww@gmail.com> Fri Mar 16 10:32:32 2018 -0700
committer Xikui Wang <xkkwww@gmail.com> Fri Mar 23 10:41:31 2018 -0700
tree d6095e28a3cd3f16ee4c3b227693b53b3b018fd3
parent 8aa13cacfbbb5c22b4bf48515c48a5b96853ce02

[NO ISSUE][RT] Remove runtime from ActiveManager after it stops

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
The runtime should not be removed before it's fully stopped. This may
causes issues when downstream operators need to access runtime during
shutdown.

Change-Id: I07c6398f46d4bb6d29b270aa71dee36e0d0917ae
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2494
Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Reviewed-by: abdullah alamoudi <bamousaa@gmail.com>

asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveManager.java

diff --git a/asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveManager.java b/asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveManager.java
index bb76204..2aebc59 100644
--- a/asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveManager.java
+++ b/asterixdb/asterix-active/src/main/java/org/apache/asterix/active/ActiveManager.java
@@ -178,7 +178,7 @@
 
     private void stopIfRunning(ActiveRuntimeId runtimeId, IActiveRuntime runtime)
             throws HyracksDataException, InterruptedException {
-        if (runtimes.remove(runtimeId) != null) {
+        if (runtimes.containsKey(runtimeId)) {
             runtime.stop();
         } else {
             LOGGER.info("Not stopping already stopped runtime " + runtimeId);