Skip to content
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

Adding OSB index specification json for lucene hnsw #552

Conversation

martin-gaievski
Copy link
Member

Signed-off-by: Martin Gaievski gaievski@amazon.com

Description

Gives example of index specification in case one need to use OSB to test lucene knn. We do have similar files fornmslib and faiss engines.

Issues Resolved

Keeping parity between Lucene hnsw and other knn engines

Check List

  • Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Martin Gaievski <gaievski@amazon.com>
@martin-gaievski martin-gaievski added Enhancements Increases software capabilities beyond original client specifications backport 2.x 2.4.0 labels Sep 14, 2022
@martin-gaievski martin-gaievski self-assigned this Sep 14, 2022
@martin-gaievski martin-gaievski requested a review from a team September 14, 2022 21:51
@codecov-commenter
Copy link

codecov-commenter commented Sep 14, 2022

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.07%. Comparing base (7fd6d78) to head (36cbf90).
Report is 391 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #552   +/-   ##
=========================================
  Coverage     84.07%   84.07%           
  Complexity     1021     1021           
=========================================
  Files           146      146           
  Lines          4195     4195           
  Branches        373      373           
=========================================
  Hits           3527     3527           
  Misses          492      492           
  Partials        176      176           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@vamshin vamshin left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks

@martin-gaievski martin-gaievski merged commit cfc6c44 into opensearch-project:main Sep 15, 2022
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 15, 2022
Signed-off-by: Martin Gaievski <gaievski@amazon.com>
(cherry picked from commit cfc6c44)
martin-gaievski pushed a commit that referenced this pull request Sep 15, 2022
Signed-off-by: Martin Gaievski <gaievski@amazon.com>
(cherry picked from commit cfc6c44)
martin-gaievski added a commit that referenced this pull request Sep 28, 2022
Signed-off-by: Martin Gaievski <gaievski@amazon.com>
@heemin32 heemin32 added v2.4.0 'Issues and PRs related to version v2.4.0' and removed 2.4.0 labels Nov 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Enhancements Increases software capabilities beyond original client specifications v2.4.0 'Issues and PRs related to version v2.4.0'
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants