[CALCITE-1550] ConventionTraitDef.plannerConversionMap is not thread safe

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Duplicate
• Fix Version/s: 1.12.0
• Assignee: Julian Hyde
• Reporter: Chunhui Shi
• Created: 22/Dec/16 00:48
• Updated: 24/Mar/17 03:20
• Resolved: 21/Feb/17 17:52

Description

We are using static instance ConventionTraitDef.INSTANCE globally and plannerConversionMap(class WeakHashMap) defined in ConventionTraitDef class is not threadsafe. And the data in the map could corrupt and cause dead loop or other data error.

private final WeakHashMap<RelOptPlanner, ConversionData>
plannerConversionMap =
new WeakHashMap<RelOptPlanner, ConversionData>();

Issue Links

• duplicates: CALCITE-1594 ConventionTraitDef.getConversionData() is not thread-safe