Skip to content

rust/sedona-functions: Improve CRS validation #501

@paleolimbot

Description

@paleolimbot

Several functions, including ST_SetCrs(), ST_SetSRID(), and constructors with an SRID argument like ST_Point()don't currently validate their input. For some of them I added the ability to register them with aCrsEnginesuch that it could be used for validation but this isn't actually used when we register the functions currently. Forthcoming functions likeST_GeomFromEWKT()andST_GeomFromEKWB()` will need this too ( #498 ).

We can either lean in to CrsEngine validation (registered with a kernel) or try to provide something via the ConfigOptions or a thread local. (The ConfigOptions are now available in some places in ScalarUDF execution but we haven't wired them into SedonaDB kernels yet: #248 ).

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions