suxiaogang223 opened a new pull request, #68837:
URL: https://github.com/apache/doris/pull/68837

   ### What problem does this PR solve?
   
   Issue Number: #65086
   
   Related PR: #65868
   
   Related PR: #66227
   
   Related PR: #68320
   
   Problem Summary:
   
   Master and branch-4.1 assigned incompatible meanings or numeric values to 
several external-table Thrift fields. In particular, `TReportExecStatusParams` 
field 34 carried opaque connector commit data on master but typed Paimon commit 
messages on branch-4.1. The external table sink partition enum and several 
Paimon, Iceberg, routing, query-option, and backend-capability fields also used 
different wire ids.
   
   This change keeps the branch-4.1 field meanings and enum values at their 
established ids, moves unreleased master-only fields to new trailing ids, and 
keeps Paimon and Iceberg commit data on independent fields. It also preserves 
the master transaction flow, final-report ownership transfer, Thrift message 
budget, acknowledgement, and cleanup behavior.
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test
       - [ ] Regression test
       - [ ] Unit Test
       - [x] Manual test (add detailed scripts or steps below)
           - `./build.sh --fe -j 12`
           - `ENABLE_PCH=OFF ./build.sh --be -j 12`
           - FE, BE, BE Java extensions, and CDC client build successfully.
           - BE `doris_be` links successfully and passes the GLIBC 2.17 
compatibility check.
           - `build-support/check-format.sh`
           - `build-support/check-build-hygiene.sh`
           - `git diff --check origin/master..HEAD`
       - [ ] 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:
       - [x] No. The change aligns wire compatibility for unreleased master 
fields.
       - [ ] Yes.
   
   - 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]

Reply via email to