[ 
https://issues.apache.org/jira/browse/CALCITE-2537?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Julian Hyde resolved CALCITE-2537.
----------------------------------
       Resolution: Fixed
    Fix Version/s: 1.18.0

Fixed in 
[a5378a32|http://git-wip-us.apache.org/repos/asf/calcite/commit/a5378a32].

> Make sure assertions are enabled before calling VolcanoPlanner#validate
> -----------------------------------------------------------------------
>
>                 Key: CALCITE-2537
>                 URL: https://issues.apache.org/jira/browse/CALCITE-2537
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>            Reporter: Laurent Goujon
>            Assignee: Laurent Goujon
>            Priority: Minor
>             Fix For: 1.18.0
>
>
> {{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



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to