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.

Sign up for GitHub

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

Jump to bottom

Server Side batching for saving UI metrics #6375

Closed
LDrago27 opened this issue Apr 9, 2024 · 0 comments · Fixed by #6721
Closed

Server Side batching for saving UI metrics #6375

LDrago27 opened this issue Apr 9, 2024 · 0 comments · Fixed by #6721
Labels
enhancement New feature or request

Comments

@LDrago27
Copy link
Collaborator

LDrago27 commented Apr 9, 2024

Is your feature request related to a problem? Please describe.
This Issue is a follow up to the PR entailed here . (#6203). As part of the above PR Ui metrics collectors were enabled , however there was a concern of performance hit if a large number of concurrent users happened to save metrics into the saved object at the same time.

Describe the solution you'd like
A server side batching solution would enable us to batch all the saved object write operations for Ui metrics into a single write operation reducing the performance hit on the backend and adversely affecting users.

Describe alternatives you've considered
@LDrago27 LDrago27 added the enhancement New feature or request label Apr 9, 2024
@LDrago27 LDrago27 mentioned this issue Apr 9, 2024
Merged
7 tasks
@LDrago27 LDrago27 mentioned this issue May 6, 2024
Merged
7 tasks
@kavilla kavilla linked a pull request May 8, 2024 that will close this issue
Add Server Side batching for UI Metric Collectors #6721
Merged
7 tasks
kavilla pushed a commit that referenced this issue May 22, 2024
@LDrago27 @opensearch-changeset-bot
Add Server Side batching for UI Metric Collectors (#6721)
972cf10 
* Add Server Side batching for UI Metric Collectors

Signed-off-by: Suchit Sahoo <suchsah@amazon.com>

* Changeset file for PR #6721 created/updated

This PR introduces Server Side batching of UI metric reports sent by clients to reduce the number of updates taking place at OpenSearch.
Issues Resolved

#6375

---------

Signed-off-by: Suchit Sahoo <suchsah@amazon.com>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this issue May 22, 2024
@github-actions @opensearch-changeset-bot
Add Server Side batching for UI Metric Collectors (#6721)
bdb702a 
* Add Server Side batching for UI Metric Collectors

Signed-off-by: Suchit Sahoo <suchsah@amazon.com>

* Changeset file for PR #6721 created/updated

This PR introduces Server Side batching of UI metric reports sent by clients to reduce the number of updates taking place at OpenSearch.
Issues Resolved

#6375

---------

Signed-off-by: Suchit Sahoo <suchsah@amazon.com>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
(cherry picked from commit 972cf10)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
kavilla pushed a commit that referenced this issue May 23, 2024
@opensearch-trigger-bot @github-actions @opensearch-changeset-bot
Add Server Side batching for UI Metric Collectors (#6721) (#6819)
ecfaed5 
* Add Server Side batching for UI Metric Collectors



* Changeset file for PR #6721 created/updated

This PR introduces Server Side batching of UI metric reports sent by clients to reduce the number of updates taking place at OpenSearch.
Issues Resolved

#6375

---------



(cherry picked from commit 972cf10)

Signed-off-by: Suchit Sahoo <suchsah@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
LDrago27 added a commit to LDrago27/OpenSearch-Dashboards that referenced this issue Jun 3, 2024
@LDrago27 @opensearch-changeset-bot
Add Server Side batching for UI Metric Collectors (opensearch-project…
151e23c 
…#6721)

* Add Server Side batching for UI Metric Collectors

Signed-off-by: Suchit Sahoo <suchsah@amazon.com>

* Changeset file for PR opensearch-project#6721 created/updated

This PR introduces Server Side batching of UI metric reports sent by clients to reduce the number of updates taking place at OpenSearch.
Issues Resolved

opensearch-project#6375

---------

Signed-off-by: Suchit Sahoo <suchsah@amazon.com>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Server Side batching for UI Metric Collectors LDrago27/OpenSearch-Dashboards
1 participant
@LDrago27