Skip to content

test(processing): avoid JDK crash in schemaless segment test - #20121

Open
FrankChen021 wants to merge 3 commits into
apache:masterfrom
FrankChen021:codex/schemaless-junit5-lifecycle-pr
Open

test(processing): avoid JDK crash in schemaless segment test#20121
FrankChen021 wants to merge 3 commits into
apache:masterfrom
FrankChen021:codex/schemaless-junit5-lifecycle-pr

Conversation

@FrankChen021

@FrankChen021 FrankChen021 commented Aug 22, 2026

Copy link
Copy Markdown
Member

Summary

  • Migrate SchemalessTestSimpleTest from class-level JUnit 4 parameterization to method-level JUnit 5 @ParameterizedTest.
  • Create fresh incremental and merged fixtures for each method-source invocation.
  • Use try-with-resources in each test method so every test invocation owns and closes its Segment.
  • Close intermediate mapped indexes while retaining the returned merged index for the owning segment.

JDK crash context

This change addresses the intermittent direct-buffer JVM crash observed in the crash reproduction PR #20028, which affected SchemalessTestSimpleTest on JDK 25.

Validation

  • Processing test-compile passed.
  • Focused test passed after the lifecycle change: 36 active executions; the existing disabled TopN test remains skipped.
  • The equivalent lifecycle workaround was also exercised in 100 separate Maven invocations on Temurin JDK 25 with Surefire retries disabled, with no JDK crash.
  • Checkstyle, SpotBugs, and git diff --check passed.

This PR is intended to validate whether explicit fixture ownership and cleanup prevent the intermittent JDK direct-buffer crash.

Copilot AI lite review requested due to automatic review settings August 22, 2026 02:20

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@FrankChen021 FrankChen021 changed the title test(processing): isolate schemaless test segments test(processing): avoid JDK crash in schemaless segment test Aug 22, 2026
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.

3 participants