On Fri, Aug 09, 2002 at 11:30:24AM +0200, Carsten Ziegeler wrote: > Bug 11131 applied - please cross-check again. Looks Good So far! :-)
Thanks Once Again! Regards, Michael Melhem > > Thanks > Carsten > > > -----Original Message----- > > From: Michael Melhem [mailto:[EMAIL PROTECTED]] > > Sent: Friday, August 09, 2002 10:50 AM > > To: [EMAIL PROTECTED] > > Subject: Re: [PATCH] Patch to complete (auto) caching-point > > functionality > > > > > > Hi Carsten! > > > > Thanks for the quick application of the patch. You are a legend mate! > > :-) > > > > Intial testing shows thats everything is fine!! > > > > However there is still a small generic problem with the pipelines > > at the moment. > > Namely [Bug 11131]. This patch should be applied so that caching-point > > (and other pipelines) would function at 100% > > > > the patch is quite small. > > In AbstractCachingProcessingPipeline.java > > > > > > - finished = false; > > - this.completeResponseIsCached = false; > > // try a shorter key > > // FIXME (CZ) calculation of PPK > > if (cachedPipelineKey.size() > 1) { > > cachedPipelineKey.removeLastKey(); > > - this.firstProcessedTransformerIndex--; > > + if (!this.completeResponseIsCached) { > > + this.firstProcessedTransformerIndex--; > > + } > > } else { > > cachedPipelineKey = null; > > } > > + finished = false; > > + this.completeResponseIsCached = false; > > } > > } > > > > Many thanks, > > Michael Melhem > > > > > > > > On Fri, Aug 09, 2002 at 09:51:32AM +0200, Carsten Ziegeler wrote: > > > Hi Michael, > > > > > > thanks for your patch - I just applied it. > > > Please cross-check if everything is working fine. > > > > > > Thanks > > > Carsten > > > > > > > -----Original Message----- > > > > From: Michael Melhem [mailto:[EMAIL PROTECTED]] > > > > Sent: Wednesday, August 07, 2002 4:25 PM > > > > To: [EMAIL PROTECTED] > > > > Cc: [EMAIL PROTECTED] > > > > Subject: Re: [PATCH] Patch to complete (auto) caching-point > > > > functionality > > > > > > > > > > > > Hi Sylvain, > > > > > > > > Hope its not still raining where your at! :-) > > > > > > > > <snip> > > > > > Yes, it looks perfectly acceptable ;) > > > > > > > > > > Would you like to resubmit your patch this way ? I'll try > > to apply it > > > > > from my laptop and its sloooow modem connection. If I > > can't, I'll ask > > > > > someone else to do it (e.g. Carsten, with which IIRC you initially > > > > > worked with on caching points). > > > > > > > > > </snip> > > > > > > > > The re-worked patch is now submitted as Bugzilla [Bug 11530]. > > > > > > > > Thanks a lot for your time!! > > > > > > > > Regards, > > > > Michael Melhem > > > > > > > > > Regards, > > > > > Sylvain > > > > > > > > > > -- > > > > > Sylvain Wallez > > > > > Anyware Technologies Apache Cocoon > > > > > http://www.anyware-tech.com mailto:[EMAIL PROTECTED] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > > For additional commands, email: [EMAIL PROTECTED] > > > > > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > For additional commands, email: [EMAIL PROTECTED] > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, email: [EMAIL PROTECTED] > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, email: [EMAIL PROTECTED] > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, email: [EMAIL PROTECTED]
