Skip to content

Conversation

@kumaramit01
Copy link
Collaborator

@kumaramit01 kumaramit01 commented Oct 31, 2025

Optimizes the expr reranker implementation by caching the ast.
Bugfix in the implementation of the sorting from the new rerankers.
The part that does the query node reranking is the

  • attach RerankFunction in task_search.go (Proxy Node)
  • handlers.go -calls the function WithRerank (Query Node)
  • result.go's function ReduceSearchOnQueryNode applies the rerank function
  • result.go EncodeSearchResultData and returns the Search Result with the query node
  • task_search.go gets the SearchResponse that is segment-reduced and query node function L1 ranking applied and then applies the proxy node ranking if applicable
image

Closes #

💸 TL;DR

📜 Details

Design Doc

Jira

🧪 Testing Steps / Validation

✅ Checks

  • CI tests (if present) are passing
  • Adheres to code style for repo
  • Contributor License Agreement (CLA) completed if not a Reddit employee

…e the expr reranker implementaton by caching the ast. Bugfix in the implementation of the sorting from the new rerankers.

Signed-off-by: Amit Kumar <[email protected]>
@kumaramit01 kumaramit01 requested a review from Xinyi7 October 31, 2025 17:36
Amit Kumar and others added 14 commits October 31, 2025 12:08
…deployed libiberty revision.

Signed-off-by: Amit Kumar <[email protected]>
…-io#45237)

issue: milvus-io#44172, milvus-io#45210, milvus-io#44851

kafka will auto reset the offset to "latest" if the offset is
Out-of-range. the recovery of milvus wal cannot read any message from
that. So once the offset is out-of-range, kafka should read from eariest
to read the latest uncleared data.


https://kafka.apache.org/documentation/#consumerconfigs_auto.offset.reset

Signed-off-by: chyezh <[email protected]>
…t the search has output fields (for other rerankers like the decay reranker), it crashes trying to prepare field data it doesn't need.

Signed-off-by: Amit Kumar <[email protected]>
Signed-off-by: Amit Kumar <[email protected]>
Signed-off-by: Amit Kumar <[email protected]>
Signed-off-by: Amit Kumar <[email protected]>
Signed-off-by: Amit Kumar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants