[jira] [Commented] (CALCITE-2342) use of assert with side-effect in Schemas

["Julian Hyde (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/CALCITE-2342?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16496091#comment-16496091 ] 

Julian Hyde commented on CALCITE-2342:
--------------------------------------

Nice catch.

> use of assert with side-effect in Schemas
> -----------------------------------------
>
>                 Key: CALCITE-2342
>                 URL: https://issues.apache.org/jira/browse/CALCITE-2342
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>            Reporter: Laurent Goujon
>            Assignee: Laurent Goujon
>            Priority: Major
>
> {{Schemas#path(CalciteSchema,Iterable<String>)}} method has the following code:
> {code:java}
> if (!rootSchema.name.isEmpty()) {
>   assert rootSchema.name.equals(iterator.next());
> }
> {code}
> Depending if assertions are enabled or not, iterator state might be different



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