Skip to content

Test every aggregate against MobilityDB's answer over the same rows - #467

Merged
estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:test/aggregates-from-catalog
Oct 18, 2026
Merged

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

Conversation

@estebanzimanyi

@estebanzimanyi estebanzimanyi commented Oct 18, 2026 •

Copy link
Copy Markdown
Member

test/sql/aggregates_catalog.test runs each of the 262 aggregate signatures MobilityDuck registers
over two rows of sample values of its first argument, the other arguments fixed, and states the
answer MobilityDB gives over the same rows in the time zone UTC, at the MobilityDB commit
vcpkg_ports/meos/portfile.cmake pins (7146de0fba). Each aggregate is called by the name the MEOS
catalog states for it, its altSqlName where it has one. A value of a type whose text carries a
coordinate or a float, the 26 types both engines give an asText with a number of decimal digits,
is written with six decimals in both engines, asText(x, 6): the last digits of a coordinate an
aggregate computes, the extent and the centroid of network points among them, differ between
builds. The test also states the set of aggregate
signatures registered under the catalog's names, which is the catalog's but for the entries over
raster and geography, types MobilityDuck does not have. The point cloud values name pcid 1, whose
schema the test registers as test/sql/tpcpatch.test does.

Measured: MobilityDB answers all 262 queries and MobilityDuck answers each as MobilityDB does,
529 assertions, but wAvgAgg(tbigint), which MobilityDB answers with the bits of its values read as
a double (5e-324 for 1), a MEOS defect the test states in place of the answer.

Why: a test per aggregate stating MobilityDB's answer is what shows an aggregate MobilityDuck
registers computing what MobilityDB computes, and the registered set stated whole is what shows a
change of the catalog reaching MobilityDuck.

@estebanzimanyi
estebanzimanyi force-pushed the test/aggregates-from-catalog branch from 84499b3 to 4c702d1 Compare October 18, 2026 17:07
test/sql/aggregates_catalog.test runs each of the 262 aggregate signatures MobilityDuck registers
over two rows of sample values of its first argument, the other arguments fixed, and states the
answer MobilityDB gives over the same rows in the time zone UTC, at the MobilityDB commit
vcpkg_ports/meos/portfile.cmake pins (7146de0fba). Each aggregate is called by the name the MEOS
catalog states for it, its altSqlName where it has one. A value of a type whose text carries a
coordinate or a float, the 26 types both engines give an asText with a number of decimal digits,
is written with six decimals in both engines, asText(x, 6): the last digits of a coordinate an
aggregate computes, the extent and the centroid of network points among them, differ between
builds. The test also states the set of aggregate
signatures registered under the catalog's names, which is the catalog's but for the entries over
raster and geography, types MobilityDuck does not have. The point cloud values name pcid 1, whose
schema the test registers as test/sql/tpcpatch.test does.

Measured: MobilityDB answers all 262 queries and MobilityDuck answers each as MobilityDB does,
529 assertions, but wAvgAgg(tbigint), which MobilityDB answers with the bits of its values read as
a double (5e-324 for 1), a MEOS defect the test states in place of the answer.

Why: a test per aggregate stating MobilityDB's answer is what shows an aggregate MobilityDuck
registers computing what MobilityDB computes, and the registered set stated whole is what shows a
change of the catalog reaching MobilityDuck.
@estebanzimanyi
estebanzimanyi force-pushed the test/aggregates-from-catalog branch from 4c702d1 to 4b8988c Compare October 18, 2026 17:24
@estebanzimanyi
estebanzimanyi merged commit 9dca28e into MobilityDB:main Oct 18, 2026
9 checks passed
@estebanzimanyi
estebanzimanyi deleted the test/aggregates-from-catalog branch October 18, 2026 17:49
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