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

[Backport 2.x] Ensure consistent filter query in KNNQueryBuilder across multiple shards #2362

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport c969f1d from #2359

…rds (#2359)

* Changed filter logic

Signed-off-by: Sahil Buddharaju <[email protected]>

* spotless

Signed-off-by: Sahil Buddharaju <[email protected]>

* Changelog

Signed-off-by: Sahil Buddharaju <[email protected]>

* Added unit tests to test multi shard filters

Signed-off-by: Sahil Buddharaju <[email protected]>

* Changed unit tests and used builder instead of constructor

Signed-off-by: Sahil Buddharaju <[email protected]>

* Spotless apply

Signed-off-by: Sahil Buddharaju <[email protected]>

* slight unit test adjustment

Signed-off-by: Sahil Buddharaju <[email protected]>

* changelog

Signed-off-by: Sahil Buddharaju <[email protected]>

---------

Signed-off-by: Sahil Buddharaju <[email protected]>
Signed-off-by: sahil <[email protected]>
Co-authored-by: Sahil Buddharaju <[email protected]>
(cherry picked from commit c969f1d)
@navneet1v navneet1v added the Bug Fixes Changes to a system or product designed to handle a programming bug/glitch label Jan 3, 2025
@navneet1v
Copy link
Collaborator

@shatejas , @jmazanec15 can one of you approve and merge this backport PR

@jmazanec15 jmazanec15 merged commit 72c6a1e into 2.x Jan 6, 2025
112 checks passed
@github-actions github-actions bot deleted the backport/backport-2359-to-2.x branch January 6, 2025 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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