xndai opened a new pull request #1385: [CALCITE-3257] Need to clear RelMetaDataQuery cache after dumping Rel… URL: https://github.com/apache/calcite/pull/1385 …Nodes and Graphviz Based on current design, RelMetadataQuery.map needs to be cleared between each rule firing. This is achieved through RelOptCluster.invalidateMetadataQuery() by VolcanoRuleCall.transformTo(). But when trace is enabled, the dump process would actually rebuild the meta data cache from previous rel tree. Then the subsequent rule firing doesn't get a chance to update rel node cost as it's been in the cache. A simple fix will just add a call to RelOptCluster.invalidateMetadataQuery() after dumping rel nodes.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
