[CALCITE-3879] Rel Id should be generated per query context instead of globally

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: core
• Assignee: Unassigned
• Reporter: Haisheng Yuan
• Created: 26/Mar/20 22:21
• Updated: 20/Aug/21 07:12

Description

The rel id will continue increasing until the class is unloaded. We have observed that the id is reaching 100 million after running for just several days. Although we don't use the id for calculation, it might be OK for integer overflow, but it doesn't make sense to generate unique id globally, it will hard to read the dumped plan and the rel id in the log, because the rel id is so large. IMO, the id should be generated per query context.

Issue Links

• is related to: CALCITE-4741 AbstractRelNode#getId can overflow into a negative value causing CompileException in certain Enumerable implement methods