-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 1.6.0
-
Component/s: None
-
Labels:None
Calcite has several methods that check internal consistency that have a 'boolean fail' argument indicating whether the method should throw or just return false if a consistency error is detected.
We propose to add a class Litmus:
interface Litmus { boolean fail(String message); boolean succeed(); }
and replace those boolean fail with Litmus litmus, so that the caller can determine policy on how to handle failures.
Methods affected include:
- RelNode.isValid(boolean)