[CALCITE-2537] Make sure assertions are enabled before calling VolcanoPlanner#validate

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.18.0
• Component/s: core
• Assignee: Laurent Goujon
• Reporter: Laurent Goujon
• Created: 05/Sep/18 23:29
• Updated: 23/Dec/18 00:12
• Resolved: 17/Sep/18 18:10

Description

VolcanoPlanner#validate() is invoked if the log level is debug or lower. But considering how enabling this method slows down the whole planning (changing it for running tests make the whole duration several hours), and this is some kind of assertion (since it throws AssertionError, I would suggest to only enable it when assertions are enabled

Issue Links

• links to: PR #820