You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@synapse.apache.org by ra...@apache.org on 2011/12/05 07:17:50 UTC

svn commit: r1210338 - in /synapse/trunk/java/modules/core/src/main/java/org/apache/synapse: config/Entry.java endpoints/ResolvingEndpoint.java

Author: rajikak
Date: Mon Dec  5 06:17:50 2011
New Revision: 1210338

URL: http://svn.apache.org/viewvc?rev=1210338&view=rev
Log:
Synchronized the endpoint init. in order to avoid double locking.

Modified:
    synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/config/Entry.java
    synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/ResolvingEndpoint.java

Modified: synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/config/Entry.java
URL: http://svn.apache.org/viewvc/synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/config/Entry.java?rev=1210338&r1=1210337&r2=1210338&view=diff
==============================================================================
--- synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/config/Entry.java (original)
+++ synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/config/Entry.java Mon Dec  5 06:17:50 2011
@@ -117,11 +117,11 @@ public class Entry implements SynapseArt
      * OMElement otherwise.
      * @return Either an OMElement or a String
      */
-    public synchronized Object getValue() {
+    public Object getValue() {
         return value;
     }
 
-    public synchronized void setValue(Object value) {
+    public void setValue(Object value) {
         this.value = value;
     }
 
@@ -129,7 +129,7 @@ public class Entry implements SynapseArt
      *
      * @return mapper
      */
-    public synchronized XMLToObjectMapper getMapper() {
+    public XMLToObjectMapper getMapper() {
         return mapper;
     }
 
@@ -138,7 +138,7 @@ public class Entry implements SynapseArt
      *
      * @param mapper XMLToObjectMapper instance
      */
-    public synchronized void setMapper(XMLToObjectMapper mapper) {
+    public void setMapper(XMLToObjectMapper mapper) {
         this.mapper = mapper;
     }
 

Modified: synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/ResolvingEndpoint.java
URL: http://svn.apache.org/viewvc/synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/ResolvingEndpoint.java?rev=1210338&r1=1210337&r2=1210338&view=diff
==============================================================================
--- synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/ResolvingEndpoint.java (original)
+++ synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/ResolvingEndpoint.java Mon Dec  5 06:17:50 2011
@@ -53,7 +53,7 @@ public class ResolvingEndpoint extends A
         }
     }
 
-    private synchronized Endpoint loadAndInitEndpoint(ConfigurationContext cc, String key) {
+    private Endpoint loadAndInitEndpoint(ConfigurationContext cc, String key) {
         Parameter parameter = cc.getAxisConfiguration().getParameter(
                 SynapseConstants.SYNAPSE_CONFIG);
         Parameter synEnvParameter = cc.getAxisConfiguration().getParameter(
@@ -70,7 +70,9 @@ public class ResolvingEndpoint extends A
 
             Endpoint ep = synCfg.getEndpoint(key);
             if (ep != null && !ep.isInitialized()) {
-                ep.init(synapseEnvironment);
+                synchronized (ep) {
+                    ep.init(synapseEnvironment);
+                }
             }
             return ep;
         }