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

   ### What problem does this PR solve?
   
   Related PRs: https://github.com/lance-format/lance-c/pull/97, 
https://github.com/lance-format/lance-c/pull/96
   
   Update the pinned lance-c revision from 
`8e9e09ecc795c91ce7f97513db94c2d91eec799d` to 
`4fafcd64525b274b113208390bbcca86bbbc093b`, the latest upstream `main` commit 
when this PR was prepared, and update the archive checksum.
   
   The new revision includes the upstream fix for cold-session vector prewarm 
when an index name matches a different schema column. It resolves the indexed 
field from segment metadata while preserving concurrent segment 
opening/prewarming. The revision also includes the row-address 
allowlist/blocklist prefilter APIs from lance-c #96.
   
   Refresh one C++ test context in the existing Foyer patch to accommodate the 
test added by #96. The patch's added/deleted code lines are unchanged; no local 
prewarm patch is introduced.
   
   ### Release note
   
   Fix Lance index prewarm failures when a vector index name matches a 
different schema column.
   
   ### Check List (For Author)
   
   - Test
       - [x] Manual test
           - Downloaded the exact pinned archive and verified its checksum 
through the Doris downloader.
           - `bash thirdparty/test/lance-prefilter-patch-test.sh 
thirdparty/src`: passed all scenarios, including fresh extraction, idempotence, 
cache invalidation, and patch-failure handling.
           - `cargo metadata --locked --offline --format-version 1` on the 
patched source: passed dependency resolution, including Foyer.
           - Compiled the patched upstream C and C++ test sources with 
`-fsyntax-only`: passed.
           - `bash -n` on the dependency scripts and `git diff --check`: passed.
           - Full Doris build and runtime regression tests were not run 
locally; requested through `run buildall`.
   - Behavior changed:
       - [x] Yes. Includes the upstream prewarm fix and new prefilter APIs.
   - Does this need documentation?
       - [x] No.
   
   ### 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