Skip to content

Conversation

@dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Oct 2, 2025

What changes were proposed in this pull request?

This PR aims to use log4j2 instead of log4j by

  • Use log4j-slf4j2-impl instead of log4j-slf4j-impl
  • Remove log4j-1.2-api

Why are the changes needed?

Apache Spark main repository has been using log4j-slf4j2-impl instead of log4j-slf4j-impl in order to use Log4J2.

Apache Spark K8s Operator repository seems to use log4j-slf4j2-impl mistakenly at the initial implementation.

implementation("org.apache.logging.log4j:log4j-slf4j-impl:$log4jVersion")

Does this PR introduce any user-facing change?

No behavior change.

How was this patch tested?

Pass the CIs.

Was this patch authored or co-authored using generative AI tooling?

No.

@dongjoon-hyun dongjoon-hyun marked this pull request as draft October 2, 2025 04:19
@dongjoon-hyun dongjoon-hyun marked this pull request as ready for review October 2, 2025 04:27
@dongjoon-hyun
Copy link
Member Author

Let me merge this to unblock other dependency fix PRs.

@dongjoon-hyun dongjoon-hyun deleted the SPARK-53783 branch October 2, 2025 04:35
@dongjoon-hyun
Copy link
Member Author

cc @jiangzho

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.

1 participant