Yukang-Lian opened a new pull request, #67608:
URL: https://github.com/apache/doris/pull/67608

   ### What problem does this PR solve?
   
   Issue Number: None
   
   Related PR: #64312
   
   Problem Summary: Backport the Cloud schema-change base-tablet refresh fix 
from #64312 to `branch-3.1`.
   
   Before capturing rowset readers and registering schema change V1, the base 
tablet now performs an uncapped rowset refresh. This prevents a stale request 
alter version from hiding newer visible rowsets across retries. The 
implementation is adapted to this branch's older `CloudTablet::sync_rowsets()` 
API.
   
   The master regression fixture depends on later cross-V1 test infrastructure 
that is absent from this branch, so this backport keeps the production fix and 
its diagnostic debug point without importing unsupported test scaffolding.
   
   The branch is rebased onto the latest `branch-3.1`.
   
   ### Release note
   
   Refresh the Cloud schema-change base tablet before calculating and 
registering V1.
   
   ### Check List (For Author)
   
   - Test: Manual / static validation
       - `clang-format` 16 `--dry-run --Werror`: passed for all changed C++ 
files
       - `git diff --check`: passed
       - Exact-base linked BE tests were not run locally; hosted CI is 
requested with `run buildall`
   - Behavior changed: Yes
       - Schema change uses the latest visible base-tablet rowsets instead of a 
query-version-capped refresh
   - Does this need documentation: 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]

Reply via email to