2010YOUY01 commented on PR #526:
URL: https://github.com/apache/sedona-db/pull/526#issuecomment-3770970934
> Thank you for opening!
>
> I think we need to actually expose this still (either via the argument or
via a separate function like `ST_GeomFromWKBUnchecked()` or `sd_wrap_wkb()`).
>
> ```python
> import sedona.db
>
> sd = sedona.db.connect()
>
> sd.sql("SELECT ST_GeomFromWkb(ST_AsBinary(ST_GeomFromText('POINT (0 1)')),
false) as s").show()
> #> SedonaError: st_geomfromwkb([Arrow(Binary), Arrow(Boolean)]): No kernel
matching arguments
> ```
Sorry for the confusion. I mistakenly thought it was already exposed due to
some local API changes 🤦🏼
I think using an additional API would be better than adding an argument:
- it makes the unsafe operation more explicit;
- it also aligns better with PostGIS as a reference system, and they don’t
have such an argument: https://postgis.net/docs/ST_GeomFromWKB.html
I’ll open another PR for this.
--
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]
