Skip to content

enhance hybrid_search interface#63

Merged
y1-l1n merged 5 commits into
oceanbase:developfrom
y1-l1n:dev
Dec 8, 2025
Merged

enhance hybrid_search interface#63
y1-l1n merged 5 commits into
oceanbase:developfrom
y1-l1n:dev

Conversation

@y1-l1n

@y1-l1n y1-l1n commented Dec 5, 2025

Copy link
Copy Markdown
Contributor

Summary

  • Added HybridSearch fluent builder with DSL helpers (DOCUMENT.contains/not_contains, K().is_in/not_in, TEXT/EMBEDDINGS) plus chaining of .query()/.knn()/.rank()/.limit()/.select(), exposed in init, documented in README, example, and integration tests.

  • hybrid_search() now accepts per-route boost, with collection forwarding updated.

  • Document filters support $not_contains.

  • Metadata filters support $in / $nin (mirrored by K().is_in/not_in), with tests expanded.

@longbingljw

Copy link
Copy Markdown
Member

Hello,seems like this refactor of hybrid_search() interface will change how it's used compared to before.

@y1-l1n

y1-l1n commented Dec 8, 2025

Copy link
Copy Markdown
Contributor Author

Hello,seems like this refactor of hybrid_search() interface will change how it's used compared to before.

These enhancements do not affect the existing hybrid search interface; they offer a more simplified and user-friendly method.

@y1-l1n y1-l1n merged commit 7e42ee8 into oceanbase:develop Dec 8, 2025
1 check passed
hnwyllmm added a commit that referenced this pull request Jan 28, 2026
## Summary
close #150 

rollback enhanced hybrid search which committed by #63 


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Refactor**
* Removed the fluent hybrid-search builder; hybrid_search now accepts
dictionary-style parameters only.
* Removed per-route boost parameters and builder precedence from hybrid
search inputs.

* **Documentation**
* Examples and docs updated to show direct dict-based hybrid_search
usage; builder-specific content removed.

* **Tests**
* Builder-focused integration tests removed; remaining tests updated to
use dict-style calls.
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.

2 participants