Bug 11131 applied - please cross-check again. 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]
