Skip to content

Check the window aggregates on the typed Spark surface - #77

Merged
estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:test/window-aggregates
Oct 10, 2026
Merged

estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:test/window-aggregates

Conversation

@estebanzimanyi

Copy link
Copy Markdown
Member

GeneratedSqlSurfaceTest.windowAggregatesAnswerWhatMobilityDBAnswers checks that wCount, wMax and
wAvg over two temporal integers and wSum over a discrete temporal float, with a window of one day,
answer what MobilityDB answers over values in partitions of their own, so the interval reaches
the transition of every partition and the combine joins their states.

Witness: with the generator of JMEOS ef197a02 the surface registers no window aggregate, the 30
overloads of wCount, wMin, wMax, wSum and wAvg left out as taking two arguments.

Measured over MobilityDB 1f41894b35, the catalog of MEOS-API ff5f838173 and the generator of
JMEOS #169 (26044b19): the 42 tests pass.

Why: a window aggregate answers on Spark as it does in PostgreSQL.

GeneratedSqlSurfaceTest.windowAggregatesAnswerWhatMobilityDBAnswers checks that wCount, wMax and
wAvg over two temporal integers and wSum over a discrete temporal float, with a window of one day,
answer what MobilityDB answers over values in partitions of their own, so the interval reaches
the transition of every partition and the combine joins their states.

Witness: with the generator of JMEOS ef197a02 the surface registers no window aggregate, the 30
overloads of wCount, wMin, wMax, wSum and wAvg left out as taking two arguments.

Measured over MobilityDB 1f41894b35, the catalog of MEOS-API ff5f838173 and the generator of
JMEOS #169 (26044b19): the 42 tests pass.

Why: a window aggregate answers on Spark as it does in PostgreSQL.
@estebanzimanyi
estebanzimanyi merged commit 76bc3e2 into MobilityDB:main Oct 10, 2026
2 checks passed
@estebanzimanyi
estebanzimanyi deleted the test/window-aggregates branch October 10, 2026 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant