[ https://issues.apache.org/jira/browse/CALCITE-3851?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17058495#comment-17058495 ]
Liya Fan commented on CALCITE-3851: ----------------------------------- [~julianhyde] Thanks a lot for your valuable comments. We have revised the PR to deprecate the old {{setImportance}} API, and replaced it with a new API named {{pruneNode}}, according to your suggestion. A test case is added for the new API. Please check if it looks good. Thank you in advance. > Replace the node importance map with a set for pruned nodes > ----------------------------------------------------------- > > Key: CALCITE-3851 > URL: https://issues.apache.org/jira/browse/CALCITE-3851 > Project: Calcite > Issue Type: Improvement > Components: core > Reporter: Liya Fan > Priority: Minor > Labels: pull-request-available > Time Spent: 50m > Remaining Estimate: 0h > > Currently, volcano planner stores rel node importances with a map. In fact, > the value of the map can only be 0. > So there is no need to store the values, and we replace the map with a set to > store nodes whose importances are 0. > This makes the code logic clearer, and avoids instability caused by comparing > floating point values. -- This message was sent by Atlassian Jira (v8.3.4#803005)