Repository navigation
[improvement](lance) Update lance-c to upstream search filters - #68776
Merged
Merged
Conversation
### What problem does this PR solve? Issue Number: N/A Related PR: N/A Problem Summary: Doris pinned lance-c at 98468344bc9d56aa7ed4a4192e844afa62b34ba1 and missed upstream apache#91 vector post-filtering and scoped FTS prefiltering. Pin the dependency to 8e9e09ecc795c91ce7f97513db94c2d91eec799d, update the archive checksum, and regenerate the local Foyer patch against the new base while keeping its implementation unchanged. ### Release note Include Lance-C vector post-filtering and scoped FTS prefiltering improvements. ### Check List (For Author) - Test: Static dependency validation: shell syntax checks, archive checksum validation, strict patch application, isolated downloader idempotence and source-tree comparison. No Doris build or unit/regression tests run. - Behavior changed: Yes; adopts upstream Lance-C apache#91 search-filter behavior. - Does this need documentation: No
Contributor
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
Contributor
Author
|
run buildall |
### What problem does this PR solve? Issue Number: N/A Related PR: lance-format/lance-c#73 Problem Summary: Doris carried an older local Foyer patch. Refresh it from the current PR head 8a08f171b66a3e01e7ba634bcfc6168a955c7289 and rebase it onto the pinned Lance-C commit 8e9e09ecc795c91ce7f97513db94c2d91eec799d, preserving the newer upstream search and prewarm changes. The PR head includes its revert of the interim cache-origin-isolation change. ### Release note Update the bundled Lance-C Foyer cache integration to match upstream PR apache#73. ### Check List (For Author) - Test: Static validation: strict patch application, downloader refresh, isolated repeated downloader run, source-tree comparison, shell syntax, and diff checks. No build or tests run. - Behavior changed: Yes; the bundled Foyer implementation now matches the current PR head, including its cache-origin-isolation revert. - Does this need documentation: No
Contributor
Author
|
run buildall |
Contributor
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
Contributor
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
yiguolei
approved these changes
Oct 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What problem does this PR solve?
Problem Summary:
lance-format/lance-c#91
Include Lance-C vector post-filtering and scoped FTS prefiltering improvements.
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)