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

Julian Hyde commented on CALCITE-3234:
--------------------------------------

Assigning to me, because I already created a fix. Sorry, I should have said.

I want to use the property to enable extra testing within one case. E.g. as 
part of CALCITE-1935,
{code}
    final int n = CalciteSystemProperty.TEST_SLOW.value() ? 80 : 5;
    final Random r = new Random(0);
    for (int i = 0; i < n; i++) {
      checkArrayList(i * i + 10, r);
    }
{code}

How would you accomplish this post-CALCITE-3141?


> For boolean properties, empty string should mean "true"
> -------------------------------------------------------
>
>                 Key: CALCITE-3234
>                 URL: https://issues.apache.org/jira/browse/CALCITE-3234
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: Julian Hyde
>            Assignee: Julian Hyde
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.21.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> For boolean properties, empty string should mean "true". For example, if I 
> add -Dcalcite.test.slow to the JVM args, CalciteSystemProperty.TEST_SLOW 
> should have value "true".
> This was broken by CALCITE-2859 (Util.getBooleanProperty, which was removed, 
> had the desired behavior).



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Reply via email to