[CALCITE-1812] Provide RelMetadataQuery from planner to rules and invalidate in transformTo

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.13.0
• Component/s: core
• Labels: performance
• Assignee: Remus Rusanu
• Reporter: Remus Rusanu
• Created: 30/May/17 21:38
• Updated: 21/Mar/19 16:23
• Resolved: 09/Jun/17 21:07

Description

Currently rules create a needed RelMetadataQuery when matched (ie. RuleXxx.onMatch. Since rules are called often, on complex query this triggers a multiplication RMQ instances used. But each new instance carries a heavy price on performance, because is the RMQ instance that hosts the cache used to memoize various metadata lookups (like getEstimatedRows, getPulledUpPredicates, uniqueness and so on). From HIVE-16757 investigation, the performance impact can be 10x.

The proposal is to have a RelMetadataQuery in the planner and pass it to the rules as a RelOptRuleCall property. Rules onMatch can make use of this instance. The rule call transformTo can handle invalidating the RMQ instance.

If code knowingly invalidates metadata properties associated with a node by means other than transformTo, it should somehow invalidate the current RMQ.

The RMQ invalidation can be made smarter, but my plan right now is to do the dumb thing and just toss it away. Will revisit this later, depending on measured impact.

Issue Links

• is blocked by: CALCITE-1816 JaninoRelMetadataProvider generated classes leak ACTIVE nodes on exception
• is related to: CALCITE-2942 Materialized view rewriting logic instantiates RelMetadataQuery each time the rule is triggered