From b7b60ddc5fb5dd05d70bf456e4fe43a5778351df Mon Sep 17 00:00:00 2001 From: Esteban Zimanyi Date: Sun, 18 Oct 2026 19:55:51 +0200 Subject: [PATCH] Call the window average wAvgAgg The test of the window aggregates calls the window average wAvgAgg, the name the MEOS catalog states for Tnumber_wavg_transfn (`@altsqlfn wAvgAgg()` on MobilityDB master) and the name JMEOS's codegen_jvm.py registers it under, an aggregate taking its catalog altSqlName. Why: the build derives the surface from the catalog of MobilityDB master, which names the aggregate wAvgAgg in the engines, MobilityDB keeping wAvg; DuckDB names its weighted average wavg and compares names without case, so the three engines take the one name. --- src/test/java/org/mobilitydb/spark/GeneratedSqlSurfaceTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/test/java/org/mobilitydb/spark/GeneratedSqlSurfaceTest.java b/src/test/java/org/mobilitydb/spark/GeneratedSqlSurfaceTest.java index 91fd4f7a..1e83d4c2 100644 --- a/src/test/java/org/mobilitydb/spark/GeneratedSqlSurfaceTest.java +++ b/src/test/java/org/mobilitydb/spark/GeneratedSqlSurfaceTest.java @@ -356,7 +356,7 @@ void windowAggregatesAnswerWhatMobilityDBAnswers() { windowAggregate("wMax", "tintFromText", tints)); assertEquals("Interp=Step;{[1@2001-01-01 00:00:00+00, 2@2001-01-02 00:00:00+00, " + "2@2001-01-04 00:00:00+00], (3@2001-01-04 00:00:00+00, 3@2001-01-05 00:00:00+00]}", - windowAggregate("wAvg", "tintFromText", tints)); + windowAggregate("wAvgAgg", "tintFromText", tints)); assertEquals("{[1@2001-01-01 00:00:00+00, 1@2001-01-02 00:00:00+00], " + "[3@2001-01-03 00:00:00+00, 3@2001-01-04 00:00:00+00]}", windowAggregate("wSum", "tfloatFromText", "{1@2001-01-01 00:00:00+00, 3@2001-01-03 00:00:00+00}"));