"alesj" wrote : &quot;adrian(a)jboss.org&quot; wrote : | | Clearly, the caching in AbstractBeanInfoFactory shouldn't be using | | classInfo.getName() in its caching. | | | What about if I just change that the key is ClassInfo instead of String? | (I've done that locally and all the tests pass) But it wouldn't be much of a cache since the ClassInfo's can be pretty transient. Also I think it will cause a memory leak. You effectively have: WeakValueHashMap<ClassInfo, BeanInfo> but the ClassInfo could itself get a hard reference on the BeanInfo via one of its "attachments", e.g. the caching of SchemaBindings in jbossxb against the TypeInfo. View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4190912#... Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&a...