Skip to content

Conversation

@jgoday
Copy link
Contributor

@jgoday jgoday commented Apr 25, 2025

Did you read the Contributor Guide?

Is this PR related to a ticket?

What changes were proposed in this PR?

Throws InvalidArgumentException when maxVertices is lower that 5.

How was this patch tested?

  • Created 'subdivideInvalidMaxVertices' test in common/src/test/java/org/apache/sedona/common/FunctionsTest.java
  • Created test in in spark/common/src/test/scala/org/apache/sedona/sql/functionTestScala.scala

Did this PR include necessary documentation updates?

  • Yes, I have updated the documentation (ST_Subdivide/ST_SubdivideExplode) in both docs/api/sql/Function.md and docs/api/flink/Function.md

@jgoday jgoday force-pushed the st_subdivide_max_vertices branch from c8a9400 to 7ee5471 Compare April 25, 2025 19:17
@jiayuasu jiayuasu added this to the sedona-1.8.0 milestone Apr 25, 2025
@jiayuasu jiayuasu added the bug label Apr 25, 2025
@jgoday jgoday force-pushed the st_subdivide_max_vertices branch from 7ee5471 to 158c3c2 Compare April 25, 2025 19:53
@jiayuasu
Copy link
Member

Don't worry about the failed Python build extension test. We will fix it in another PR.

@jiayuasu jiayuasu merged commit e2a8d63 into apache:master Apr 26, 2025
30 of 39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants