Skip to content

[improve][ci] Run filesystem offload tests also on Java 25 since Hadoop has been upgraded to 3.5.0#25482

Merged
merlimat merged 1 commit intoapache:masterfrom
lhotari:lh-run-filesystem-offload-tests
Apr 7, 2026
Merged

[improve][ci] Run filesystem offload tests also on Java 25 since Hadoop has been upgraded to 3.5.0#25482
merlimat merged 1 commit intoapache:masterfrom
lhotari:lh-run-filesystem-offload-tests

Conversation

@lhotari
Copy link
Copy Markdown
Member

@lhotari lhotari commented Apr 7, 2026

Motivation

Hadoop has been upgraded to 3.5.0, which is compatible with Java 25. The filesystem offload tests were previously skipped on Java 25 due to Hadoop 3.4.x incompatibility. Now that the dependency has been upgraded, these skip conditions are no longer needed.

Modifications

  • Removed the @BeforeClass Java version check that skipped tests on Java 25 in TestFileSystemOffload and TestOffloadDeletionFS.
  • Removed the unused imports (JavaVersion, SystemUtils, SkipException, BeforeClass) from both test classes.
  • Removed the Gradle Test task disabling logic in tiered-storage/file-system/build.gradle.kts that prevented tests from running on Java 25.

Verifying this change

  • Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.

The existing filesystem offload integration tests (TestFileSystemOffload, TestOffloadDeletionFS) and the tiered-storage/file-system unit tests will now run on Java 25, verifying the Hadoop 3.5.0 compatibility.

Does this pull request potentially affect one of the following parts:

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

@merlimat merlimat merged commit 3938609 into apache:master Apr 7, 2026
80 of 82 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants