axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From hemap...@apache.org
Subject svn commit: r547773 - /webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/engine/DispatchPhase.java
Date Fri, 15 Jun 2007 20:02:33 GMT
Author: hemapani
Date: Fri Jun 15 13:02:32 2007
New Revision: 547773

URL: http://svn.apache.org/viewvc?view=rev&rev=547773
Log:
remove the dispatch phase optimization for now https://issues.apache.org/jira/browse/AXIS2-2191

Modified:
    webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/engine/DispatchPhase.java

Modified: webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/engine/DispatchPhase.java
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/engine/DispatchPhase.java?view=diff&rev=547773&r1=547772&r2=547773
==============================================================================
--- webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/engine/DispatchPhase.java
(original)
+++ webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/engine/DispatchPhase.java
Fri Jun 15 13:02:32 2007
@@ -13,6 +13,7 @@
 import org.apache.axis2.description.AxisOperation;
 import org.apache.axis2.description.AxisService;
 import org.apache.axis2.description.AxisServiceGroup;
+import org.apache.axis2.description.WSDL2Constants;
 import org.apache.axis2.i18n.Messages;
 import org.apache.axis2.transport.RequestResponseTransport;
 import org.apache.axis2.transport.TransportListener;
@@ -105,17 +106,22 @@
             msgContext.setAxisService(msgContext.getServiceContext().getAxisService());
         }
 
-        //TODO: The same thing should probably happen for a IN-OUT if addressing is enabled
and the replyTo/faultTo are not anonymous 
+        
+        //We do not send the status even though it is In-Only to allow Sandesha to send a
ack
+        //https://issues.apache.org/jira/browse/AXIS2-2191. At F2F 2007 summer june, we decided
may be 
+        //providing a option might not give us that much. I will keep the code commented
so we might want to do this later. 
+//        if (msgContext.getAxisOperation().getMessageExchangePattern()
+//                .equals(WSDL20_2004_Constants.MEP_URI_IN_ONLY)) {
+//            Object requestResponseTransport =
+//                    msgContext.getProperty(RequestResponseTransport.TRANSPORT_CONTROL);
+//            if (requestResponseTransport != null) {
+//                ((RequestResponseTransport) requestResponseTransport)
+//                        .acknowledgeMessage(msgContext);
+//            }
+//        }
+        
         if (msgContext.getAxisOperation().getMessageExchangePattern()
-                .equals(WSDL20_2004_Constants.MEP_URI_IN_ONLY)) {
-            Object requestResponseTransport =
-                    msgContext.getProperty(RequestResponseTransport.TRANSPORT_CONTROL);
-            if (requestResponseTransport != null) {
-                ((RequestResponseTransport) requestResponseTransport)
-                        .acknowledgeMessage(msgContext);
-            }
-        } else if (msgContext.getAxisOperation().getMessageExchangePattern()
-                .equals(WSDL20_2004_Constants.MEP_URI_IN_OUT))
+                .equals(WSDL2Constants.MEP_URI_IN_OUT))
         {   // OR, if 2 way operation but the response is intended to not use the response
channel of a 2-way transport
             // then we don't need to keep the transport waiting.
             Object requestResponseTransport =



---------------------------------------------------------------------
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