ws-sandesha-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Deepal jayasinghe <deep...@gmail.com>
Subject Re: svn commit: r579262 - /webservices/sandesha/trunk/java/modules/core/src/main/java/org/apache/sandesha2/workers/SenderWorker.java
Date Tue, 25 Sep 2007 16:19:28 GMT

> Author: gatfora
> Date: Tue Sep 25 07:35:44 2007
> New Revision: 579262
>
> URL: http://svn.apache.org/viewvc?rev=579262&view=rev
> Log:
> Undo changes made to SenderWorker as this breaks the Sandesha unit tests.  The particular
case broken is when a sync ack is sent on an Async request/response, the OpContext was being
completed so no async response would ever be sent
>   
hmm , according to the code and by debugging what I understood was it is
handling the response. So in the case of response processing one of the
thing Axis2 does is set the operation context and all the other contexts
if they are available. That is why I fixed that , in the meantime I got
an NPE in Axis2 due to this when using RM. The NPE is due to unexpected
dispatching , to fix that issue the only way is to fill the response
message context with the request message context data.


Thanks
Deepal
> Modified:
>     webservices/sandesha/trunk/java/modules/core/src/main/java/org/apache/sandesha2/workers/SenderWorker.java
>
> Modified: webservices/sandesha/trunk/java/modules/core/src/main/java/org/apache/sandesha2/workers/SenderWorker.java
> URL: http://svn.apache.org/viewvc/webservices/sandesha/trunk/java/modules/core/src/main/java/org/apache/sandesha2/workers/SenderWorker.java?rev=579262&r1=579261&r2=579262&view=diff
> ==============================================================================
> --- webservices/sandesha/trunk/java/modules/core/src/main/java/org/apache/sandesha2/workers/SenderWorker.java
(original)
> +++ webservices/sandesha/trunk/java/modules/core/src/main/java/org/apache/sandesha2/workers/SenderWorker.java
Tue Sep 25 07:35:44 2007
> @@ -547,10 +547,6 @@
>  				responseMessageContext.setProperty(HTTPConstants.MTOM_RECEIVED_CONTENT_TYPE, requestMsgOpCtx
>  								.getProperty(HTTPConstants.MTOM_RECEIVED_CONTENT_TYPE));
>  
> - responseMessageContext.setOperationContext(requestMsgOpCtx);
> -                if ( !requestMsgOpCtx.isComplete()) {
> -                    requestMsgOpCtx.addMessageContext(responseMessageContext);
> -                }
>  				//If the response MsgCtx was not available Axis2 would hv put the transport info
into a 
>  				//HashMap, getting the data from it.
>  				HashMap transportInfoMap = (HashMap) msgCtx.getProperty(Constants.Configuration.TRANSPORT_INFO_MAP);
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: sandesha-dev-unsubscribe@ws.apache.org
> For additional commands, e-mail: sandesha-dev-help@ws.apache.org
>
>
>   


-- 
Thanks,
Deepal
................................................................
"The highest tower is built one brick at a time"


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


Mime
View raw message