branch-4.1: [chore](paimon) remove paimon-cpp dependency by suxiaogang223 · Pull Request #67378 · apache/doris

suxiaogang223 · 2026-09-01T09:47:30Z

What problem does this PR solve?

Issue Number: N/A

Remove paimon-cpp completely from branch-4.1 while retaining deprecated Thrift wire values for rolling-upgrade compatibility. Keep its thirdparty sources, Arrow patches, and standard scripts aligned with master; branch-4.1 does not publish a separate thirdparty package and consumes the package produced from master.

Release note

Remove the deprecated paimon-cpp reader and build dependency.

Check List (For Author)

Test:
- Paimon FE unit tests
- Clean Arrow 24 patch application and idempotent download flow
- Thirdparty scripts, shell syntax, workflow YAML, and diff checks
Behavior changed:
- Yes. Paimon logical splits use JNI; deprecated wire values remain for compatibility.
Does this need documentation:
- No

hello-stephen · 2026-09-01T09:47:36Z

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?

suxiaogang223 · 2026-09-01T10:50:29Z

run buildall

hello-stephen · 2026-09-01T17:05:38Z

BE UT Coverage Report

Increment line coverage 0.00% (0/5) 🎉

Increment coverage report
Complete coverage report

Category	Coverage
Function Coverage	59.58% (25801/43307)
Line Coverage	44.18% (264914/599586)
Region Coverage	40.08% (210043/524094)
Branch Coverage	41.55% (96967/233356)

hello-stephen · 2026-09-01T19:07:36Z

BE Regression && UT Coverage Report

Increment line coverage 0.00% (0/5) 🎉

Increment coverage report
Complete coverage report

Category	Coverage
Function Coverage	74.32% (31317/42138)
Line Coverage	58.52% (348119/594841)
Region Coverage	55.22% (290107/525329)
Branch Coverage	56.06% (130685/233114)

suxiaogang223 · 2026-09-02T02:21:13Z

run buildall

### What problem does this PR solve? Issue Number: N/A Related PR: apache#67378 Problem Summary: Doris still downloads, builds, links, and carries the deprecated paimon-cpp reader even though master plans serialized Paimon splits for the JNI reader. Remove the paimon-cpp thirdparty package, BE reader implementation, obsolete session-variable plumbing, and related tests. Keep the deprecated PaloInternalService field, TPaimonReaderType.PAIMON_CPP enum value, and IGNORE_PAIMON_CPP option for rolling-upgrade and configuration compatibility. Preserve Arrow as an independent shared thirdparty component so existing prebuilts remain usable and future automation artifacts no longer include paimon-cpp. ### Release note Remove the deprecated paimon-cpp reader and its build dependency. Paimon reads continue through the JNI and native readers. ### Check List (For Author) - Test: Unit Test and manual test - mvn -f fe/pom.xml -pl :fe-connector-paimon -am install -Dmaven.build.cache.enabled=false (553 tests passed, 1 live test skipped) - thirdparty Arrow and ADBC lifecycle tests - thirdparty download/mirror/checksum/retry script tests - build-support/check-build-hygiene.sh - shell syntax, YAML parsing, and git diff --check - Behavior changed: Yes. The paimon-cpp reader and enable_paimon_cpp_reader session variable are removed; deprecated wire fields remain for compatibility. - Does this need documentation: No

hello-stephen · 2026-09-02T03:20:19Z

Cloud UT Coverage Report

Increment line coverage 🎉

Increment coverage report
Complete coverage report

Category	Coverage
Function Coverage	78.34% (1942/2479)
Line Coverage	64.91% (34550/53231)
Region Coverage	52.39% (33115/63212)
Branch Coverage	55.03% (10174/18488)

### What problem does this PR solve? Issue Number: N/A Related PR: apache#67378, apache#67432 Problem Summary: Remove the deprecated paimon-cpp package, reader implementation, build integration, tests, and Paimon-specific Arrow patch. Retain deprecated wire and configuration values for compatibility. Fold Arrow source variables back into the standard thirdparty manifest and remove the temporary fingerprint, recovery, and lifecycle scripts so the next master automation package is the final cleaned artifact. ### Release note Remove the deprecated paimon-cpp reader and build dependency. Paimon reads continue through the JNI and native readers. ### Check List (For Author) - Test: Unit Test and manual test - Paimon FE connector reactor: 553 tests passed, 1 live test skipped - Clean Arrow 24 patch application and Arrow package build - Thirdparty script tests, build hygiene, shell syntax, workflow YAML, and diff checks - Behavior changed: Yes. The paimon-cpp reader and enable_paimon_cpp_reader session variable are removed; deprecated wire values remain for compatibility. - Does this need documentation: No

suxiaogang223 · 2026-09-02T07:40:35Z

run buildall

### What problem does this PR solve? Issue Number: N/A Related PR: apache#67385, apache#67432 Problem Summary: Remove the deprecated paimon-cpp source, build, reader, configuration, and test dependencies while retaining deprecated Thrift wire values for rolling-upgrade compatibility. Keep branch-4.1 aligned with master by using the standard thirdparty manifest and removing the temporary Arrow fingerprint, recovery, lifecycle, and Paimon-specific patch files. The shared thirdparty package continues to be produced from master. ### Release note Remove the deprecated paimon-cpp reader and build dependency. ### Check List (For Author) - Test: Unit Test and manual test - Paimon FE unit tests - Clean Arrow 24 patch application and idempotent download flow - Thirdparty script tests, shell syntax, workflow YAML, and diff checks - Behavior changed: Yes. Paimon logical splits use JNI; deprecated wire values remain for compatibility. - Does this need documentation: No

### What problem does this PR solve? Issue Number: N/A Related PR: apache#67378, apache#67432 Problem Summary: Remove the deprecated paimon-cpp package, reader implementation, build integration, tests, and Paimon-specific Arrow patch. Retain deprecated wire and configuration values for compatibility. Fold Arrow source variables back into the standard thirdparty manifest and remove the temporary fingerprint, recovery, and lifecycle scripts so the next master automation package is the final cleaned artifact. ### Release note Remove the deprecated paimon-cpp reader and build dependency. Paimon reads continue through the JNI and native readers. ### Check List (For Author) - Test: Unit Test and manual test - Paimon FE connector reactor: 553 tests passed, 1 live test skipped - Clean Arrow 24 patch application and Arrow package build - Thirdparty script tests, build hygiene, shell syntax, workflow YAML, and diff checks - Behavior changed: Yes. The paimon-cpp reader and enable_paimon_cpp_reader session variable are removed; deprecated wire values remain for compatibility. - Does this need documentation: No

suxiaogang223 · 2026-09-02T09:32:25Z

run buildall

hello-stephen · 2026-09-02T09:56:24Z

Cloud UT Coverage Report

Increment line coverage 🎉

Increment coverage report
Complete coverage report

Category	Coverage
Function Coverage	78.34% (1942/2479)
Line Coverage	64.89% (34539/53231)
Region Coverage	52.27% (33044/63212)
Branch Coverage	55.00% (10168/18488)

hello-stephen · 2026-09-02T17:39:25Z

BE Regression && UT Coverage Report

Increment line coverage 0.00% (0/5) 🎉

Increment coverage report
Complete coverage report

Category	Coverage
Function Coverage	74.40% (31352/42138)
Line Coverage	58.56% (348334/594841)
Region Coverage	55.27% (290328/525329)
Branch Coverage	56.15% (130883/233114)

### What problem does this PR solve? Issue Number: N/A Related PR: #67378, #67432 Remove paimon-cpp completely from the thirdparty manifest, BE reader, build integration, tests, and Arrow patch set. Also fold the Arrow cleanup from #67432 into the standard thirdparty workflow, removing the temporary fingerprint, recovery, and lifecycle scripts. Deprecated Thrift enum/field values and `IGNORE_PAIMON_CPP` remain only for compatibility. This master change is the source used to publish the shared master/branch-4.1 thirdparty package. ### Release note Remove the deprecated paimon-cpp reader and build dependency. Paimon reads continue through JNI and native readers. ### Check List (For Author) - Test: - [x] Paimon FE connector: 553 passed, 1 live test skipped - [x] Clean Arrow 24 patch application and package build - [x] Thirdparty scripts, build hygiene, shell syntax, workflow YAML, and diff checks - Behavior changed: - [x] Yes. paimon-cpp is removed; deprecated wire values remain for compatibility. - Does this need documentation: - [x] No

suxiaogang223 mentioned this pull request Sep 1, 2026

[chore] add branch-4.1 thirdparty automation apache/doris-thirdparty#412

Closed

suxiaogang223 changed the title ~~[chore](paimon) remove paimon-cpp dependency~~ [branch-4.1][chore](paimon) remove paimon-cpp dependency Sep 1, 2026

suxiaogang223 mentioned this pull request Sep 1, 2026

[chore](paimon) remove paimon-cpp dependency #67385

Merged

5 tasks

suxiaogang223 changed the title ~~[branch-4.1][chore](paimon) remove paimon-cpp dependency~~ branch-4.1: [chore](paimon) remove paimon-cpp dependency Sep 2, 2026

suxiaogang223 marked this pull request as ready for review September 2, 2026 02:02

suxiaogang223 requested a review from yiguolei as a code owner September 2, 2026 02:02

suxiaogang223 force-pushed the codex/remove-paimon-cpp-phase1 branch from 79747a2 to 5478d34 Compare September 2, 2026 02:18

suxiaogang223 force-pushed the codex/remove-paimon-cpp-phase1 branch from 5478d34 to 7e3596d Compare September 2, 2026 07:36

suxiaogang223 force-pushed the codex/remove-paimon-cpp-phase1 branch from 7e3596d to 77d29b7 Compare September 2, 2026 09:31

yiguolei approved these changes Sep 2, 2026

View reviewed changes

yiguolei merged commit 57636ca into apache:branch-4.1 Sep 2, 2026
25 of 31 checks passed

suxiaogang223 deleted the codex/remove-paimon-cpp-phase1 branch September 3, 2026 02:01

suxiaogang223 mentioned this pull request Sep 4, 2026

branch-4.1: [fix](paimon) tolerate removed paimon-cpp session variable #67472 #67522

Closed

4 tasks

Navigation Menu

Uh oh!

branch-4.1: [chore](paimon) remove paimon-cpp dependency - #67378

branch-4.1: [chore](paimon) remove paimon-cpp dependency#67378
yiguolei merged 1 commit into
apache:branch-4.1from
suxiaogang223:codex/remove-paimon-cpp-phase1

suxiaogang223 commented Sep 1, 2026 •

edited

Loading

Uh oh!

hello-stephen commented Sep 1, 2026

Uh oh!

suxiaogang223 commented Sep 1, 2026

Uh oh!

hello-stephen commented Sep 1, 2026

Uh oh!

hello-stephen commented Sep 1, 2026

Uh oh!

suxiaogang223 commented Sep 2, 2026

Uh oh!

hello-stephen commented Sep 2, 2026

Uh oh!

suxiaogang223 commented Sep 2, 2026

Uh oh!

suxiaogang223 commented Sep 2, 2026

Uh oh!

hello-stephen commented Sep 2, 2026

Uh oh!

hello-stephen commented Sep 2, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

suxiaogang223 commented Sep 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What problem does this PR solve?

Release note

Check List (For Author)

Uh oh!

hello-stephen commented Sep 1, 2026

Uh oh!

suxiaogang223 commented Sep 1, 2026

Uh oh!

hello-stephen commented Sep 1, 2026

BE UT Coverage Report

Uh oh!

hello-stephen commented Sep 1, 2026

BE Regression && UT Coverage Report

Uh oh!

suxiaogang223 commented Sep 2, 2026

Uh oh!

hello-stephen commented Sep 2, 2026

Cloud UT Coverage Report

Uh oh!

suxiaogang223 commented Sep 2, 2026

Uh oh!

suxiaogang223 commented Sep 2, 2026

Uh oh!

hello-stephen commented Sep 2, 2026

Cloud UT Coverage Report

Uh oh!

hello-stephen commented Sep 2, 2026

BE Regression && UT Coverage Report

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

suxiaogang223 commented Sep 1, 2026 •

edited

Loading