Author: dkulp
Date: Wed Jun 11 08:44:24 2008
New Revision: 666715
URL: http://svn.apache.org/viewvc?rev=666715&view=rev
Log:
Merged revisions 666253 via svnmerge from
https://svn.apache.org/repos/asf/cxf/trunk
........
r666253 | bharath | 2008-06-10 14:18:39 -0400 (Tue, 10 Jun 2008) | 1 line
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/branches/2.0.x-fixes/ (props changed)
cxf/branches/2.0.x-fixes/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java
Propchange: cxf/branches/2.0.x-fixes/
------------------------------------------------------------------------------
Binary property 'svnmerge-integrated' - no diff available.
Modified:
cxf/branches/2.0.x-fixes/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java
URL:
http://svn.apache.org/viewvc/cxf/branches/2.0.x-fixes/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java?rev=666715&r1=666714&r2=666715&view=diff
==============================================================================
---
cxf/branches/2.0.x-fixes/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java
(original)
+++
cxf/branches/2.0.x-fixes/rt/core/src/main/java/org/apache/cxf/wsdl11/WSDLManagerImpl.java
Wed Jun 11 08:44:24 2008
@@ -152,7 +152,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);
@@ -211,7 +211,6 @@
ResourceManagerWSDLLocator wsdlLocator = new
ResourceManagerWSDLLocator(url,
catLocator,
bus);
-
Definition def = reader.readWSDL(wsdlLocator);
synchronized (definitionsMap) {
definitionsMap.put(url, def);
