[TAP5-1078] Refactor the management of ServiceProxyProvider

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Won't Fix
• Affects Version/s: 5.1.0.5
• Component/s: tapestry-ioc
• Assignee: Unassigned
• Reporter: Igor Drobiazko
• Created: 25/Mar/10 10:51
• Updated: 25/Jul/13 21:33
• Resolved: 25/Jul/13 21:32

Description

Currently the ServiceProxyProvider/Registry is stored in a static field of SerializationSupport and retrieved via static methods. This approach works fine if Tapestry IoC JAR is located in WEB-INF/lib of an application. Unfortunately this approach is not compatible with OSGi where Tapestry IoC bundle is shared by several bundles, each creating its own Registry. The registry does a SerializationSupport #setProvider() at startup. Starting several Registries inside a JVM it is not possible to make sure that the Registry reference in SerializationSupport is always the same.

I realized the problem starting two OSGi bundles, each starting its own Registry. When one of these bundles is stopped, its Registry is shut down. In this case we see the following log message:

25.03.2010 10:47:04,680 # ERROR # org.apache.tapestry5.ioc.internal.SerializationSupport # [SerializationSupport.setProvider] # Setting a new service proxy provider when there's already an existing provider. This may indicate that you have multiple IoC Registries.

We should refactor the management of ServiceProxyProvider. I think it is sufficient to move the logic of SerializationSupport into InternalRegistry.