About the Async MR case.. I checked the code and things seem to have
gone to the correct places...

But we need more testing!!!

~Thilina

On 10/15/06, Sanjiva Weerawarana <[EMAIL PROTECTED]> wrote:
Deepal did u update the AsyncMRs? Those need to change of course.

Also, looks like you broke the Spring stuff; see the mail from Robert.

Sanjiva.
On Sat, 2006-10-14 at 09:28 +0600, Thilina Gunarathne wrote:
> A quick comment...
>
> Did you test the Async message receivers... My instincts say they
> might be broken..
>
> Thanks,
> Thilina
>
> On 10/14/06, Deepal Jayasinghe <[EMAIL PROTECTED]> wrote:
> > Hi Thilina;
> >
> > Dont worry it wont break any thing (its a small change), and its API
> > change we need to have that for this release.
> >
> > P:S - we discussed that in the mailing list and IRC before we doing that.
> >
> > > Hi all,
> > > I would like if we can reconsider this commit...
> > >
> > > I know it's a bug, but at this point I feel it's worth to live with it
> > > rather than doing major changes...  IMHO  a change like this needs to
> > > be tested for a while... We need to learn from our past bad
> > > experiences of last minute changes...
> > >
> > > Even at this moment I feel that this will get broken at the
> > > AsyncMessageReceiver, since we are spawning a seperate thread...
> > >
> >
> >
> >
>
>


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]




--
http://webservices.apache.org/~thilina/
http://thilinag.blogspot.com/

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to