Details
-
Type:
Bug
-
Status:
Resolved
-
Priority:
Trivial
-
Resolution: Fixed
-
Affects Version/s: 2.2
-
Fix Version/s: 2.3
-
Component/s: PicoContainer Gems
-
Labels:None
-
Number of attachments :
Description
The trick I used with Slf4j & Log4j of getting the log category in a separate string and serializing that particular string doesn't work with CommonsLogging since it appears that the CommonsLogging Log object doesn't have a getName()/getCategory() or any similar method.
I'm not sure if this is fixable.
Activity
Upon further debugging, it appears that CommonsLogging DOES serialize correctly, so I don't need a getName(). [Saw the wrong member was null and panicked. :( ]
Michael Rimov
added a comment - Upon further debugging, it appears that CommonsLogging DOES serialize correctly, so I don't need a getName(). [Saw the wrong member was null and panicked. :( ]
JCL does not guarantee serializability! It depends completely on the used logger implementation. However, all delivered Log-Adapters of JCL are serializable except the adapter for the Avalon Logger - since the Avalon Logger is not serializable. Therefore serialization for CLCM works only if the user selects a logger implementation that is serializable. You should update the Javadoc stating this fact (and remove those <string>Bug.</strong> elements).
- Jörg
Jörg Schaible
added a comment - JCL does not guarantee serializability! It depends completely on the used logger implementation. However, all delivered Log-Adapters of JCL are serializable except the adapter for the Avalon Logger - since the Avalon Logger is not serializable. Therefore serialization for CLCM works only if the user selects a logger implementation that is serializable. You should update the Javadoc stating this fact (and remove those <string>Bug.</strong> elements).
Jörg
Done. Thanks for keeping me honest 
Michael Rimov
added a comment - Done. Thanks for keeping me honest 
I forgot:
The current workaround is to use the CommonsComponentMonitor constructor that takes either a Class or a String as the category name instead of providing a Logger directly.