Skip to content

ATLAS-4916 : Improve APIs where AtlasAccessRequest is used #374

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

VinayakMarraiya230920
Copy link
Contributor

@VinayakMarraiya230920 VinayakMarraiya230920 commented Jun 5, 2025

What changes were proposed in this pull request?

1.Refactored code to reduce the time taken for processing classification updates for large numbers of entities.
2.Optimized internal method calls to avoid redundant operations.

How was this patch tested?

1.This patch was tested by measuring the time taken by the bulk/classification API to handle 260 items, both before and after the code changes.

2.The test was done using different numbers of classifications (3, 5, 10, and 20). The results were averaged over three runs to ensure accuracy.

Results after code changes:
Classifications Entities Time Taken (seconds)
20 260 93.48
10 260 39.33
5 260 25.55
3 260 18.07

Results before code changes:
Classifications Entities Time Taken (seconds)
20 260 157.67
10 260 78.84
5 260 39.42
3 260 23.65

3.The reading above was taken three times and then averaged to get the final result.
4.Additionally, logs for method-level performance changes were verified in metrics.log.

@VinayakMarraiya230920 VinayakMarraiya230920 force-pushed the ATLAS-4916 branch 2 times, most recently from 86ee07a to 58feb78 Compare June 5, 2025 12:36
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.

1 participant