[ https://issues.apache.org/jira/browse/CALCITE-3085?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16846395#comment-16846395 ]
Laurent Goujon commented on CALCITE-3085: ----------------------------------------- This is why I did check on Github. [~danny0405] are you aware of any class using the stack? > Unused stack field in RelShuttleImpl > ------------------------------------ > > Key: CALCITE-3085 > URL: https://issues.apache.org/jira/browse/CALCITE-3085 > Project: Calcite > Issue Type: Improvement > Components: core > Reporter: Laurent Goujon > Assignee: Laurent Goujon > Priority: Minor > Labels: pull-request-available > Time Spent: 10m > Remaining Estimate: 0h > > {{RelShuttleImpl}} class has a protected {{stack}} field which is being > populated when going over children, but those content is actually never used. > In Calcite code base, no subclasses are actually using the content of the > stack ({{CorelMapBuilder}} is populating the stack but does not read the > content back either). > Searching code on github didn't show any usage of it either (but this is not > foolprof). > As maintaining this stack has a non-negligible impact on memory/performance, > I would suggest to remove the field. -- This message was sent by Atlassian JIRA (v7.6.3#76005)