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 23585] - Localton of ev_cache.ser file in DefaultEventRegistryImpl class
Date Tue, 11 Nov 2003 15:14:32 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=23585>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=23585

Localton of ev_cache.ser file in DefaultEventRegistryImpl class





------- Additional Comments From javageoff@yahoo.com  2003-11-11 15:14 -------
Actually, I initially stayed away from the idea of storing the event info in the
Store/Cache because I hadn't worked out whether there would be "issues" doing so
and the serialization was just a quick solution while I worked that out
mentally.  After you (?) suggested it again in our discussions I went back and
coded a solution that did just that and committed it.  It worked in my initial
tests but there are these outstanding issues which I can't address right now
because of time constraints: 

1) Really test the thing to make sure there are no weird problems.
2) Work out the naming convention mess I created by naming the serializing
solution the "Default".
3) Change the default configuration for the block to use the
StoreEventRegistryImpl instead of the serializing version.

My proposed solution for 2) would be to copy the existing code in
DefaultEventRegistryImpl to a new SerializingEventRegistryImpl, remove all code
from DefaultEventRegistryImpl, make it extend the new Serializing Impl and mark
Default as deprecated.  That may be overkill but would maintain
back-compatibility for anyone already using it and isn't any real work.  

If someone can fully test the new Store version, I can do the rest pretty
quickly.  I'll leave the bug open for now since I can't act right away.

Geoff

Mime
View raw message