[Discuss] Mock member publishes health stats even after it is terminated

[Rajkumar Rajaratnam]

Hi Devs,

If you enable debug logs, you will experience $Subject. The reason is that
we don't shutdown the healthStatNotifierExecutorService when we terminate
the mock member. This should be the fix.


diff --git
a/components/org.apache.stratos.mock.iaas/src/main/java/org/apache/stratos/mock/iaas/services/impl/MockInstance.java
b/components/org.apache.stratos.mock.iaas/src/main/java/org/apache/stratos/mock/iaas/services/impl/Mock
index ab3d319..43fcf7d 100644
---
a/components/org.apache.stratos.mock.iaas/src/main/java/org/apache/stratos/mock/iaas/services/impl/MockInstance.java
+++
b/components/org.apache.stratos.mock.iaas/src/main/java/org/apache/stratos/mock/iaas/services/impl/MockInstance.java
@@ -75,6 +75,8 @@ public class MockInstance implements Runnable,
Serializable {
             sleep(1000);
         }

+        stopHealthStatisticsPublisher();
+
         if (log.isInfoEnabled()) {
             log.info(String.format("Mock member terminated: [member-id]
%s", mockMemberContext.getMemberId()));
         }
@@ -139,6 +141,10 @@ public class MockInstance implements Runnable,
Serializable {
         }
     }

+    private void stopHealthStatisticsPublisher() {
+               healthStatNotifierExecutorService.shutdownNow();
+       }

Thanks.

-- 
Rajkumar Rajaratnam
Committer & PMC Member, Apache Stratos
Software Engineer, WSO2

Mobile : +94777568639
Blog : rajkumarr.com