[jira] [Commented] (CALCITE-3062) Do not populate provenanceMap if not used

["Danny Chan (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/CALCITE-3062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16837740#comment-16837740 ] 

Danny Chan commented on CALCITE-3062:
-------------------------------------

As far as i know, the provenanceMap is used for rule matching when there can not produce a target traits set nodes, saying the convention, is should not be only constrained to DEBUG mode.

> 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)