Author: alessio.soldano(a)jboss.com
Date: 2012-12-13 14:11:49 -0500 (Thu, 13 Dec 2012)
New Revision: 17072
Modified:
common/trunk/src/main/java/org/jboss/ws/common/deployment/DefaultLifecycleHandler.java
common/trunk/src/main/java/org/jboss/ws/common/deployment/DefaultLifecycleHandlerFactory.java
common/trunk/src/main/java/org/jboss/ws/common/deployment/EndpointHandlerDeploymentAspect.java
Log:
[JBWS-3577] RequestHandler and LifecycleHandler usage as singletons
Modified:
common/trunk/src/main/java/org/jboss/ws/common/deployment/DefaultLifecycleHandler.java
===================================================================
---
common/trunk/src/main/java/org/jboss/ws/common/deployment/DefaultLifecycleHandler.java 2012-12-13
19:11:07 UTC (rev 17071)
+++
common/trunk/src/main/java/org/jboss/ws/common/deployment/DefaultLifecycleHandler.java 2012-12-13
19:11:49 UTC (rev 17072)
@@ -36,6 +36,17 @@
*/
public class DefaultLifecycleHandler implements LifecycleHandler
{
+ private static DefaultLifecycleHandler me;
+
+ public static DefaultLifecycleHandler getInstance()
+ {
+ if (me == null)
+ {
+ me = new DefaultLifecycleHandler();
+ }
+ return me;
+ }
+
public void start(Endpoint ep)
{
EndpointState state = ep.getState();
Modified:
common/trunk/src/main/java/org/jboss/ws/common/deployment/DefaultLifecycleHandlerFactory.java
===================================================================
---
common/trunk/src/main/java/org/jboss/ws/common/deployment/DefaultLifecycleHandlerFactory.java 2012-12-13
19:11:07 UTC (rev 17071)
+++
common/trunk/src/main/java/org/jboss/ws/common/deployment/DefaultLifecycleHandlerFactory.java 2012-12-13
19:11:49 UTC (rev 17072)
@@ -30,8 +30,15 @@
*/
public class DefaultLifecycleHandlerFactory extends LifecycleHandlerFactory
{
+ @Deprecated
public LifecycleHandler newLifecycleHandler()
{
return new DefaultLifecycleHandler();
}
+
+ @Override
+ public LifecycleHandler getLifecycleHandler()
+ {
+ return DefaultLifecycleHandler.getInstance();
+ }
}
Modified:
common/trunk/src/main/java/org/jboss/ws/common/deployment/EndpointHandlerDeploymentAspect.java
===================================================================
---
common/trunk/src/main/java/org/jboss/ws/common/deployment/EndpointHandlerDeploymentAspect.java 2012-12-13
19:11:07 UTC (rev 17071)
+++
common/trunk/src/main/java/org/jboss/ws/common/deployment/EndpointHandlerDeploymentAspect.java 2012-12-13
19:11:49 UTC (rev 17072)
@@ -52,22 +52,24 @@
@Override
public void start(final Deployment dep)
{
+ final RequestHandler reqHandler = getRequestHandler();
+ final LifecycleHandler lcHandler = getLifecycleHandler();
for (final Endpoint ep : dep.getService().getEndpoints())
{
- ep.setRequestHandler(getRequestHandler());
- ep.setLifecycleHandler(getLifecycleHandler());
+ ep.setRequestHandler(reqHandler);
+ ep.setLifecycleHandler(lcHandler);
ep.setInvocationHandler(getInvocationHandler(ep));
}
}
private RequestHandler getRequestHandler()
{
- return spiProvider.getSPI(RequestHandlerFactory.class).newRequestHandler();
+ return spiProvider.getSPI(RequestHandlerFactory.class).getRequestHandler();
}
private LifecycleHandler getLifecycleHandler()
{
- return spiProvider.getSPI(LifecycleHandlerFactory.class).newLifecycleHandler();
+ return spiProvider.getSPI(LifecycleHandlerFactory.class).getLifecycleHandler();
}
private InvocationHandler getInvocationHandler(final Endpoint ep)
Show replies by date