Skip to content

update hadoop-aws from hadoop3-3.1.4 to hadoop3-3.3.4#10676

Open
PenneyHuang wants to merge 1 commit into
apache:devfrom
PenneyHuang:dev-updata-hadoop-aws-version
Open

update hadoop-aws from hadoop3-3.1.4 to hadoop3-3.3.4#10676
PenneyHuang wants to merge 1 commit into
apache:devfrom
PenneyHuang:dev-updata-hadoop-aws-version

Conversation

@PenneyHuang
Copy link
Copy Markdown

Purpose of this pull request

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

@davidzollo davidzollo added the First-time contributor First-time contributor label Mar 31, 2026
@davidzollo
Copy link
Copy Markdown
Contributor

Thanks for your first contribution.

Please open the CI check as stated at https://github.com/apache/seatunnel/pull/10676/checks?check_run_id=69288452923

@DanielLeens
Copy link
Copy Markdown
Contributor

I fetched this PR locally. The Hadoop 3.3.4 upgrade itself looks straightforward, but I found one regression in the engine server module that should be fixed before merge.

In seatunnel-engine/seatunnel-engine-server/pom.xml, the change drops <scope>test</scope> from com.squareup.okhttp:mockwebserver while keeping the dependency used only in src/test/java/org/apache/seatunnel/engine/server/event/JobEventHttpReportHandlerTest.java. That promotes a pure test helper into the main runtime dependency set of seatunnel-engine-server, which will unnecessarily leak it into the shipped server artifact and downstream dependency graph.

Please keep mockwebserver as a test-scoped dependency. The module-specific compiler-plugin override added in the same diff also looks suspicious and should be justified separately if it is really required for this upgrade.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core SeaTunnel core module document e2e First-time contributor First-time contributor Zeta

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants