Skip to content

Commit

Permalink
[#6307] feat(flink): remove log4j from Gravitino Flink connector (#6308)
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

remove log4j from Gravitino Flink connector

### Why are the changes needed?

Fix: #6307 

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
setup an local flink cluster, run flink SQL
  • Loading branch information
FANNG1 authored Jan 17, 2025
1 parent 7945692 commit abe75e0
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 2 deletions.
8 changes: 8 additions & 0 deletions flink-connector/flink-runtime/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,14 @@ val scalaVersion: String = "2.12"
val artifactName = "gravitino-${project.name}_$scalaVersion"
val baseName = "${rootProject.name}-flink-connector-runtime-${flinkMajorVersion}_$scalaVersion"

configurations.all {
resolutionStrategy.eachDependency {
if (requested.group == "org.apache.logging.log4j") {
throw GradleException("Forbidden dependency 'org.apache.logging.log4j' found!")
}
}
}

dependencies {
implementation(project(":clients:client-java-runtime", configuration = "shadow"))
implementation(project(":flink-connector:flink"))
Expand Down
8 changes: 6 additions & 2 deletions flink-connector/flink/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,12 @@ val scalaVersion: String = "2.12"
val artifactName = "${rootProject.name}-flink-${flinkMajorVersion}_$scalaVersion"

dependencies {
implementation(project(":core"))
implementation(project(":catalogs:catalog-common"))
implementation(project(":core")) {
exclude("org.apache.logging.log4j")
}
implementation(project(":catalogs:catalog-common")) {
exclude("org.apache.logging.log4j")
}
implementation(libs.guava)

compileOnly(project(":clients:client-java-runtime", configuration = "shadow"))
Expand Down

0 comments on commit abe75e0

Please sign in to comment.