paleolimbot commented on PR #1108:
URL: https://github.com/apache/sedona-db/pull/1108#issuecomment-5171797826

   > I believe null for a SRID is the correct value where the CRS does not have 
a SRID.
   
   It's true that there's no other way to check if an item has a CRS that is 
identifiable (we could add 
   
   > Users cannot catch a per-row error so we should be extremely cautious 
about letting a single row fail a whole query.
   
   Yes, but they may get bogus results. For example `SELECT 
ST_Intersects(some_col, ST_SetSRID(other_col, ST_SRID(other_col)) FROM foofy` 
will silently omit rows where `some_col` has an unidentifiable result instead 
of erroring and refusing to return a possibly meaningless result.
   
   We could also put in the documentation of `ST_SRID()` to use `ST_Crs()` and 
`ST_SetCrs()` instead.


-- 
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]

Reply via email to