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

[DOC] Search request slow logs #5298

Merged
merged 3 commits into from
Feb 1, 2024

Conversation

dzane17
Copy link
Contributor

@dzane17 dzane17 commented Oct 18, 2023

Description

Documentation for new search request slow log feature. Target version is 2.12.

OpenSearch PR: opensearch-project/OpenSearch#10650
GitHub Issue: opensearch-project/OpenSearch#9642

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.

Copy link
Collaborator

@natebower natebower left a comment

Choose a reason for hiding this comment

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

@kolchfa-aws Just one change!

_monitoring-your-cluster/logs.md Outdated Show resolved Hide resolved
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.

Thank you, @dzane17! To which versions is this applicable?

@kolchfa-aws kolchfa-aws self-assigned this Oct 19, 2023
@kolchfa-aws kolchfa-aws added 6 - Done but waiting to merge PR: The work is done and ready to merge v2.12.0 release-notes PR: Include this PR in the automated release notes labels Oct 19, 2023
@kolchfa-aws kolchfa-aws added this to the v2.12 milestone Oct 19, 2023
@dzane17 dzane17 force-pushed the slowlog-doc-2 branch 2 times, most recently from 0cef5c4 to 619ca95 Compare November 17, 2023 20:45
@dzane17
Copy link
Contributor Author

dzane17 commented Nov 17, 2023

@natebower @kolchfa-aws I rebased this PR. Can you please take a look again? The dev PR is now merged so documentation is good to go

Signed-off-by: David Zane <davizane@amazon.com>
_query-dsl/index.md Outdated Show resolved Hide resolved
Signed-off-by: kolchfa-aws <105444904+kolchfa-aws@users.noreply.github.com>
@kolchfa-aws
Copy link
Collaborator

@dzane17 Looks good. Thanks!

@hdhalter hdhalter merged commit 6f3a235 into opensearch-project:main Feb 1, 2024
3 checks passed
@hdhalter hdhalter added the 3 - Done Issue is done/complete label Feb 1, 2024
@hdhalter hdhalter removed the 6 - Done but waiting to merge PR: The work is done and ready to merge label Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Done Issue is done/complete release-notes PR: Include this PR in the automated release notes v2.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants