-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Initial support for vespa search #174
Conversation
f8c2b7b
to
a7bbb5d
Compare
a7bbb5d
to
91d4aeb
Compare
da0fb59
to
516f0b5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looking good so far - is it meant to be still draft?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests look good - we perhaps need to chat about cpr_data_access
and the models contained therein. Just to fully understand what the intention is and the future of this is.
Description
Support for optionally using a configured vespa instance for search.
Type of change
Please select the option(s) below that are most relevant:
How Has This Been Tested?
New tests added & test vespa instance added to docker compose environment.
Reviewer Checklist