[CALCITE-3487] Should not hard code RelMetadataQuery class in VolcanoPlanner.isValid()

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.22.0
• Labels: pull-request-available
• Assignee: Xiening Dai
• Reporter: Xiening Dai
• Created: 08/Nov/19 17:28
• Updated: 06/Mar/20 04:33
• Resolved: 08/Nov/19 23:22

Description

With CALCITE-3446, a calcite user can choose to provide a sub-class of RelMetadataQuery through RelOptCluster.setMetadataQuery(). We cannot assume the base class RelMetadataQuery is always used in planner.

Issue Links

• causes: CALCITE-3503 NPE at VolcanoPlanner#isValid when DEBUG is enabled
• links to: GitHub Pull Request #1569