[CALCITE-3234] For boolean properties, empty string should mean "true"

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.21.0
• Labels: pull-request-available
• Assignee: Julian Hyde
• Reporter: Julian Hyde
• Created: 07/Aug/19 19:38
• Updated: 10/Sep/19 21:37
• Resolved: 09/Aug/19 20:47

Description

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).

Issue Links

• links to: GitHub Pull Request #1363