-
Notifications
You must be signed in to change notification settings - Fork 14
ekump/APMSP-2185 extract transport module for trace exporter #1164
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
ekump/APMSP-2185 extract transport module for trace exporter #1164
Conversation
BenchmarksComparisonBenchmark execution time: 2025-07-31 17:41:46 Comparing candidate commit 426e0f4 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 53 metrics, 2 unstable metrics. 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
Group 13
Group 14
BaselineOmitted due to size. |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1164 +/- ##
==========================================
+ Coverage 70.90% 71.26% +0.35%
==========================================
Files 346 347 +1
Lines 54084 54256 +172
==========================================
+ Hits 38350 38665 +315
+ Misses 15734 15591 -143
🚀 New features to boost your workflow:
|
7f4b6ef
to
da8b3ef
Compare
cc90c44
to
5090f06
Compare
da8b3ef
to
426e0f4
Compare
/merge |
View all feedbacks in Devflow UI.
The expected merge time in
|
What does this PR do?
Just moving transport related code out of the main Trace Exporter module into a separate transport one. No functionality changes.
Motivation
Trace Exporter cleanup ahead of rearchitecture work
Additional Notes
Anything else we should know when reviewing?
How to test the change?
Describe here in detail how the change can be validated.