paleolimbot commented on issue #464: URL: https://github.com/apache/sedona-db/issues/464#issuecomment-3665876073
Thanks for filing! This one is tricky because we need to declare an output type based on the input arguments and this function is frequently used. I'm inclined to have `ST_GeogFromText()` default to returning a type-level CRS with 4326 because that's almost always is what people mean (`ST_GeomFromText()` should maybe stick with a missing type-level CRS like it does now...not sure if that one parses EWKT too). We could also have `ST_GeogFromText(<string>)` return an `item_crs` but that would probably be annoying since it would force everything that it interacts with into item level CRS types. ...or we could just silently ignore EWKT srids. Probably it would be better user experience to thrown an error along the lines of `Use ST_GeogFromEWKT()` (and add that function). I should have the item-level CRS type finished up soon. -- 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]
