Skip to content

Conversation

reakaleek
Copy link
Member

@reakaleek reakaleek commented Sep 5, 2025

As explained in https://www.elastic.co/docs/solutions/search/hybrid-semantic-text for a truly hybrid search

@reakaleek reakaleek requested a review from a team as a code owner September 5, 2025 19:23
@reakaleek reakaleek requested a review from Mpdreamz September 5, 2025 19:23
Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

Small nits otherwise LGTM!

response.ElasticsearchServerError?.Error?.Reason ?? "Unknown");
}
else
{
Copy link
Member

Choose a reason for hiding this comment

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

braces style violation

Copy link
Member Author

Choose a reason for hiding this comment

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

@reakaleek reakaleek requested a review from Mpdreamz September 8, 2025 13:26
@reakaleek reakaleek merged commit 2990567 into main Sep 8, 2025
19 checks passed
@reakaleek reakaleek deleted the feature/use-retrievers branch September 8, 2025 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants