Thanks Reka. Will do. On Sun, Jan 4, 2015 at 9:35 PM, Reka Thirunavukkarasu <[email protected]> wrote:
> +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 > > > -- -- 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
