AbstractContext.setName() method needs refactoring, coz MBean registrations 
also happens there.
-----------------------------------------------------------------------------------------------

                 Key: SYNAPSE-726
                 URL: https://issues.apache.org/jira/browse/SYNAPSE-726
             Project: Synapse
          Issue Type: Bug
    Affects Versions: NIGHTLY
            Reporter: Sameera Jayasoma
             Fix For: 2.1


public abstract class AbstractEndpoint{
......
 public void setName(String endpointName) {
        this.endpointName = endpointName;
        metricsMBean = new EndpointView(endpointName, this);
        MBeanRegistrar.getInstance().registerMBean(metricsMBean, "Endpoint", 
endpointName);
    }
............
}

IMHO, we need move this MBean registration code out of this method. setName() 
should only set the name. Now it is doing some unrelated actions. Due to bit of 
code, we faced a memory leak recently. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to