-
Notifications
You must be signed in to change notification settings - Fork 9
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
Deduplicate Debugger diagnostics in sidecar #736
Conversation
91174be
to
698ee10
Compare
BenchmarksComparisonBenchmark execution time: 2024-11-15 16:42:40 Comparing candidate commit a059912 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 51 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
BaselineOmitted due to size. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #736 +/- ##
==========================================
- Coverage 70.58% 70.57% -0.02%
==========================================
Files 295 296 +1
Lines 43096 43275 +179
==========================================
+ Hits 30420 30542 +122
- Misses 12676 12733 +57
|
698ee10
to
8c256a6
Compare
Signed-off-by: Bob Weinand <[email protected]>
8c256a6
to
fc9228a
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.
I can't comment on the rust code as I don't have experience with rust. Approving because the logic looks sound to me. Thanks!
We were sending too much data and that made our backend sad.
Sending less data so that our backend can be happy again.
This avoids sending multiple diagnostics for individual runtime ids.