> With few other changes I have committed the patch. I must say this is a > great improvement and this also fixes the shutdown issue.
Great to hear that. Ruwan, thanks for reviewing and applying this patch! What do you and others think about my suggestion of moving all the server related classes from the top level to a new sub package server. Here is my original comment from the JIRA: > > While working on this patch I thought about moving some of the classes > in the synapse top level package to a new server subpackage. I actually > did not perform this change to ease the review, but I still think it would > be a good idea. > > So what about moving the following classes: > > Axis2SynapseController > > JmxAdapter > > ServerConfigurationInformation > > ServerConfigurationInformationFactory > > ServerContextInformation > > ServerManager > > ServerState > > ServerStateDetectionStrategy > > SynapseController > > SynapseControllerFactory > > SynapseServer > > to > > synapse.server > > and > > ServerManagerView > > ServerManagerViewMBean > > to > > synapse.server.mbean (maybe also renaming them to ServerManagerControl > and ServerManagerControlMBean.
