Repository navigation
[VL] Fix Java tests not test #10011
Description
Activity
jinchengchenghh commented
on Jun 19, 2025 ContributorAuthorMore actions[INFO] Running org.apache.gluten.fs.OnHeapFileSystemTest [ERROR] Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0 s <<< FAILURE! -- in org.apache.gluten.fs.OnHeapFileSystemTest [ERROR] org.apache.gluten.fs.OnHeapFileSystemTest.testRoundTrip -- Time elapsed: 0 s <<< FAILURE! java.lang.AssertionError at io.netty.util.internal.PlatformDependent.allocateDirectNoCleaner(PlatformDependent.java:729) at org.apache.gluten.fs.OnHeapFileSystemTest.testRoundTrip(OnHeapFileSystemTest.java:38) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) at java.base/java.lang.reflect.Method.invoke(Method.java:569)@apache/gluten-committers
Guys, please pause on merging PRs at the moment. Let's get this issue fixed first, otherwise all Java tests may not be run on CI.
We don't have many Java tests so this should not take long to fix.
What's the issue?
@FelixYBW Could be related to some pom settings on surefire option 'skipTests', we can do a trivial fix to enable the base test first.
@jinchengchenghh I ran locally all the Java tests (except for CH backend) and saw some Iceberg test failures:
[ERROR] Failures: [ERROR] org.apache.iceberg.spark.source.TestSparkReaderDeletes.testMultiplePosDeleteFiles [ERROR] Run 1: TestSparkReaderDeletes>DeleteReadTests.testMultiplePosDeleteFiles:445->DeleteReadTests.checkDeleteCount:211 [Table should contain expected number of deletes] expected: 3L but was: 0L [ERROR] Run 2: TestSparkReaderDeletes>DeleteReadTests.testMultiplePosDeleteFiles:445->DeleteReadTests.checkDeleteCount:211 [Table should contain expected number of deletes] expected: 3L but was: 0L [ERROR] Run 3: TestSparkReaderDeletes>DeleteReadTests.testMultiplePosDeleteFiles:445->DeleteReadTests.checkDeleteCount:211 [Table should contain expected number of deletes] expected: 3L but was: 0L(Hopefully they are not related to my work on #9996)
We may be able to disable these tests first (add @ignore on the classes) and send a PR to enable other Java tests first? To unblock other ongoing PRs from merging.
@apache/gluten-committers
I had a local test and the Java tests are basically still passing (except for some iceberg test), it should be fine to continue on merging PRs, sorry for the waiting here.
Backend
VL (Velox)
Bug description