Skip to content

Answer the temporal aggregates as MobilityDB answers them - #58

Merged
estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:test/temporal-aggregates
Sep 28, 2026
Merged

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

Conversation

@estebanzimanyi

Copy link
Copy Markdown
Member

GeneratedSurfaceTest runs tCount, tSum, tMinAgg, tMaxAgg and mergeAgg over
three temporal integers, and tAndAgg and tOrAgg over two temporal booleans,
each value in a partition of its own, so the transition runs in every
partition and the combine joins the partial states the shuffle carries
between them. Each answer is the text MobilityDB answers for the same
aggregate over the same values in UTC. tSum over a temporal integer and a
temporal float in two partitions raises the refusal of a mix of types.

Measured. Against JMEOS with the generated aggregates, the libmeos of
MobilityDB 510c3184f6 and the catalog derived from its installed headers,
the suite runs 20 tests, all passing; with the combine of the aggregator
dropped, tCount answers null and the test fails.

Why. The generated surface registers the aggregates, and a partial state
that crosses executors is the one path the scalar tests never take.

GeneratedSurfaceTest runs tCount, tSum, tMinAgg, tMaxAgg and mergeAgg over
three temporal integers, and tAndAgg and tOrAgg over two temporal booleans,
each value in a partition of its own, so the transition runs in every
partition and the combine joins the partial states the shuffle carries
between them. Each answer is the text MobilityDB answers for the same
aggregate over the same values in UTC. tSum over a temporal integer and a
temporal float in two partitions raises the refusal of a mix of types.

Measured. Against JMEOS with the generated aggregates, the libmeos of
MobilityDB 510c3184f6 and the catalog derived from its installed headers,
the suite runs 20 tests, all passing; with the combine of the aggregator
dropped, tCount answers null and the test fails.

Why. The generated surface registers the aggregates, and a partial state
that crosses executors is the one path the scalar tests never take.
@estebanzimanyi
estebanzimanyi merged commit bf91d9c into MobilityDB:main Sep 28, 2026
2 checks passed
@estebanzimanyi
estebanzimanyi deleted the test/temporal-aggregates branch September 28, 2026 21:56
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