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

Use cat snapshot API to get the number of snapshot for a repository #1242

Merged

Conversation

skumawat2025
Copy link
Contributor

@skumawat2025 skumawat2025 commented Jan 21, 2025

Description

This PR addresses a performance issue in the repository page loading process by replacing the Get Snapshot API with the Cat Snapshot API for retrieving the number of snapshots. The Cat Snapshot API uses minimal information from backend storage calls, which should reduce load time and improve performance when loading repository information.

Issues Resolved

[List any issues this PR will resolve]

Check List

  • Commits are signed per the DCO using --signoff

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

@skumawat2025
Copy link
Contributor Author

@skumawat2025 skumawat2025 changed the title Use cat snapshot to get the number of snapshot for a repository Use cat snapshot API to get the number of snapshot for a repository Jan 21, 2025
@RamakrishnaChilaka RamakrishnaChilaka merged commit c459af0 into opensearch-project:main Jan 22, 2025
10 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jan 24, 2025
Signed-off-by: Sandeep Kumawat <[email protected]>
Co-authored-by: Sandeep Kumawat <[email protected]>
(cherry picked from commit c459af0)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
skumawat2025 added a commit to skumawat2025/index-management-dashboards-plugin that referenced this pull request Jan 31, 2025
skumawat2025 added a commit to skumawat2025/index-management-dashboards-plugin that referenced this pull request Jan 31, 2025
skumawat2025 added a commit to skumawat2025/index-management-dashboards-plugin that referenced this pull request Jan 31, 2025
RamakrishnaChilaka pushed a commit that referenced this pull request Jan 31, 2025
skumawat2025 added a commit to skumawat2025/index-management-dashboards-plugin that referenced this pull request Jan 31, 2025
skumawat2025 added a commit to skumawat2025/index-management-dashboards-plugin that referenced this pull request Jan 31, 2025
RamakrishnaChilaka pushed a commit that referenced this pull request Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants