Skip to content

Share search property filter dispatch - #1962

Merged
igordayen merged 1 commit into
embabel:mainfrom
arnabnandy7:fix/shareSearchFilterDispatch
Aug 23, 2026
Merged

Share search property filter dispatch#1962
igordayen merged 1 commit into
embabel:mainfrom
arnabnandy7:fix/shareSearchFilterDispatch

Conversation

@arnabnandy7

Copy link
Copy Markdown
Contributor

Summary

Share property-filter dispatch between vector and text search.

  • Extract VectorSearch.searchWithFilter and TextSearch.searchWithFilter helpers
  • Delegate VectorSearchTools and TextSearchTools to the shared helpers
  • Preserve native filtering and post-filtering fallback behavior
  • Preserve topK inflation, result ordering, and truncation
  • Add focused tests for native text filtering and fallback post-filtering

Testing

mvn -f embabel-agent-rag/embabel-agent-rag-core/pom.xml \
  -Dtest=ToolishRagTest#TextSearchToolsTests test

Fixes #1960.

@arnabnandy7
arnabnandy7 marked this pull request as ready for review August 21, 2026 03:28
@igordayen
igordayen requested a review from jasperblues August 21, 2026 19:38
@igordayen igordayen added this to the 1.5.1-Release🔵 milestone Aug 21, 2026

@igordayen igordayen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@arnabnandy7 - thanks for contributing, few non-blocking observations. thanks

@arnabnandy7
arnabnandy7 force-pushed the fix/shareSearchFilterDispatch branch 2 times, most recently from 2fb9b4f to 0fb5edb Compare August 22, 2026 03:13
Signed-off-by: Arnab Nandy <arnab_nandy7@yahoo.com>

@jasperblues jasperblues left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Follow-up raised as #1970, not blocking.

@igordayen
igordayen merged commit a58e520 into embabel:main Aug 23, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Share property-filter dispatch between vector and text search

3 participants