synapse-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ruwan Linton <ruwan.lin...@gmail.com>
Subject Re: svn commit: r749766 - /synapse/trunk/java/modules/transports/src/main/java/org/apache/synapse/transport/nhttp/HttpCoreRequestResponseTransport.java
Date Wed, 04 Mar 2009 14:22:37 GMT
Done !

Thanks,
Ruwan

On Wed, Mar 4, 2009 at 2:24 PM, Andreas Veithen
<andreas.veithen@gmail.com>wrote:

> If it is properly documented in the code and if you add a comment to
> SYNAPSE-493 so that we can keep track of this, then I'm fine with it.
>
> Andreas
>
> On Wed, Mar 4, 2009 at 03:31, Ruwan Linton <ruwan.linton@gmail.com> wrote:
> > Andreas,
> >
> > I didn't note this earlier. My intention was to switch back to sandesha2
> > because the current RM implementation that Synapse is using for the
> moment
> > (WSO2 Mercury) is not supported by the WSO2 any more at least in this
> year
> > and I wanted to move back to sandesha2.
> >
> > If I describe the history of the RM implementation switching we couldn't
> get
> > Sandesha2 to work with NIO HTTP transport after the 1.0 release of
> synapse
> > and that is why we moved to the Mercury implementation and now I don't
> want
> > synapse to depend on that because the project is not live at this moment.
> >
> > So I suggest that we use this as a work around till we get this fixed in
> > Sandesha2.... I have tested all the scenarios which might be affected by
> > this change and if there is any issue please do let me know. I will add a
> > TODO documentation in to this piece of code as well.
> >
> > (We need to get RM to work with Sandesha2 for the 1.3 release)
> >
> > Thanks,
> > Ruwan
> >
> > On Wed, Mar 4, 2009 at 4:18 AM, Andreas Veithen <
> andreas.veithen@gmail.com>
> > wrote:
> >>
> >> Ruwan,
> >>
> >> This change was proposed in SYNAPSE-493, but there the conclusion was
> >> that this change would be in contradiction with the available
> >> documentation in Axis2 and that it is more likely an issue in
> >> Sandesha2.
> >>
> >> Andreas
> >>
> >> On Tue, Mar 3, 2009 at 22:37,  <ruwan@apache.org> wrote:
> >> > Author: ruwan
> >> > Date: Tue Mar  3 21:37:47 2009
> >> > New Revision: 749766
> >> >
> >> > URL: http://svn.apache.org/viewvc?rev=749766&view=rev
> >> > Log:
> >> > set the status to WAITING for fixing some RM scenarios, I have tested
> >> > with all dual channel invocations and it seems to work fine
> >> >
> >> > Modified:
> >> >
> >> >
>  synapse/trunk/java/modules/transports/src/main/java/org/apache/synapse/transport/nhttp/HttpCoreRequestResponseTransport.java
> >> >
> >> > Modified:
> >> >
> synapse/trunk/java/modules/transports/src/main/java/org/apache/synapse/transport/nhttp/HttpCoreRequestResponseTransport.java
> >> > URL:
> >> >
> http://svn.apache.org/viewvc/synapse/trunk/java/modules/transports/src/main/java/org/apache/synapse/transport/nhttp/HttpCoreRequestResponseTransport.java?rev=749766&r1=749765&r2=749766&view=diff
> >> >
> >> >
> ==============================================================================
> >> > ---
> >> >
> synapse/trunk/java/modules/transports/src/main/java/org/apache/synapse/transport/nhttp/HttpCoreRequestResponseTransport.java
> >> > (original)
> >> > +++
> >> >
> synapse/trunk/java/modules/transports/src/main/java/org/apache/synapse/transport/nhttp/HttpCoreRequestResponseTransport.java
> >> > Tue Mar  3 21:37:47 2009
> >> > @@ -39,7 +39,7 @@
> >> >  public class HttpCoreRequestResponseTransport implements
> >> > RequestResponseTransport {
> >> >
> >> >     private static final Log log =
> >> > LogFactory.getLog(HttpCoreRequestResponseTransport.class);
> >> > -    private RequestResponseTransportStatus status =
> >> > RequestResponseTransportStatus.INITIAL;
> >> > +    private RequestResponseTransportStatus status =
> >> > RequestResponseTransportStatus.WAITING;
> >> >     private MessageContext msgContext = null;
> >> >     private boolean responseWritten = false;
> >> >
> >> >
> >> >
> >> >
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org
> >> For additional commands, e-mail: dev-help@synapse.apache.org
> >>
> >
> >
> >
> > --
> > Ruwan Linton
> > http://wso2.org - "Oxygenating the Web Services Platform"
> > http://ruwansblog.blogspot.com/
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org
> For additional commands, e-mail: dev-help@synapse.apache.org
>
>


-- 
Ruwan Linton
http://wso2.org - "Oxygenating the Web Services Platform"
http://ruwansblog.blogspot.com/

Mime
View raw message