suxiaogang223 opened a new pull request, #67378:
URL: https://github.com/apache/doris/pull/67378
### What problem does this PR solve?
Issue Number: N/A
Related PR: N/A
Problem Summary:
This is phase one of removing paimon-cpp. It removes the source, build,
link, reader, configuration, and test dependencies while keeping the existing
Arrow prebuilt artifacts reusable.
- Paimon logical splits now use the JNI reader; physical Parquet/ORC splits
keep the existing native routing.
- `TQueryOptions.enable_paimon_cpp_reader` field 201 and
`TPaimonReaderType.PAIMON_CPP` remain as deprecated wire values. New FE plans
no longer emit or set them.
- Arrow validation keeps the current semantic fingerprint and accepts the
existing legacy marker. Extra paimon-cpp files in an old prebuilt archive are
ignored.
- The existing Arrow patch remains in phase one so the shared prebuilt
fingerprint does not change before replacement artifacts are published.
### Release note
Remove the paimon-cpp reader and third-party build dependency.
### Check List (For Author)
- Test
- [ ] Regression test
- [ ] Unit Test
- [x] Manual test (add detailed scripts or steps below)
- `thirdparty/test/arrow-lifecycle-test.sh`
- Shell syntax checks for changed build scripts
- Thrift syntax generation for `PaloInternalService.thrift`
- Workflow YAML parsing and `git diff --check`
- FE Checkstyle completed with zero violations before rebase; the
targeted Maven test was blocked by missing generated Thrift Java classes in the
local environment.
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason
- Behavior changed:
- [ ] No.
- [x] Yes. Paimon logical splits no longer select paimon-cpp and use JNI
instead.
- Does this need documentation?
- [x] No.
- [ ] Yes.
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
