[
https://issues.apache.org/jira/browse/CALCITE-3062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16837894#comment-16837894
]
Laurent Goujon commented on CALCITE-3062:
-----------------------------------------
Can you clarify what do you mean by that? There's only one logger and one level
involved which is {{CalciteTrace.getPlannerTracer()}}/{{DEBUG}} and the
behavior is pretty clear. The change I suggest is to not populate the map if
the log level is at least not {{DEBUG}}.
> Do not populate provenanceMap if not used
> -----------------------------------------
>
> Key: CALCITE-3062
> URL: https://issues.apache.org/jira/browse/CALCITE-3062
> Project: Calcite
> Issue Type: Improvement
> Components: core
> Reporter: Laurent Goujon
> Assignee: Laurent Goujon
> Priority: Major
> Labels: pull-request-available
> Time Spent: 10m
> Remaining Estimate: 0h
>
> {{VolcanoPlanner}}'s field {{provenanceMap}} tracks the provenance of each
> node being added to the planner, but the information is only used when the
> log level of the planner of the debug, so when planner is not running in
> debug mode, this data goes to waste and used memory unnecessary.
> The planner should be changed so that the information is only captured if
> used later.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
