You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sedona.apache.org by ji...@apache.org on 2026/08/07 04:42:30 UTC
(sedona) branch master updated: [CI] Fix Snowflake V2 empty polygon orientation tests (#3242)
This is an automated email from the ASF dual-hosted git repository.
jiayuasu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sedona.git
The following commit(s) were added to refs/heads/master by this push:
new 456648883f7 [CI] Fix Snowflake V2 empty polygon orientation tests (#3242)
456648883f7 is described below
commit 456648883f77cd309754a2934f865ac2a0fcba15
Author: Jia Yu <ji...@apache.org>
AuthorDate: Thu Aug 6 21:42:24 2026 -0700
[CI] Fix Snowflake V2 empty polygon orientation tests (#3242)
---
.../org/apache/sedona/snowflake/snowsql/TestFunctionsV2.java | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/snowflake-tester/src/test/java/org/apache/sedona/snowflake/snowsql/TestFunctionsV2.java b/snowflake-tester/src/test/java/org/apache/sedona/snowflake/snowsql/TestFunctionsV2.java
index ce24b3a6210..73c2da9d15f 100644
--- a/snowflake-tester/src/test/java/org/apache/sedona/snowflake/snowsql/TestFunctionsV2.java
+++ b/snowflake-tester/src/test/java/org/apache/sedona/snowflake/snowsql/TestFunctionsV2.java
@@ -1211,8 +1211,9 @@ public class TestFunctionsV2 extends TestBase {
verifySqlSingleRes(
"SELECT sedona.ST_IsPolygonCW(ST_GeomFromWKT('POLYGON ((20 35, 45 20, 30 5, 10 10, 10 30, 20 35), (30 20, 20 25, 20 15, 30 20))'))",
true);
- verifySqlSingleRes("SELECT sedona.ST_IsPolygonCW(ST_GeomFromWKT('POLYGON EMPTY'))", true);
- verifySqlSingleRes("SELECT sedona.ST_IsPolygonCW(ST_GeomFromWKT('MULTIPOLYGON EMPTY'))", true);
+ // Empty geometries cannot be tested here: Snowflake's native GEOMETRY parses
+ // 'POLYGON EMPTY' to NULL, so the UDF never receives an empty geometry.
+ // Empty inputs are covered by the WKB-based tests in TestFunctions.
}
@Test
@@ -1244,8 +1245,9 @@ public class TestFunctionsV2 extends TestBase {
verifySqlSingleRes(
"SELECT sedona.ST_IsPolygonCCW(ST_GeomFromWKT('POLYGON ((20 35, 10 30, 10 10, 30 5, 45 20, 20 35),(30 20, 20 15, 20 25, 30 20))'))",
true);
- verifySqlSingleRes("SELECT sedona.ST_IsPolygonCCW(ST_GeomFromWKT('POLYGON EMPTY'))", true);
- verifySqlSingleRes("SELECT sedona.ST_IsPolygonCCW(ST_GeomFromWKT('MULTIPOLYGON EMPTY'))", true);
+ // Empty geometries cannot be tested here: Snowflake's native GEOMETRY parses
+ // 'POLYGON EMPTY' to NULL, so the UDF never receives an empty geometry.
+ // Empty inputs are covered by the WKB-based tests in TestFunctions.
}
@Test