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] Move query categorization changes to plugin #29

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link

Backport 811f4a5 from #16.

* Move query categorization changes to plugin

Signed-off-by: Siddhant Deshmukh <[email protected]>

* Fix SearchSourceBuilder read/write test failures

Signed-off-by: Siddhant Deshmukh <[email protected]>

* Fix tests

Signed-off-by: Siddhant Deshmukh <[email protected]>

* Fix starting and stopping query insights service

Signed-off-by: Siddhant Deshmukh <[email protected]>

* Unit tests for feature enable/disable and refactor logic

Signed-off-by: Siddhant Deshmukh <[email protected]>

---------

Signed-off-by: Siddhant Deshmukh <[email protected]>
(cherry picked from commit 811f4a5)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@ansjcy
Copy link
Member

ansjcy commented Jul 17, 2024

We need to merge this opensearch-project/OpenSearch#14762 to make the build pass before merging.

@jainankitk
Copy link
Collaborator

We need to merge this opensearch-project/OpenSearch#14762 to make the build pass before merging.

That is merged already, we should get the build to pass now!

@deshsidd
Copy link
Collaborator

Trying to rerun the builds, although might take some time before it can pick up changes from core

@deshsidd
Copy link
Collaborator

Will need to wait for the new core build to be picked up as still failing with the following : java.lang.IllegalArgumentException: Cannot register setting [search.query.metrics.enabled] twice
Will try later today.

@deshsidd deshsidd merged commit 2f706af into 2.x Jul 18, 2024
10 checks passed
@ansjcy ansjcy deleted the backport/backport-16-to-2.x branch August 1, 2024 00:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants