"adrian(a)jboss.org" wrote : | The brain dead part is that it gets used at runtime in the BasicMetaDataRepository | as the key to a map which it then freezes to avoid mutablity. | The issue being that you still could incur contention because | of the synchronization on the scopes within the ScopeKey traversed in equals()/hashCode(). | I assume you mean in the current implementation (even with freeze, synchronization is needed). With the proposed changes, we don't need the synchronization on the scopes within the (immutable) ScopeKey. View the original post : http://www.jboss.org/index.html?module=bb&op=viewtopic&p=4249020#... Reply to the post : http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&a...