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

   ### What problem does this PR solve?
   
   Issue Number: None
   
   Related PR: #66475
   
   Problem Summary:
   
   This PR separately submits the Meta Service changes from #66475. During a 
partition-level inverted-index format rollout, a rowset can use a format that 
differs from its shared schema metadata. The recycler and checker previously 
selected index file paths from shared schema metadata or its cache, which could 
target V1 paths for a rowset whose files use V3.
   
   The change persists the actual inverted-index storage format in rowset and 
tablet metadata. The recycler and checker now prefer the rowset format and 
retain schema and schema-KV fallback for metadata written before the field 
existed. A Cloud unit test creates a V3 rowset with V1 shared schema metadata 
and verifies that recycler removes the V3 index file.
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test <!-- At least one of them must be included. -->
       - [ ] Regression test
       - [x] Unit Test
           - `./run-cloud-ut.sh --run 
--filter=recycler_test:RecyclerTest.delete_tmp_rowset_data_with_rowset_idx_v3`
       - [ ] Manual test (add detailed scripts or steps below)
       - [ ] 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 <!-- Add your reason?  -->
   
   - Behavior changed:
       - [ ] No.
       - [x] Yes. Meta Service recycler and checker use each rowset's persisted 
inverted-index format when selecting index files.
   
   - Does this need documentation?
       - [x] No.
       - [ ] Yes. <!-- Add document PR link here. eg: 
https://github.com/apache/doris-website/pull/1214 -->
   
   ### Check List (For Reviewer who merge this PR)
   
   - [ ] Confirm the release note
   - [ ] Confirm test cases
   - [ ] Confirm document
   - [ ] Add branch pick label <!-- Add branch pick label that this PR should 
merge into -->


-- 
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