@malloydata/render 0.0.123-dev240207191052 → 0.0.123-dev240208155844

package/dist/bundle/bundled_renderer.js

@@ -56008,12 +56008,17 @@ QUERY: ${(0, util_1.inspect)(opDesc, { breakLength: 72, depth: Infinity })}`);
           } else {
             if ((props === null || props === void 0 ? void 0 : props.orderBys) && props.orderBys.length > 0) {
               const isAnalytic = (0, malloy_types_1.expressionIsAnalytic)(overload.returnType.expressionType);
+              if (!isAnalytic) {
+                if (!this.inExperiment("aggregate_order_by", true)) {
+                  props.orderBys[0].log("Enable experiment `aggregate_order_by` to use `order_by` with an aggregate function");
+                }
+              }
               if (dialectOverload.supportsOrderBy || isAnalytic) {
                 const allowExpression = dialectOverload.supportsOrderBy !== "only_default";
                 const allObs = props.orderBys.flatMap((orderBy) => isAnalytic ? orderBy.getAnalyticOrderBy(fs) : orderBy.getAggregateOrderBy(fs, allowExpression));
                 frag.orderBy = allObs;
               } else {
-                props.orderBys[0].log(`Function ${this.name} does not support order_by`);
+                props.orderBys[0].log(`Function \`${this.name}\` does not support \`order_by\``);
               }
             }
             if ((props === null || props === void 0 ? void 0 : props.limit) !== void 0) {
@@ -66493,7 +66498,6 @@ QUERY: ${(0, util_1.inspect)(opDesc, { breakLength: 72, depth: Infinity })}`);
           return this.astAt(new ast.FunctionOrderBy(f4, dir), pcx);
         }
         visitAggregateOrderByStatement(pcx) {
-          this.inExperiment("function_order_by", pcx);
           return this.visitAggregateOrdering(pcx.aggregateOrdering());
         }
         visitAggregateOrdering(pcx) {
@@ -66894,7 +66898,6 @@ QUERY: ${(0, util_1.inspect)(opDesc, { breakLength: 72, depth: Infinity })}`);
           return new ast.ExprProps(this.getFieldExpr(pcx.fieldExpr()), statements);
         }
         visitPartitionByStatement(pcx) {
-          this.inExperiment("partition_by", pcx);
           return this.astAt(new ast.PartitionBy(pcx.id().map((idCx) => this.astAt(new ast.PartitionByFieldReference([
             this.astAt(new ast.FieldName((0, parse_utils_1.idToStr)(idCx)), idCx)
           ]), idCx))), pcx);