-
Notifications
You must be signed in to change notification settings - Fork 151
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
Improve sampler #2761
Improve sampler #2761
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2761 +/- ##
============================================
- Coverage 79.52% 77.96% -1.57%
Complexity 2216 2216
============================================
Files 201 227 +26
Lines 22619 26644 +4025
Branches 0 989 +989
============================================
+ Hits 17987 20772 +2785
- Misses 4632 5346 +714
- Partials 0 526 +526
Flags with carried forward coverage won't be shown. Click here to find out more. see 26 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
94ab3b5
to
2b7101a
Compare
873f794
to
3a9942b
Compare
Description
There have been error when more than one request was running on parallel due to the sampler not managing it properly. This PR refactor the sampler to stop these errors.
APPSEC-53993
Reviewer checklist