Advance the MEOS commit and regenerate the UDF surface - #347
Merged
Conversation
estebanzimanyi
force-pushed
the
tooling/refresh-generated-surface
branch
from
August 28, 2026 11:13
4d5ca12 to
eb78a98
Compare
estebanzimanyi
force-pushed
the
tooling/refresh-generated-surface
branch
from
August 28, 2026 12:51
eb78a98 to
caa0483
Compare
DuckDB routes scalar and aggregate functions into one catalog set, so a name holds either a scalar or an aggregate, never both. The refreshed surface publishes tAnd and tOr as scalars, which MobilityDB declares over (tbool,tbool) and the boolean-mixed pairs, so those aggregates answer to TandAgg and TorAgg alone, as merge, tMin, tMax, appendInstant, appendSequence and minDistance already do. An aggregate registered onto a name a scalar holds reaches AggregateFunctionCatalogEntry, which carries no AlterEntry override, and the extension fails to load with "Cannot AlterEntry without client context". The npoint position accessor is addressed as getPosition, the name the surface publishes.
MobilityDB answers getResolution, cellArea, cellToParent, cellToPoint and isValidCell for the temporal cell indexes from one tcellindex kernel, under the bare slot names. The kernel carries no family token, so the generator's name heuristic scoped it to no type and the whole group went unregistered; the catalog's own sqlSignatures name the overloads each slot is created for, so the kernel defers to them. cellToPoint is quadbin's alone, H3 answering it geodetically from its own kernel, and cellToBoundary stays unregistered because its answer leaves the family - tgeography for H3, tgeometry for quadbin - which the return heuristic states as the operand's own type. meos_cellindex.h joins the headers the binding wraps, the kernel being declared there.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The committed
src/generated/drifted from a fresh derivation againstMobilityDB/MobilityDB master; this regenerates it and moves
_MEOS_REFinvcpkg_ports/meos/portfile.cmaketo the commit it isderived from, so libmeos and the surface stay one commit.
Opened by the
Refresh generated surface from MobilityDB masterworkflow.