You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@synapse.apache.org by ch...@apache.org on 2007/03/30 10:19:11 UTC

svn commit: r523983 - /webservices/synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/SALoadbalanceEndpoint.java

Author: chathura_ce
Date: Fri Mar 30 01:19:09 2007
New Revision: 523983

URL: http://svn.apache.org/viewvc?view=rev&rev=523983
Log:
Session affinity load balance endpoint should build the envelope in the first request of the session. So that it can resend the message, if the selected endpoint has failed.

Modified:
    webservices/synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/SALoadbalanceEndpoint.java

Modified: webservices/synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/SALoadbalanceEndpoint.java
URL: http://svn.apache.org/viewvc/webservices/synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/SALoadbalanceEndpoint.java?view=diff&rev=523983&r1=523982&r2=523983
==============================================================================
--- webservices/synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/SALoadbalanceEndpoint.java (original)
+++ webservices/synapse/trunk/java/modules/core/src/main/java/org/apache/synapse/endpoints/SALoadbalanceEndpoint.java Fri Mar 30 01:19:09 2007
@@ -147,7 +147,7 @@
             // this is the first request. so an endpoint has not been bound to this session and we
             // are free to failover if the currently selected endpoint is not working. but for
             // failover to work, we have to build the soap envelope.
-            //synMessageContext.getEnvelope().build();
+            synMessageContext.getEnvelope().build();
 
             // we should also indicate that this is the first message in the session. so that
             // onFault(...) method can resend only the failed attempts for the first message.
@@ -158,8 +158,7 @@
 
             // endpoints given by session dispatchers may not be active. therefore, we have check
             // it here.
-            if (endpoint.isActive(synMessageContext)) {
-                //synMessageContext.getEnvelope().build(); todo: why this is needed here
+            if (endpoint.isActive(synMessageContext)) {                
                 endpoint.send(synMessageContext);
             } else {
                 informFailure(synMessageContext);



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