[CALCITE-4747] Remove outdated graph edges in HepPlanner

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.28.0
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Jianhui Dong
• Created: 19/Aug/21 04:13
• Updated: 18/Sep/21 14:33
• Resolved: 09/Sep/21 07:30

Description

 

After we use HepPlanner to optimize the RelNode,  we need to replace old RelNode with the optimized RelNode, and update the graph in HepPlanner, but calcite only remove the outdated RelNode(HepRelVertex) in line 970, the edges between these vertices are still in the graph, which may cause a memory leak? IMO, I think maybe we should also remove the outdated edge.

Issue Links

• links to: GitHub Pull Request #2501