suxiaogang223 opened a new pull request, #67522:
URL: https://github.com/apache/doris/pull/67522
### What problem does this PR solve?
Issue Number: N/A
Related PR: #67472, #67378
Problem Summary:
branch-4.1 removed the paimon-cpp reader and enable_paimon_cpp_reader
session variable in #67378. Older clients and replayed state may still
reference the removed variable and receive an unknown-system-variable error.
Master PR #67472 adds the name to the existing generic
removed-session-variable compatibility allowlist. branch-4.1 does not contain
that framework, so this backport resolves the cherry-pick conflict by adding
equivalent handling for this variable across SET, non-master FE SET, legacy
reads, and Nereids reads. It also adds a focused unit test for the
compatibility behavior.
### Release note
None
### Check List (For Author)
- Test:
- [x] ./run-fe-ut.sh --run
org.apache.doris.qe.VariableMgrTest#testRemovedPaimonCppReaderVariableCompatibility
(1 test passed; 15 reactor modules succeeded)
- [x] git diff origin/branch-4.1...HEAD --check
- Behavior changed:
- [x] Yes. References to the removed enable_paimon_cpp_reader variable
are tolerated.
- Does this need documentation:
- [x] No
--
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]
