axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From barre...@apache.org
Subject svn commit: r649213 - /webservices/axis2/trunk/java/modules/metadata/src/org/apache/axis2/jaxws/description/impl/EndpointDescriptionImpl.java
Date Thu, 17 Apr 2008 18:19:38 GMT
Author: barrettj
Date: Thu Apr 17 11:19:35 2008
New Revision: 649213

URL: http://svn.apache.org/viewvc?rev=649213&view=rev
Log:
Add call to AxisService.releaseSchemaList in EndpointDescriptionImpl constructors.

Modified:
    webservices/axis2/trunk/java/modules/metadata/src/org/apache/axis2/jaxws/description/impl/EndpointDescriptionImpl.java

Modified: webservices/axis2/trunk/java/modules/metadata/src/org/apache/axis2/jaxws/description/impl/EndpointDescriptionImpl.java
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/metadata/src/org/apache/axis2/jaxws/description/impl/EndpointDescriptionImpl.java?rev=649213&r1=649212&r2=649213&view=diff
==============================================================================
--- webservices/axis2/trunk/java/modules/metadata/src/org/apache/axis2/jaxws/description/impl/EndpointDescriptionImpl.java
(original)
+++ webservices/axis2/trunk/java/modules/metadata/src/org/apache/axis2/jaxws/description/impl/EndpointDescriptionImpl.java
Thu Apr 17 11:19:35 2008
@@ -211,7 +211,6 @@
                             ServiceDescriptionImpl parent) {
         this(theClass, portName, dynamicPort, parent, null, null);
     }
-
     EndpointDescriptionImpl(Class theClass, QName portName, boolean dynamicPort,
                             ServiceDescriptionImpl parent, 
                             DescriptionBuilderComposite sparseComposite,
@@ -265,7 +264,10 @@
         } catch (Exception e) {
             String msg = Messages.getMessage("endpointDescriptionErr2",e.getClass().getName(),parent.getClass().getName());
             throw ExceptionFactory.makeWebServiceException(msg, e);
+        } finally {
+            releaseAxisServiceResources();
         }
+        
     }
     
     EndpointDescriptionImpl(ServiceDescriptionImpl parent, String serviceImplName) {
@@ -536,6 +538,9 @@
         
         // Configure any available WebServiceFeatures on the endpoint.
         configureWebServiceFeatures();
+        
+        // REVIEW: there are some throws above that won't cause the release
+        releaseAxisServiceResources();
     }
 
     /**
@@ -571,6 +576,8 @@
         buildEndpointDescriptionFromAnnotations();
         
         configureWebServiceFeatures();
+        
+        releaseAxisServiceResources();
 
         // The anonymous AxisOperations are currently NOT added here.  The reason 
         // is that (for now) this is a SERVER-SIDE code path, and the anonymous operations
@@ -1010,6 +1017,13 @@
 
         //TODO: Set other things on AxisService here, this function may have to be
         //      moved to after we create all the AxisOperations
+    }
+
+    private void releaseAxisServiceResources() {
+        // release the schema list in the AxisService
+        if (axisService != null) {
+            axisService.releaseSchemaList();
+        }
     }
 
     private void buildDescriptionHierachy() {



---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org


Mime
View raw message