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=16704>.
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=16704

DefaultModuleConfigFactory should implement serializable





------- Additional Comments From [EMAIL PROTECTED]  2003-02-04 22:49 -------
The problem was repored due to Tomcat logging exception when 
serializing/deserializing the ServletContext attributes (when you stop-start 
the tomcat). This occurs because we add the ModuleConfigImpl to the context. 
This class holds a reference to the factory which created it 
(DefaultModuleConfigFactory) which is not Serializable.
I feel that the solution should be to replace the Object reference from 
ModuleConfigImpl with a Factory name reference (i.e. store the name of the 
factory instead of the reference to an object). Am attaching a patch for the 
same. Have also reverted the Factory to be non-serializable - believe it makes 
sense to make all Config objects Serializable and all Factories non-
serializable (unless there's a specific requirement otherwise).
Will still recommend making the ModuleConfig interface implement Serializable 
(have not included that in the patch)

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

Reply via email to