[CHAIN-63] Replace Context with Map<String, ? extends Object>

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Duplicate
• Affects Version/s: 2.0
• Fix Version/s: 2.0
• Assignee: Simone Tripodi
• Reporter: Ales Dolecek
• Created: 10/Jan/12 16:15
• Updated: 10/Jan/12 17:35
• Resolved: 10/Jan/12 16:36

Description

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.