Skip to content

fix: fast_search should ignore any unindexed data for vector search#6007

Merged
BubbleCal merged 1 commit intomainfrom
yang/vector-fast-search-no-index-empty
Feb 25, 2026
Merged

fix: fast_search should ignore any unindexed data for vector search#6007
BubbleCal merged 1 commit intomainfrom
yang/vector-fast-search-no-index-empty

Conversation

@BubbleCal
Copy link
Copy Markdown
Contributor

Before this, lance still performs flat search if there's no vector index.

@github-actions github-actions Bot added the bug Something isn't working label Feb 25, 2026
@BubbleCal BubbleCal requested a review from westonpace February 25, 2026 13:43
@github-actions
Copy link
Copy Markdown
Contributor

Code Review

LGTM - This is a clean, minimal fix.

Summary

The PR correctly aligns vector search behavior with the existing FTS behavior (scanner.rs:3369-3371) when fast_search is enabled and no index exists: return empty results instead of falling back to flat search.

Verification

  • Uses the correct KNN_INDEX_SCHEMA for the EmptyExec
  • Test properly verifies the behavior by creating a dataset without a vector index (TestVectorDataset::new does not create an index by default)
  • Test asserts normal search returns results while fast search returns 0 rows

No issues found.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@BubbleCal BubbleCal merged commit 20164ae into main Feb 25, 2026
30 checks passed
@BubbleCal BubbleCal deleted the yang/vector-fast-search-no-index-empty branch February 25, 2026 16:04
wjones127 pushed a commit to wjones127/lance that referenced this pull request Feb 25, 2026
…ance-format#6007)

Before this, lance still performs flat search if there's no vector
index.
wjones127 pushed a commit to wjones127/lance that referenced this pull request Feb 25, 2026
…ance-format#6007)

Before this, lance still performs flat search if there's no vector
index.
wjones127 pushed a commit that referenced this pull request Feb 26, 2026
…6007)

Before this, lance still performs flat search if there's no vector
index.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants