-
Notifications
You must be signed in to change notification settings - Fork 1.6k
[chore] Bytes based batching for profiles #12564
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
Conversation
Codecov ReportAttention: Patch coverage is
❌ Your patch status has failed because the patch coverage (79.54%) is below the target coverage (95.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #12564 +/- ##
==========================================
- Coverage 91.46% 91.38% -0.08%
==========================================
Files 483 484 +1
Lines 26624 26695 +71
==========================================
+ Hits 24351 24396 +45
- Misses 1791 1817 +26
Partials 482 482 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
8c64251
to
ddedc5b
Compare
ddedc5b
to
e43b829
Compare
Fixed conflicts. Can you review? @dmitryax @bogdandrutu |
2d45881
to
6979d6f
Compare
7ed4333
to
7057322
Compare
I believe this actually needs the changelog item now given that we enabled the bytes batching already |
Signed-off-by: Israel Blancas <[email protected]>
7057322
to
117d259
Compare
Added. I think we are ready to go. |
63a22b2
Description
This PR implements serialized bytes based batching for metrics.
Link to tracking issue
#3262
Continuation of #12299 made by @sfc-gh-sili.
Related PRs: