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

Deprecate max_token_score in neural sparse search #6554

Merged

Conversation

zhichao-aws
Copy link
Member

Description

The max_token_score field in neural sparse search was deprecated in 2.12 release. (ref PR). To avoid causing confusion for users, we should also update the information in documentation.

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

Checklist

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

Signed-off-by: zhichao-aws <[email protected]>
@zhichao-aws
Copy link
Member Author

This PR should be backported to 2.12 and 2.x branch.

Copy link
Collaborator

@kolchfa-aws kolchfa-aws left a comment

Choose a reason for hiding this comment

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

Just one comment. Thanks, @zhichao-aws!

_query-dsl/specialized/neural-sparse.md Outdated Show resolved Hide resolved
@kolchfa-aws kolchfa-aws self-assigned this Mar 1, 2024
@kolchfa-aws kolchfa-aws added the backport 2.12 PR: Backport label for 2.12 label Mar 1, 2024
@kolchfa-aws kolchfa-aws merged commit 5f486ab into opensearch-project:main Mar 1, 2024
3 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Mar 1, 2024
* deprecated max_token_score

Signed-off-by: zhichao-aws <[email protected]>

* Update _query-dsl/specialized/neural-sparse.md

Signed-off-by: kolchfa-aws <[email protected]>

---------

Signed-off-by: zhichao-aws <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
(cherry picked from commit 5f486ab)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
kolchfa-aws added a commit that referenced this pull request Mar 1, 2024
* deprecated max_token_score



* Update _query-dsl/specialized/neural-sparse.md



---------




(cherry picked from commit 5f486ab)

Signed-off-by: zhichao-aws <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: kolchfa-aws <[email protected]>
oeyh pushed a commit to oeyh/documentation-website that referenced this pull request Mar 14, 2024
…#6554)

* deprecated max_token_score

Signed-off-by: zhichao-aws <[email protected]>

* Update _query-dsl/specialized/neural-sparse.md

Signed-off-by: kolchfa-aws <[email protected]>

---------

Signed-off-by: zhichao-aws <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.12 PR: Backport label for 2.12
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants