@malloydata/malloy 0.0.320 → 0.0.321

package/dist/lang/composite-source-utils.js

@@ -925,7 +925,7 @@ function isSingleValueFilterNode(e) {
                 return [];
             if ((result.parsed.operator === 'null' && !result.parsed.not) ||
                 (result.kind === 'boolean' &&
-                    ['false', 'true'].includes(result.parsed.operator) &&
+                    ['true', '=false', '=true'].includes(result.parsed.operator) &&
                     !result.parsed.not) ||
                 (result.kind === 'date' &&
                     result.parsed.operator === 'in' &&

package/dist/model/filter_compilers.d.ts

@@ -3,7 +3,7 @@ import type { Dialect, QueryInfo } from '../dialect';
 export declare const FilterCompilers: {
     compile(t: string, c: FilterExpression | null, x: string, d: Dialect, qi?: QueryInfo): string;
     numberCompile(nc: NumberFilter, x: string, d: Dialect): string;
-    booleanCompile(bc: BooleanFilter, x: string, _d: Dialect): string;
+    booleanCompile(bc: BooleanFilter, x: string, d: Dialect): string;
     stringCompile(sc: StringFilter, x: string, d: Dialect): string;
     temporalCompile(tc: TemporalFilter, x: string, d: Dialect, t: "date" | "timestamp", qi?: QueryInfo): string;
 };

package/dist/model/filter_compilers.js

@@ -100,19 +100,52 @@ exports.FilterCompilers = {
                     .join(` ${nc.operator.toUpperCase()} `);
         }
     },
-    booleanCompile(bc, x, _d) {
-        switch (bc.operator) {
-            case 'true':
-                return bc.not ? `NOT COALESCE(${x}, false)` : `COALESCE(${x}, false)`;
-            case 'false':
+    booleanCompile(bc, x, d) {
+        const px = `(${x})`;
+        /*
+         * We have the following truth table for boolean filters.
+         * The default malloy operations treat null as false. The '='
+         * variants exist for cases where that is not desired.
+         *
+         * filter expression | x=true | x=false | x=null
+         * true              |   T    |   F     |   F
+         * not true          |   F    |   T     |   T
+         * =true             |   T    |   F     |   NULL
+         * not =true         |   F    |   T     |   NULL
+         * false             |   F    |   T     |   T
+         * not false         |   T    |   F     |   F
+         * =false            |   F    |   T     |   NULL
+         * not =false        |   T    |   F     |   NULL
+         */
+        if (bc.operator === '=true') {
+            return bc.not ? `NOT ${px}` : x;
+        }
+        else if (bc.operator === '=false') {
+            return bc.not ? x : `NOT ${px}`;
+        }
+        else if (bc.operator === 'null') {
+            return bc.not ? `${px} IS NOT NULL` : `${px} IS NULL`;
+        }
+        // For some databases checking NULL combined with a boolean check
+        // is faster than a COALESCE, for now, just detect if the expression
+        // is just a column reference, and if so, don't use COALECSE.
+        const quoteChar = d.sqlMaybeQuoteIdentifier('select')[0];
+        const isColumn = x.match(`^[()${quoteChar}\\w.]+$`);
+        if (isColumn) {
+            if (bc.operator === 'true') {
                 return bc.not
-                    ? `COALESCE(${x} != false, true)`
-                    : `COALESCE(${x} = false, false)`;
-            case 'false_or_null':
-                return bc.not ? `COALESCE(${x}, false)` : `NOT COALESCE(${x}, false)`;
-            case 'null':
-                return bc.not ? `${x} IS NOT NULL` : `${x} IS NULL`;
+                    ? `${px} IS NULL OR ${px} = false`
+                    : `${px} IS NOT NULL AND ${px}`;
+            }
+            return bc.not
+                ? `${px} IS NOT NULL AND ${px}` // not false: exclude null
+                : `${px} IS NULL OR ${px} = false`; // false: include null
+        }
+        if (bc.operator === 'true') {
+            return bc.not ? `NOT COALESCE(${x}, false)` : `COALESCE(${x}, false)`;
         }
+        // else bc.operator === 'false'
+        return bc.not ? `COALESCE(${x}, false)` : `NOT COALESCE(${x}, false)`;
     },
     stringCompile(sc, x, d) {
         switch (sc.operator) {

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const MALLOY_VERSION = "0.0.320";
+export declare const MALLOY_VERSION = "0.0.321";

package/dist/version.js

@@ -2,5 +2,5 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.MALLOY_VERSION = void 0;
 // generated with 'generate-version-file' script; do not edit manually
-exports.MALLOY_VERSION = '0.0.320';
+exports.MALLOY_VERSION = '0.0.321';
 //# sourceMappingURL=version.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@malloydata/malloy",
-  "version": "0.0.320",
+  "version": "0.0.321",
   "license": "MIT",
   "exports": {
     ".": "./dist/index.js",
@@ -41,9 +41,9 @@
     "generate-version-file": "VERSION=$(npm pkg get version --workspaces=false | tr -d \\\")\necho \"// generated with 'generate-version-file' script; do not edit manually\\nexport const MALLOY_VERSION = '$VERSION';\" > src/version.ts"
   },
   "dependencies": {
-    "@malloydata/malloy-filter": "0.0.320",
-    "@malloydata/malloy-interfaces": "0.0.320",
-    "@malloydata/malloy-tag": "0.0.320",
+    "@malloydata/malloy-filter": "0.0.321",
+    "@malloydata/malloy-interfaces": "0.0.321",
+    "@malloydata/malloy-tag": "0.0.321",
     "antlr4ts": "^0.5.0-alpha.4",
     "assert": "^2.0.0",
     "jaro-winkler": "^0.2.8",