@malloydata/malloy 0.0.225-dev250106171109 → 0.0.225-dev250111002123

package/dist/dialect/snowflake/snowflake.js

@@ -145,7 +145,7 @@ class SnowflakeDialect extends dialect_1.Dialect {
     sqlUnnestAlias(source, alias, _fieldList, _needDistinctKey, isArray, _isInNestedPipeline) {
         const as = this.sqlMaybeQuoteIdentifier(alias);
         if (isArray) {
-            return `,LATERAL FLATTEN(INPUT => ${source}) AS ${alias}_1, LATERAL (SELECT ${alias}_1.INDEX, object_construct('value', ${alias}_1.value) as value ) as ${as}`;
+            return `LEFT JOIN lateral flatten(input => ${source}) as ${as}`;
         }
         else {
             // have to have a non empty row or it treats it like an inner join :barf-emoji:
@@ -197,9 +197,11 @@ class SnowflakeDialect extends dialect_1.Dialect {
         if (childName === '__row_id') {
             return `"${parentAlias}".INDEX::varchar`;
         }
-        else if (parentType === 'array[scalar]' ||
-            parentType === 'array[record]') {
-            const arrayRef = `"${parentAlias}".value:${sqlName}`;
+        else if (parentType.startsWith('array')) {
+            let arrayRef = `"${parentAlias}".value`;
+            if (parentType === 'array[record]') {
+                arrayRef += `:${sqlName}`;
+            }
             switch (childType) {
                 case 'record':
                 case 'array':

package/dist/lang/ast/field-space/dynamic-space.js

@@ -119,7 +119,6 @@ class DynamicSpace extends static_space_1.StaticSpace {
             const fields = [];
             const joins = [];
             const turtles = [];
-            const fixupJoins = [];
             for (const [name, spaceEntry] of this.entries()) {
                 if (spaceEntry instanceof struct_space_field_base_1.StructSpaceFieldBase) {
                     joins.push([name, spaceEntry]);
@@ -139,7 +138,7 @@ class DynamicSpace extends static_space_1.StaticSpace {
                     if (!error_factory_1.ErrorFactory.didCreate(joinStruct)) {
                         fieldIndices.set(name, this.sourceDef.fields.length);
                         this.sourceDef.fields.push(joinStruct);
-                        fixupJoins.push([field.join, joinStruct]);
+                        field.join.fixupJoinOn(this, joinStruct);
                     }
                 }
                 else {
@@ -155,10 +154,6 @@ class DynamicSpace extends static_space_1.StaticSpace {
                     // }
                 }
             }
-            // If we have join expressions, we need to now go back and fill them in
-            for (const [join, missingOn] of fixupJoins) {
-                join.fixupJoinOn(this, missingOn);
-            }
             // Add access modifiers at the end so views don't obey them
             for (const [name, access] of this.newAccessModifiers) {
                 const index = this.sourceDef.fields.findIndex(f => { var _a; return (_a = f.as) !== null && _a !== void 0 ? _a : f.name === name; });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@malloydata/malloy",
-  "version": "0.0.225-dev250106171109",
+  "version": "0.0.225-dev250111002123",
   "license": "MIT",
   "exports": {
     ".": "./dist/index.js",