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

Fix shard level rescoring disabled setting flag #2352

Conversation

naveentatikonda
Copy link
Member

@naveentatikonda naveentatikonda commented Dec 24, 2024

Description

Fix shard level rescoring disabled index setting flag(index.knn.disk.vector.shard_level_rescoring_disabled) to enable shard level rescoring.

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

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.

@naveentatikonda naveentatikonda added Bug Fixes Changes to a system or product designed to handle a programming bug/glitch backport 2.x labels Dec 24, 2024
@naveentatikonda naveentatikonda force-pushed the fix_shard_level_rescoring_setting branch from b6780d5 to 5d62f00 Compare December 24, 2024 23:38
navneet1v
navneet1v previously approved these changes Dec 24, 2024
Copy link
Collaborator

@navneet1v navneet1v left a comment

Choose a reason for hiding this comment

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

Thanks for fixing the bug.

@navneet1v
Copy link
Collaborator

@shatejas , @jmazanec15 , @Vikasht34 please take a look at this please.

Copy link
Collaborator

@shatejas shatejas left a comment

Choose a reason for hiding this comment

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

This looks good.

There aren't good ways to have automated tests for this, can we please put a debugger and make sure the right code path is executed

@naveentatikonda
Copy link
Member Author

naveentatikonda commented Dec 25, 2024

This looks good.

There aren't good ways to have automated tests for this, can we please put a debugger and make sure the right code path is executed

Verified with debugger. It is working as expected. Can you pls review it again. Thanks!

@naveentatikonda naveentatikonda merged commit c728f02 into opensearch-project:main Dec 25, 2024
33 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Dec 25, 2024
naveentatikonda added a commit that referenced this pull request Dec 26, 2024
(cherry picked from commit c728f02)

Co-authored-by: Naveen Tatikonda <[email protected]>
buddharajusahil pushed a commit to buddharajusahil/k-NN that referenced this pull request Dec 31, 2024
Gankris96 pushed a commit to Gankris96/k-NN that referenced this pull request Dec 31, 2024
Gankris96 pushed a commit to Gankris96/k-NN that referenced this pull request Dec 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Bug Fixes Changes to a system or product designed to handle a programming bug/glitch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants