You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@sedona.apache.org by "ZhuochengShang (via GitHub)" <gi...@apache.org> on 2025/08/08 18:31:20 UTC

[PR] [GH-]new Geography constructor from point, polyline, polygon [sedona]

ZhuochengShang opened a new pull request, #2254:
URL: https://github.com/apache/sedona/pull/2254

   
   ## What changes were proposed in this PR?
   Modify GeogFromWKB so that it supports both input as String and Byte[]
   Add new constructor functions :
   
   ST_S2PointFromText
   ST_S2LineFromText
   ST_S2LinestringFromText
   ST_S2PolyFromText
   
   ST_S2MPointFromText
   ST_S2MLineFromText
   ST_S2MPolyFromText
   
   ST_S2PointFromWKB
   ST_S2LineFromWKT
   


-- 
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: issues-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [GH-2255]new Geography constructor from point, polyline, polygon [sedona]

Posted by "jiayuasu (via GitHub)" <gi...@apache.org>.
jiayuasu commented on PR #2254:
URL: https://github.com/apache/sedona/pull/2254#issuecomment-3169346215

   I don't think we need constructors like ST_PointFromText because there is no way we can differentiate it from the geometry one.
   
   Please remove these from this PR and instead let's focus on the following counterpart functions of the geometry side (https://sedona.apache.org/latest/api/sql/Constructor/).
   
   * ST_GeogFromWKB (the Binary type input, not the string type input)
   * ST_GeogFromEWKB (the Binary type input, not the string type input)
   * ST_GeogFromEWKT
   * ST_GeogFromGeohash
   * ST_GeogFromText (alias to ST_GeogFromWKT)
   
   Let's start with ST_GeogFromWKB. Please do one function per PR.
   


-- 
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: issues-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [GH-2255]new Geography constructor from point, polyline, polygon [sedona]

Posted by "ZhuochengShang (via GitHub)" <gi...@apache.org>.
ZhuochengShang closed pull request #2254: [GH-2255]new Geography constructor from point, polyline, polygon
URL: https://github.com/apache/sedona/pull/2254


-- 
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: issues-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org