Repository navigation
Call the MEOS functions whose name Flink owns by their class-prefixed name - #79
Merged
estebanzimanyi merged 1 commit intoOct 11, 2026
Conversation
… name Flink reserves set and unnest and defines lower, so the generated surfaces register the MEOS functions under the name the catalog states for Flink and Spark (altSqlName): spanLower for lower(floatspan), setMake for set(integer[]), setUnnest for unnest(intset), temporalUnnest for unnest(tint), geoTransform for transform(tgeompoint, integer). The SQL surface test calls them by these names, unquoted, and checks that lower over a floatspan is not a MEOS function in Flink, unqualified or qualified by the catalog. The module test checks the same with the module loaded, listed after core or before it: lower stays Flink's own and spanLower answers from the module. The test also states two queries of the flights dashboards: a temporal point transformed to ETRS89-LAEA (SRID 3035) keeps its instants and takes the new SRID, and stboxOverlaps, the function of && between a temporal point and the box expandSpace gives another, answers true for a box widened past the distance between the two trips and false for one short of it.
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.
Flink reserves set and unnest and defines lower, so the generated surfaces register the MEOS
functions under the name the catalog states for Flink and Spark (altSqlName): spanLower for
lower(floatspan), setMake for set(integer[]), setUnnest for unnest(intset), temporalUnnest for
unnest(tint), geoTransform for transform(tgeompoint, integer). The SQL surface test calls them by
these names, unquoted, and checks that lower over a floatspan is not a MEOS function in Flink,
unqualified or qualified by the catalog. The module test checks the same with the module loaded,
listed after core or before it: lower stays Flink's own and spanLower answers from the module.
The test also states two queries of the flights dashboards: a temporal point transformed to
ETRS89-LAEA (SRID 3035) keeps its instants and takes the new SRID, and stboxOverlaps, the
function of && between a temporal point and the box expandSpace gives another, answers true
for a box widened past the distance between the two trips and false for one short of it.