[CALCITE-5679] HepPlanner#buildFinalPlan: do not clear metadata cache if RelNode has not changed

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.35.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Ruben Q L
• Reporter: Ruben Q L
• Created: 28/Apr/23 13:50
• Updated: 27/Jul/23 10:22
• Resolved: 29/Apr/23 18:17

Description

Minor optimization:

HepPlanner#buildFinalPlan clears the metadata cache info of the impacted RelNode after replacing its inputs, but in the cases where the RelNode does not change (e.g. when it has no inputs) this clear cache should not be necessary (unless I'm missing something).

Issue Links

• is related to: CALCITE-3791 HepPlanner doesn't clear metadata cache for the ancestors of discarded node when a transformation happens
• links to: GitHub Pull Request #3181