[
https://issues.apache.org/jira/browse/CALCITE-4747?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17403465#comment-17403465
]
Jianhui Dong commented on CALCITE-4747:
---------------------------------------
Thank you for your feedback. I think when these vertices were removed(line
970), the edges between the vertices are also unreacheable, but these edges
were still in the graph, which may cause a memory leak.
> it seemes that collecGarbage in HepPlanner dose not clean outdated edges?
> -------------------------------------------------------------------------
>
> Key: CALCITE-4747
> URL: https://issues.apache.org/jira/browse/CALCITE-4747
> Project: Calcite
> Issue Type: Improvement
> Reporter: Jianhui Dong
> Priority: Minor
> Attachments: image-2021-08-19-12-12-41-534.png
>
>
> !image-2021-08-19-12-12-41-534.png!
> 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.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
