-
Notifications
You must be signed in to change notification settings - Fork 8
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
[APMSP-1391] Wait for spawned children #651
base: main
Are you sure you want to change the base?
Conversation
BenchmarksComparisonBenchmark execution time: 2024-09-27 13:23:14 Comparing candidate commit 60480db in PR branch Found 0 performance improvements and 1 performance regressions! Performance is the same for 50 metrics, 2 unstable metrics. scenario:benching deserializing traces from msgpack to their internal representation
CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
BaselineOmitted due to size. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #651 +/- ##
==========================================
+ Coverage 73.82% 73.85% +0.03%
==========================================
Files 257 257
Lines 37318 37301 -17
==========================================
- Hits 27549 27548 -1
+ Misses 9769 9753 -16
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - But can you just associate a JIRA ticket to this PR before merging?
8a28d32
to
ba13c1c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
ba13c1c
to
d80a2b9
Compare
d80a2b9
to
60480db
Compare
What does this PR do?
Actively waiting for spawned children in the builder in order to avoid concurrency problems.
Motivation
Letting child process on their own could lead to issues if the parent process finish before them or another subsequent command is added that operates on the same resources.