Author: bharath
Date: Tue Jun 10 11:18:39 2008
New Revision: 666253
URL: http://svn.apache.org/viewvc?rev=666253&view=rev
Log:
Removed the fix made for CXF-1639. (Now we add Strings as keys to the
WSDLDefinition Map). From the code it looks like we are never putting literal
Strings as keys in maps on client or server side. So the leak could be because
of something else. We need to profile again and see whether the issue is still
there and if so why. Leaving the issue open till then.
Modified:
cxf/trunk/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java
Modified:
cxf/trunk/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java
URL:
http://svn.apache.org/viewvc/cxf/trunk/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java?rev=666253&r1=666252&r2=666253&view=diff
==============================================================================
--- cxf/trunk/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java
(original)
+++ cxf/trunk/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java
Tue Jun 10 11:18:39 2008
@@ -154,7 +154,7 @@
*
* @see org.apache.cxf.wsdl.WSDLManager#getDefinition(java.net.URL)
*/
- public Definition getDefinition(URL url) throws WSDLException {
+ public Definition getDefinition(URL url) throws WSDLException {
synchronized (definitionsMap) {
if (definitionsMap.containsKey(url)) {
return definitionsMap.get(url);
@@ -214,7 +214,11 @@
ResourceManagerWSDLLocator wsdlLocator = new
ResourceManagerWSDLLocator(url,
catLocator,
bus);
- return reader.readWSDL(wsdlLocator);
+ Definition def = reader.readWSDL(wsdlLocator);
+ synchronized (definitionsMap) {
+ definitionsMap.put(url, def);
+ }
+ return def;
}
private void registerInitialExtensions() throws BusException {
