paleolimbot opened a new issue, #501: URL: https://github.com/apache/sedona-db/issues/501
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 a `CrsEngine` such that it could be used for validation but this isn't actually used when we register the functions currently. Forthcoming functions like `ST_GeomFromEWKT()` and `ST_GeomFromEKWB()` will need this too ( https://github.com/apache/sedona-db/pull/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: https://github.com/apache/sedona-db/issues/248 ). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
