Create a search pipeline on hot swap, enabling hybrid search #3867
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
A new search pipeline is added when hot swapping that enables hybrid searching in OpenSearch (docs for reference)
Specific Changes in this PR
Meadow.Search.Index.create_search_pipeline/4
, which is now called during hot swaps.Meadow.Search.Index.create_vector_pipeline/4
to ``Meadow.Search.Index.create_ingest_pipeline/4`, in order to match the OpenSearch naming scheme./_search/pipeline/[name]
and/_ingest/pipeline/[name]
.Version bump required by the PR
See Semantic Versioning 2.0.0 for help discerning which is required.
Steps to Test
Meadow.Data.Indexer.reindex_all()
to create the search pipelineiex
:Meadow.Search.Config.embedding_model_id()
(will likely be a cold start)es-proxy
to start the OpenSearch dashboard and run a query:Also please let developers know if there are any special instructions to test this in the development environment.
🚀 Deployment Notes
Note - if you check any of these boxes go to the (always open)
main
<-staging
PR and add detailed notes and instructions to help out others who may end up deploying your changes to productionmix meadow.pipeline.setup
run)miscellany
Tested/Verified