cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 30356] - CIncludeTransformer + event aware caching
Date Wed, 28 Jul 2004 05:51:50 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=30356>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=30356

CIncludeTransformer + event aware caching





------- Additional Comments From oliver.powell@tvnz.co.nz  2004-07-28 05:51 -------
Also, a few days ago Corin Moss (cmoss@tvnz.co.nz) posted the following question
on the dev mailing list:

>"We're implementing Event Aware caching at the moment, and it's working
>well.  We're migrating from the Prism based cache which we've used 
>until now.  Our only hurdle at the moment is proving to be the 
>CIncludeTransformer.  Expires based caching is hard wired right into it 
>(and the DefaultIncludeCacheManager).  Has anyone done any work on 
>anything like this?  I've not been able to find anything in CVS.  I 
>suspect that it won't be easy, as the methods that need to be 
>overridden are split between the implementation, and its helper 
>classes.
>
>Any thoughts?"
>
>

He received the following response from Unico Hommes:

"I hadn't noticed this before but you seem to be correct. What we need to 
do is make CIncludeTransformer support a different cache validity 
generation method appart from the current expires one. Similar to the 
one used by TraversableGenerator. The TG builds up an aggregated 
validity object as it progresses through its generation of events. Each 
time it encounters a new Source to include it adds the validity object 
from that source to the aggregated validity. Since EventValidity objects 
do not need a new validity for comparison when determing whether they 
are valid or not this works out nicely."

I think the changes posted with this bug follow Unico's recommendation. Please
feel free to critique the code!

Mime
View raw message