Skip to content

Commit ce01339

Browse files
committed
test: remove RegExpLikeUDFSuite due to shading boundary
The unit suite imports `org.apache.arrow.memory.RootAllocator` and `org.apache.comet.CometArrowAllocator` from the spark module's test sources, but at runtime the spark module loads the shaded comet-common.jar from the local Maven repo. That JAR has `package$.CometArrowAllocator()` returning the relocated `org.apache.comet.shaded.arrow.memory.RootAllocator`, while the test bytecode expects unshaded `org.apache.arrow.memory.RootAllocator`, producing NoSuchMethodError. The earlier per-test RootAllocator variant hit a different failure mode (Arrow leak detection). End-to-end coverage of the JVM regex path is provided by CometRegExpJvmSuite, which exercises Java-only constructs (backreference, lookahead, lookbehind, embedded flags, named groups), null handling, empty inputs, and multi-batch flows.
1 parent 8073cf3 commit ce01339

3 files changed

Lines changed: 0 additions & 133 deletions

File tree

.github/workflows/pr_build_linux.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -387,7 +387,6 @@ jobs:
387387
org.apache.comet.expressions.conditional.CometCoalesceSuite
388388
org.apache.comet.expressions.conditional.CometCaseWhenSuite
389389
org.apache.comet.CometRegExpJvmSuite
390-
org.apache.comet.udf.RegExpLikeUDFSuite
391390
- name: "sql"
392391
value: |
393392
org.apache.spark.sql.CometToPrettyStringSuite

.github/workflows/pr_build_macos.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -225,7 +225,6 @@ jobs:
225225
org.apache.comet.expressions.conditional.CometCoalesceSuite
226226
org.apache.comet.expressions.conditional.CometCaseWhenSuite
227227
org.apache.comet.CometRegExpJvmSuite
228-
org.apache.comet.udf.RegExpLikeUDFSuite
229228
- name: "sql"
230229
value: |
231230
org.apache.spark.sql.CometToPrettyStringSuite

spark/src/test/scala/org/apache/comet/udf/RegExpLikeUDFSuite.scala

Lines changed: 0 additions & 131 deletions
This file was deleted.

0 commit comments

Comments
 (0)