This is an automated email from the ASF dual-hosted git repository.
mchades pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/main by this push:
new c1daac4089 [#11275] improvement(deps): Upgrade libthrift to 0.12.0 in
remaining modules (#11276)
c1daac4089 is described below
commit c1daac4089114922993302f889271e1324bc4754
Author: geyanggang <[email protected]>
AuthorDate: Fri May 29 15:12:05 2026 +0800
[#11275] improvement(deps): Upgrade libthrift to 0.12.0 in remaining
modules (#11276)
### What changes were proposed in this pull request?
Add `constraints { ... libs.thrift }` in the following modules to
upgrade the transitive libthrift from 0.9.3 to 0.12.0:
- `iceberg/iceberg-common` — `implementation` constraint
- `catalogs/hive-metastore-common` — `compileOnly` +
`testImplementation` constraints
- `flink-connector/flink-common` — `compileOnly` + `testImplementation`
constraints
- `flink-connector/v1.18/flink` — `compileOnly` + `testImplementation`
constraints
- `flink-connector/v1.19/flink` — `compileOnly` + `testImplementation`
constraints
- `flink-connector/v1.20/flink` — `compileOnly` + `testImplementation`
constraints
- `trino-connector/integration-test` — `testImplementation` constraint
### Why are the changes needed?
Follow-up to #11056. libthrift 0.9.3 is still pulled transitively in the
above modules. Version 0.12.0 fixes known issues in SASL negotiation and
large message handling while maintaining binary compatibility with Hive
2.x/3.x.
Fix: #11275
### Does this PR introduce _any_ user-facing change?
No. The Thrift wire protocol is backward-compatible.
### How was this patch tested?
- `iceberg-rest-server` integration tests — pass
- `catalog-hive` integration tests — pass
- `catalog-lakehouse-iceberg` integration tests — pass
- `flink-connector:flink-1.20` integration tests — pass
- `hive-metastore-common` integration tests — pass
- Verified all affected modules resolve libthrift to 0.12.0 via
`./gradlew :module:dependencies`
---
catalogs/hive-metastore-common/build.gradle.kts | 6 ++++++
flink-connector/flink-common/build.gradle.kts | 6 ++++++
flink-connector/v1.18/flink/build.gradle.kts | 6 ++++++
flink-connector/v1.19/flink/build.gradle.kts | 6 ++++++
flink-connector/v1.20/flink/build.gradle.kts | 6 ++++++
iceberg/iceberg-common/build.gradle.kts | 5 +++++
trino-connector/integration-test/build.gradle.kts | 5 +++++
7 files changed, 40 insertions(+)
diff --git a/catalogs/hive-metastore-common/build.gradle.kts
b/catalogs/hive-metastore-common/build.gradle.kts
index 10db17f962..736ce064df 100644
--- a/catalogs/hive-metastore-common/build.gradle.kts
+++ b/catalogs/hive-metastore-common/build.gradle.kts
@@ -24,6 +24,12 @@ plugins {
}
dependencies {
+ // Force upgrade for outdated transitive libthrift pulled by Hive Metastore
+ constraints {
+ compileOnly(libs.thrift)
+ testImplementation(libs.thrift)
+ }
+
compileOnly(project(":api"))
compileOnly(project(":common"))
compileOnly(project(":core"))
diff --git a/flink-connector/flink-common/build.gradle.kts
b/flink-connector/flink-common/build.gradle.kts
index a8803808b1..6e121d52a7 100644
--- a/flink-connector/flink-common/build.gradle.kts
+++ b/flink-connector/flink-common/build.gradle.kts
@@ -40,6 +40,12 @@ val scalaVersion: String = "2.12"
val artifactName = "${rootProject.name}-flink-common"
dependencies {
+ // Force upgrade for outdated transitive libthrift pulled by Hive Metastore
+ constraints {
+ compileOnly(libs.thrift)
+ testImplementation(libs.thrift)
+ }
+
implementation(project(":catalogs:catalog-common")) {
exclude("org.apache.logging.log4j")
}
diff --git a/flink-connector/v1.18/flink/build.gradle.kts
b/flink-connector/v1.18/flink/build.gradle.kts
index c8d59d0241..83841d02e7 100644
--- a/flink-connector/v1.18/flink/build.gradle.kts
+++ b/flink-connector/v1.18/flink/build.gradle.kts
@@ -39,6 +39,12 @@ val scalaVersion: String = "2.12"
val artifactName =
"${rootProject.name}-flink-${flinkMajorVersion}_$scalaVersion"
dependencies {
+ // Force upgrade for outdated transitive libthrift pulled by Hive Metastore
+ constraints {
+ compileOnly(libs.thrift)
+ testImplementation(libs.thrift)
+ }
+
implementation(commonProject)
compileOnly(project(":clients:client-java-runtime", configuration =
"shadow"))
diff --git a/flink-connector/v1.19/flink/build.gradle.kts
b/flink-connector/v1.19/flink/build.gradle.kts
index 9ca0201df6..755da86ada 100644
--- a/flink-connector/v1.19/flink/build.gradle.kts
+++ b/flink-connector/v1.19/flink/build.gradle.kts
@@ -39,6 +39,12 @@ val scalaVersion: String = "2.12"
val artifactName =
"${rootProject.name}-flink-${flinkMajorVersion}_$scalaVersion"
dependencies {
+ // Force upgrade for outdated transitive libthrift pulled by Hive Metastore
+ constraints {
+ compileOnly(libs.thrift)
+ testImplementation(libs.thrift)
+ }
+
implementation(commonProject)
compileOnly(project(":clients:client-java-runtime", configuration =
"shadow"))
diff --git a/flink-connector/v1.20/flink/build.gradle.kts
b/flink-connector/v1.20/flink/build.gradle.kts
index bbcd018e1c..81e1c421d2 100644
--- a/flink-connector/v1.20/flink/build.gradle.kts
+++ b/flink-connector/v1.20/flink/build.gradle.kts
@@ -39,6 +39,12 @@ val scalaVersion: String = "2.12"
val artifactName =
"${rootProject.name}-flink-${flinkMajorVersion}_$scalaVersion"
dependencies {
+ // Force upgrade for outdated transitive libthrift pulled by Hive Metastore
+ constraints {
+ compileOnly(libs.thrift)
+ testImplementation(libs.thrift)
+ }
+
implementation(commonProject)
compileOnly(project(":clients:client-java-runtime", configuration =
"shadow"))
diff --git a/iceberg/iceberg-common/build.gradle.kts
b/iceberg/iceberg-common/build.gradle.kts
index 16d1963a4b..bd0bd45f92 100644
--- a/iceberg/iceberg-common/build.gradle.kts
+++ b/iceberg/iceberg-common/build.gradle.kts
@@ -25,6 +25,11 @@ plugins {
}
dependencies {
+ // Force upgrade for outdated transitive libthrift pulled by Hive Metastore
+ constraints {
+ implementation(libs.thrift)
+ }
+
implementation(project(":api"))
implementation(project(":catalogs:catalog-common"))
implementation(project(":catalogs:hadoop-common")) {
diff --git a/trino-connector/integration-test/build.gradle.kts
b/trino-connector/integration-test/build.gradle.kts
index 5110e8e949..f66c9e3649 100644
--- a/trino-connector/integration-test/build.gradle.kts
+++ b/trino-connector/integration-test/build.gradle.kts
@@ -28,6 +28,11 @@ repositories {
}
dependencies {
+ // Force upgrade for outdated transitive libthrift pulled by Hive Metastore
+ constraints {
+ testImplementation(libs.thrift)
+ }
+
testImplementation(project(":api"))
testImplementation(project(":clients:client-java"))
testImplementation(project(":common"))
