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

   ### What problem does this PR solve?
   
   Issue Number: N/A
   
   Related PR: https://github.com/apache/doris-thirdparty/pull/402
   
   Problem Summary:
   
   Advance `contrib/clucene` from `c51b5cc9adc` to `abe2b71a1c1`, which 
contains the fix for block-WAND reads through composite readers 
(`MultiTermDocs`). The prior submodule revision did not expose or correctly 
maintain block-postings state across segment-reader boundaries. The updated 
revision globalizes physical block document IDs, invalidates cached block 
metadata when a cross-reader skip occurs, preserves short tail blocks, and 
corrects the first-block posting count after a skip-list lookup.
   
   The upstream change includes composite-reader regression coverage for 
511/512/tail block layouts, cross-reader `skipToBlock()` transitions, and short 
child readers.
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test <!-- At least one of them must be included. -->
       - [x] Regression test: upstream CLucene PR #402 adds `TestReadRange.cpp` 
coverage for composite-reader block reads and skips.
       - [x] Unit Test: upstream CLucene PR #402 CI passed `CLucene UT 
(Linux)`; parent-repository `git diff --check origin/master...HEAD` and `git 
fsck --no-progress --connectivity-only HEAD --no-dangling` pass locally.
       - [ ] 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. Doris inherits the corrected composite-reader block-WAND 
behavior from the updated CLucene revision.
   
   - 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