[CHAIN-40] ContextBase should use LinkedHashMap

• Type: Improvement
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Component/s: Base
• Assignee: Unassigned
• Reporter: Cédric BODIN
• Created: 10/Oct/07 13:33
• Updated: 04/Jul/13 19:37

Description

org.apache.commons.chain.impl.ContextBase should use java.util.LinkedHashMap (instead of java.util.HashMap) : as its superclass, and as the java.util.Map implementation for its "descriptors" attribute.
Indeed LinkedHashMap allow "insertion-order" iteration over the context, which is an additional information about the chain execution (to know the order of the commands into the chain).
Moreover LinkedHashMap is a HashMap : so this enhancement is "transparent" for users who didn't/don't/won't want to presume about context's iteration order.

Thanks in advance.
Cédric BODIN

Issue Links

• is superceded by: CHAIN-101 Change ContextMap from an inheritance based to a delegation based design