You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "uros-db (via GitHub)" <gi...@apache.org> on 2026/02/07 00:56:36 UTC

[PR] [SPARK][Geo][SQL] Enable WKB parsing for Geography [spark]

uros-db opened a new pull request, #54193:
URL: https://github.com/apache/spark/pull/54193

   ### What changes were proposed in this pull request?
   Implement Geography coordinate validation in WKB parser:
   
   - Longitude must be between -180 and 180 (inclusive).
   - Latitude must be between -90 and 90 (inclusive).
   
   
   ### Why are the changes needed?
   Enable Geography parsing from WKB.
   
   
   ### Does this PR introduce _any_ user-facing change?
   Yes, Geography coordinate values are now limited to the supported ranges for longitude and latitude.
   
   
   ### How was this patch tested?
   Added new unit tests.
   
   
   ### Was this patch authored or co-authored using generative AI tooling?
   Yes.
   


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Re: [PR] [SPARK][Geo][SQL] Enable WKB parsing for Geography [spark]

Posted by "uros-db (via GitHub)" <gi...@apache.org>.
uros-db closed pull request #54193: [SPARK][Geo][SQL] Enable WKB parsing for Geography
URL: https://github.com/apache/spark/pull/54193


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Re: [PR] [SPARK][Geo][SQL] Enable WKB parsing for Geography [spark]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #54193:
URL: https://github.com/apache/spark/pull/54193#issuecomment-3863214261

   ## ⚠️ Pull Request Title Validation
   
   This pull request title does not contain a JIRA issue ID.
   
   Please update the title to either:
   - Include a JIRA ID: `[SPARK-12345] Your description`
   - Mark as minor change: `[MINOR] Your description`
   
   For minor changes that don't require a JIRA ticket (e.g., typo fixes), please prefix the title with `[MINOR]`.
   
   ---
   *This comment was automatically generated by GitHub Actions*


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org