This is an automated email from the ASF dual-hosted git repository.
voonhous pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hudi.git
The following commit(s) were added to refs/heads/master by this push:
new 7a3e2f3ad69b fix(tests): reach TFramedTransport at the package
libthrift 0.14 moved it to (#19830)
7a3e2f3ad69b is described below
commit 7a3e2f3ad69b7b6727e36c9dc64acdda89bc7ffe
Author: Y Ethan Guo <[email protected]>
AuthorDate: Wed Sep 2 21:06:23 2026 -0700
fix(tests): reach TFramedTransport at the package libthrift 0.14 moved it
to (#19830)
HiveTestService imports TFramedTransport from the pre-0.14 package while
overriding TServerTransport.accept(), which is final below 0.14, so no
single
libthrift satisfies it. The build only gets away with it because the full
hive-exec bundles a 0.14-era transport package; consumers of the tests jar
that
take hive-exec with the core classifier resolve TServerTransport from
libthrift
and fail verification.
The libthrift test dependency moves to hive.libthrift.version, which already
models the thrift the Hive client is built against, leaving thrift.version
to
the metaserver.
---
hudi-sync/hudi-hive-sync/pom.xml | 2 +-
.../src/test/java/org/apache/hudi/hive/testutils/HiveTestService.java | 2 +-
hudi-utilities/pom.xml | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/hudi-sync/hudi-hive-sync/pom.xml b/hudi-sync/hudi-hive-sync/pom.xml
index 45b8686edb54..1823cde2ca90 100644
--- a/hudi-sync/hudi-hive-sync/pom.xml
+++ b/hudi-sync/hudi-hive-sync/pom.xml
@@ -163,7 +163,7 @@
<dependency>
<groupId>org.apache.thrift</groupId>
<artifactId>libthrift</artifactId>
- <version>${thrift.version}</version>
+ <version>${hive.libthrift.version}</version>
<scope>test</scope>
</dependency>
diff --git
a/hudi-sync/hudi-hive-sync/src/test/java/org/apache/hudi/hive/testutils/HiveTestService.java
b/hudi-sync/hudi-hive-sync/src/test/java/org/apache/hudi/hive/testutils/HiveTestService.java
index e3d53932461a..2bfcfbb47af7 100644
---
a/hudi-sync/hudi-hive-sync/src/test/java/org/apache/hudi/hive/testutils/HiveTestService.java
+++
b/hudi-sync/hudi-hive-sync/src/test/java/org/apache/hudi/hive/testutils/HiveTestService.java
@@ -40,13 +40,13 @@ import org.apache.thrift.TProcessor;
import org.apache.thrift.protocol.TBinaryProtocol;
import org.apache.thrift.server.TServer;
import org.apache.thrift.server.TThreadPoolServer;
-import org.apache.thrift.transport.TFramedTransport;
import org.apache.thrift.transport.TServerSocket;
import org.apache.thrift.transport.TServerTransport;
import org.apache.thrift.transport.TSocket;
import org.apache.thrift.transport.TTransport;
import org.apache.thrift.transport.TTransportException;
import org.apache.thrift.transport.TTransportFactory;
+import org.apache.thrift.transport.layered.TFramedTransport;
import java.io.File;
import java.io.IOException;
diff --git a/hudi-utilities/pom.xml b/hudi-utilities/pom.xml
index ce64846a6534..1ccd29b30f72 100644
--- a/hudi-utilities/pom.xml
+++ b/hudi-utilities/pom.xml
@@ -601,7 +601,7 @@
<dependency>
<groupId>org.apache.thrift</groupId>
<artifactId>libthrift</artifactId>
- <version>${thrift.version}</version>
+ <version>${hive.libthrift.version}</version>
<scope>test</scope>
</dependency>
</dependencies>
