+1 to remove this..Yah..It is properly implemented in the ClusterMonitor
class..

Thanks,
Reka

On Sun, Jan 4, 2015 at 1:27 AM, Lahiru Sandaruwan <[email protected]> wrote:

> Hi guys,
>
> In AbstractClusterMonitor, $subject seems not completed. But it is also
> overridden in ClusterMonitor class. Therefore this might not get executed.
> Do we need this anymore?
>
>
>     @Override
>     public void onParentStatusEvent(MonitorStatusEvent statusEvent) {
>         if (statusEvent.getStatus() == GroupStatus.Terminating ||
>                 statusEvent.getStatus() == ApplicationStatus.Terminating) {
>             ClusterStatusEventPublisher.sendClusterTerminatingEvent(appId, 
> this.getServiceId(),
>                     clusterId, statusEvent.getInstanceId());
>         } else if (statusEvent.getStatus() == ClusterStatus.Created ||
>                 statusEvent.getStatus() == GroupStatus.Created) {
>             Application application = 
> ApplicationHolder.getApplications().getApplication(this.appId);
>             Group group = 
> application.getGroupRecursively(statusEvent.getId());
>             //TODO*****starting a new instance of this monitor
>             //createGroupInstance(group, statusEvent.getInstanceId());
>         }
>         // send the ClusterTerminating event
> //        if (statusEvent.getStatus() == GroupStatus.Terminating || 
> statusEvent.getStatus() ==
> //                ApplicationStatus.Terminating) {
> //
> //        }
>     }
>
> Thanks.
> --
> --
> Lahiru Sandaruwan
> Committer and PMC member, Apache Stratos,
> Senior Software Engineer,
> WSO2 Inc., http://wso2.com
> lean.enterprise.middleware
>
> email: [email protected] blog: http://lahiruwrites.blogspot.com/
> linked-in: http://lk.linkedin.com/pub/lahiru-sandaruwan/16/153/146
>
>


-- 
Reka Thirunavukkarasu
Senior Software Engineer,
WSO2, Inc.:http://wso2.com,
Mobile: +94776442007

Reply via email to