[
https://issues.apache.org/jira/browse/CHAIN-101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13695384#comment-13695384
]
Benedikt Ritter commented on CHAIN-101:
---------------------------------------
{quote}
since the majority agrees on your PoV, feel free to provide the necessary fix
for this.
{quote}
Well, right now we are still discussing this, right? ;) If there are strong
arguments against delegation, let's not do it.
I agree, that implementing the complete Map interface is a draw back. But I
don't see a point in the thread safety provided by ConcurrentHashMap. The
Context interface explicitly states, that Context implementations do not have
to be thread safe (of course is does not hurt if they are).
> Change ContextMap from an inheritance based to a delegation based design
> ------------------------------------------------------------------------
>
> Key: CHAIN-101
> URL: https://issues.apache.org/jira/browse/CHAIN-101
> Project: Commons Chain
> Issue Type: Improvement
> Affects Versions: 2.0
> Reporter: Benedikt Ritter
> Fix For: 2.0
>
>
> Currently ContextMap extends ConcurrentHashMap, but there is nothing we gain
> from this design decision. A delegation based implementation that takes a Map
> as constructor arguments and delegates calls to the wrapped map yields higher
> flexibility.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
