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

Update search profile API documentation with concurrency related changes #5999

Merged
merged 4 commits into from
Feb 1, 2024

Conversation

ticheng-aws
Copy link
Contributor

Description

We need to update the document for the query profile part here in the next release. We will have some new fields similar to the aggregation profile like max/min/avg_slice_time_in_nanos and time/count stats for the query breakdowns except create_weight type. More details can be found #9248.

Issues Resolved

Resolves #4281

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.

@ticheng-aws ticheng-aws changed the title Update _api-reference/profile.md Update search profile API documentation with concurrency related changes Dec 26, 2023
@ticheng-aws ticheng-aws marked this pull request as ready for review December 26, 2023 19:53
@ticheng-aws ticheng-aws self-assigned this Dec 26, 2023
@hdhalter hdhalter added this to the v2.12 milestone Dec 26, 2023
@hdhalter hdhalter added 1 - Backlog - DEV Developer assigned to issue is responsible for creating PR. and removed 1 - Backlog - DEV Developer assigned to issue is responsible for creating PR. labels Dec 26, 2023
@hdhalter hdhalter added the 4 - Doc review PR: Doc review in progress label Dec 26, 2023
Signed-off-by: Ticheng Lin <ticheng@amazon.com>
@kolchfa-aws kolchfa-aws added the release-notes PR: Include this PR in the automated release notes label Jan 2, 2024
Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>
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, @ticheng-aws! Please confirm the technical accuracy of my changes, and then we'll be ready to move the PR to editorial review.

_api-reference/profile.md Outdated Show resolved Hide resolved
_api-reference/profile.md Outdated Show resolved Hide resolved
_api-reference/profile.md Outdated Show resolved Hide resolved
@ticheng-aws ticheng-aws requested a review from sohami January 4, 2024 00:25
Signed-off-by: Ticheng Lin <ticheng@amazon.com>
Signed-off-by: Fanit Kolchina <kolchfa@amazon.com>
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 kolchfa-aws removed the 4 - Doc review PR: Doc review in progress label Jan 5, 2024
@kolchfa-aws kolchfa-aws added the 6 - Done but waiting to merge PR: The work is done and ready to merge label Jan 5, 2024
@hdhalter hdhalter merged commit 796008b into opensearch-project:main Feb 1, 2024
4 checks passed
@hdhalter hdhalter added 3 - Done Issue is done/complete and removed 6 - Done but waiting to merge PR: The work is done and ready to merge labels 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.

[DOC] Update search profile API documentation with concurrency related changes
5 participants