Bump Thrift to 0.25.0

[HTHou]

Description

• Upgrade Apache Thrift Java runtime from 0.24.0 to 0.25.0.
• Upgrade iotdb-tools-thrift from 0.23.0.0 to 0.25.0.0.
• Add the Apache staging repository orgapacheiotdb-1203 so CI can resolve the release candidate.
• Update LICENSE-binary accordingly.

Tests

• GitHub Actions CI.

[Copilot]

Copilot review overview

🟡 Changes recommended

Frame-bound allocation checks remain ineffective, and the CI matrix adds a deprecated runner.

Review effort: Balanced
Findings: 1 · 1

Open (2)

• Enforce frame-sized budget for protocol allocations · New
• Replace deprecated ubuntu-22.04-arm runner with ubuntu-24.04-arm · New

What changed in this PR

Upgrades the Java Thrift runtime and compiler tooling to 0.25.0 while adapting custom transports and CI.

Changes:

• Bumps Thrift dependencies and licensing metadata.
• Adds message-budget reset support and regression coverage.
• Expands multi-platform client CI.

File	Description
pom.xml	Updates Thrift versions and adds staging repository.
LICENSE-binary	Updates bundled libthrift version.
TElasticFramedTransport.java	Resets frame message-size budgets.
NonOpenTransport.java	Implements the new transport API method.
TElasticFramedTransportTest.java	Tests consecutive frame reads.
multi-language-client.yml	Adds ARM CI and architecture-specific caching.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Copilot review overview

🟡 Changes recommended

The compressed/Snappy transport bypasses the new per-frame budget reset and can fail on long-lived connections.

Review effort: Balanced
Findings: 1

Open (1)

• Compressed frames skip budget resets and eventually exhaust message budget · New

Resolved since last review (2)

• Enforce frame-sized budget for protocol allocations
• Replace deprecated ubuntu-22.04-arm runner with ubuntu-24.04-arm

[Copilot]

Copilot review overview

🔵 Needs a closer look

Root-POM-only updates still skip the C++ job because the internal path detector omits pom.xml.

Review effort: Balanced
Findings: 1

Open (1)

• Compressed frames skip budget resets and eventually exhaust message budget

Previously missed (1)

In code that hasn't changed since last review

Include pom.xml in C++ changed-path detection

.github/​workflows/​multi-language-client.yml:9

Adding pom.xml only to the workflow-level path filter does not enable C++ validation for a root-POM-only change: the cpp case in Detect changed client paths still omits pom.xml, so updates such as iotdb-tools-thrift.version start this workflow but leave the C++ job skipped. Add pom.xml to that case as well.

[Copilot]

Copilot review overview

🟡 Changes recommended

Message-budget limits can reject valid large frames, and the new tests do not reliably detect missing resets.

Review effort: Balanced
Findings: 2 · 1

Open (3)

• Align compressed transport frame limit with endpoint configuration · New
• Reconcile endpoint and configured Thrift frame size limits · New
• Use budget-consuming transport to verify frame budget resets · New

Resolved since last review (1)

• Compressed frames skip budget resets and eventually exhaust message budget

[Copilot]

Copilot review overview

🟡 Changes recommended

The workflow can still skip C++ validation, uses a deprecated ARM runner, and introduces untranslated exception text.

Review effort: Balanced
Findings: 2

Open (2)

• Localize new RPC exception message · New
• Align compressed transport frame limit with endpoint configuration

Resolved since last review (2)

• Reconcile endpoint and configured Thrift frame size limits
• Use budget-consuming transport to verify frame budget resets

Previously missed (1)

In code that hasn't changed since last review

Include pom.xml in C++ path change detection

.github/​workflows/​multi-language-client.yml:9

A root-POM-only change now starts this workflow, but the Detect changed client paths switch does not include pom.xml in its C++ case (line 85). It therefore sets cpp=false, so changes to the root iotdb-tools-thrift.version or other C++ build inputs still skip the C++ matrix. Add pom.xml to that case as well.

[Copilot]

Copilot review overview

🟡 Changes recommended

CI path detection, the deprecated ARM runner, and the i18n key naming must be corrected.

Review effort: Balanced
Findings: 1 · 1

Open (2)

• Align compressed transport frame limit with endpoint configuration
• Use deterministic i18n key for frame size exception · New

Resolved since last review (1)

• Localize new RPC exception message

Previously missed (2)

In code that hasn't changed since last review

C++ workflow omits root pom.xml dependency changes

.github/​workflows/​multi-language-client.yml:9

Adding the root POM to the workflow trigger does not make the C++ job run for a future root-POM-only dependency bump: the changes job's C++ case (line 85) still omits pom.xml, so needs.changes.outputs.cpp remains false. Include pom.xml in that C++ path case so Thrift/compiler bumps actually exercise the C++ client.

Use supported ubuntu-24.04-arm runner image

.github/​workflows/​multi-language-client.yml:113

This adds an already-deprecated runner image. GitHub began deprecating ubuntu-22.04-arm on September 17, 2026, warns of longer queues and brownout failures, and will retire it on April 17, 2027; use the supported ubuntu-24.04-arm label instead to avoid making this CI matrix short-lived and intermittently failing.

[Copilot]

Copilot review overview

🔵 Needs a closer look

The runtime/compiler upgrade and core RPC transport accounting changes affect compatibility across all generated services and client languages.

Review effort: Balanced
Findings: 1 · 1

Open (2)

• Align compressed transport frame limit with endpoint configuration
• Use deterministic i18n key for frame size exception

[Copilot]

Copilot review overview

🔵 Needs a closer look

The repository-wide runtime and compiler upgrade affects RPC behavior and multiple client platforms, requiring final human validation of CI results.

Review effort: Balanced
Findings: None

Resolved since last review (2)

• Align compressed transport frame limit with endpoint configuration
• Use deterministic i18n key for frame size exception