[GROOVY-9631] Replace legacy data structure with Java collection

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.0.0-alpha-1
• Labels: breaking
• Assignee: Daniel Sun
• Reporter: Daniel Sun
• Created: 09/Jul/20 07:40
• Updated: 29/Jan/21 16:48
• Resolved: 27/Jul/20 23:47

Description

BACKGROUND
    Groovy home-made data structures are lack of maintainence for a long time, so it's better to replace them with mature and continuously improving data structure from JDK

SCOPE

1. Deprecate redundant AbstractConcurrentMapBase, AbstractConcurrentMap, ManagedConcurrentMap, and their related inner classes, e.g. Entry
2. Replace ManagedConcurrentMap with ManagedIdentityConcurrentMap, which is base on ConcurrentHashMap
3. Deprecate ComplexKeyHashMap, SingleKeyHashMap and TripleKeyHashMap. Note: TripleKeyHashMap is only used by never called method
4. Polish Index and MethodIndex in the class MetaClassImpl, they derive from SingleKeyHashMap

BENEFITS

1. Better maintainence and readability
2. Continuously improving data structure from JDK

PULL REQUEST

• https://github.com/apache/groovy/pull/1306

 

Issue Links

• is cloned by: GROOVY-9921 Replace legacy data structure with Java collection(SingleKeyHashMap)