Copilot commented on code in PR #2855:
URL: https://github.com/apache/sedona/pull/2855#discussion_r3125629146
##########
spark/common/src/test/scala/org/apache/sedona/sql/geography/GeographyFunctionTest.scala:
##########
@@ -141,6 +141,30 @@ class GeographyFunctionTest extends TestBaseScala {
.first()
assertTrue(!row.getBoolean(0))
}
+
+ it("ST_Equals same point") {
+ val row = sparkSession
+ .sql("""
+ SELECT ST_Equals(
+ ST_GeogFromWKT('POINT (1 2)', 4326),
+ ST_GeogFromWKT('POINT (1 2)', 4326)
+ ) AS result
Review Comment:
The new Geography ST_Equals coverage only exercises point/point via Spark
SQL. Since ST_Equals is intended to use S2 boolean ops for arbitrary regions,
please add at least one non-point (e.g., polygon) Spark SQL case and a
DataFrame API case (similar to the existing ST_Contains DataFrame tests) to
ensure the Geography dispatch works across both APIs and common geometry kinds.
--
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]
