-
Notifications
You must be signed in to change notification settings - Fork 412
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
chore(asm): avoid double calls on some aspect front-ends #9344
Conversation
Signed-off-by: Juanjo Alvarez <[email protected]>
Signed-off-by: Juanjo Alvarez <[email protected]>
Datadog ReportBranch report: ✅ 0 Failed, 109746 Passed, 3758 Skipped, 6m 25.39s Total duration (41m 56.16s time saved) |
BenchmarksBenchmark execution time: 2024-05-22 12:02:47 Comparing candidate commit 2010dbf in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 209 metrics, 9 unstable metrics. |
Signed-off-by: Juanjo Alvarez <[email protected]>
Signed-off-by: Juanjo Alvarez <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9344 +/- ##
===========================================
- Coverage 76.02% 10.25% -65.77%
===========================================
Files 1294 1266 -28
Lines 122816 121219 -1597
===========================================
- Hits 93370 12437 -80933
- Misses 29446 108782 +79336 ☔ View full report in Codecov by Sentry. |
Description
Some aspects (index, slice, ljust, format, encode, decode and replace) were calling the original function and then potentially the native aspect, which would call again the original function. This reworks these aspect front-ends so only one call is ever done, which should improve performance a little.
This refactor also arised and error where we were not checking the CPython error with the new
has_pyerror()
helper in the nativeapi_aspect_index
so it fixed that too.Checklist
changelog/no-changelog
is set@DataDog/apm-tees
.Reviewer Checklist