[jira] [Commented] (CHAIN-63) Replace Context with Map

["Ales Dolecek (Commented) (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/CHAIN-63?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13183392#comment-13183392 ] 

Ales Dolecek commented on CHAIN-63:
-----------------------------------

Ahh - sorry. I looked checked out current trunk and there is still Context as "extends Map<String, Object>".

Where is the development going on? And BTW CHAIN-58 does not mention to aim for complete drop of Context - maybe it's "understood", but it better be stated explicitly. After all 1.x chains also use "Context extens Map" without any further advantage.

Ales
                
> Replace Context with Map<String, ? extends Object>
> --------------------------------------------------
>
>                 Key: CHAIN-63
>                 URL: https://issues.apache.org/jira/browse/CHAIN-63
>             Project: Commons Chain
>          Issue Type: Improvement
>    Affects Versions: 2.0
>            Reporter: Ales Dolecek
>            Assignee: Simone Tripodi
>            Priority: Minor
>             Fix For: 2.0
>
>
> Hello,
>   I always wandered why you *force* chain users to use the Context interface which adds nothing over Map. I would understand that if the Context would not inherit from map and only required
> Object put(String, Object)
> Object get(String)
> But since Context itself IS Map then you gain nothing while disallowing use of Map implementations. I know that sou suggest that Context shall implement "Attribute-Property Transparency" but it is only recomendation and is not forced by commons chains anyway.
> And if you define execute like
> Object execute(Map<String, ? extends Object> context)
> it can still accept Context instances and all its subclasses.
> The downside is that you would have to give up the retrieve method introduced to Context. And BTW the Context imposes hard restriction see my other bug.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira