-
Notifications
You must be signed in to change notification settings - Fork 10
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
feature: ddcommon-net2 crate #766
base: levi/split-net
Are you sure you want to change the base?
Conversation
BenchmarksComparisonBenchmark execution time: 2024-12-04 23:00:05 Comparing candidate commit 3c7a240 in PR branch Found 1 performance improvements and 6 performance regressions! Performance is the same for 44 metrics, 2 unstable metrics. scenario:credit_card/is_card_number/x371413321323331
scenario:credit_card/is_card_number_no_luhn/ 3782-8224-6310-005
scenario:credit_card/is_card_number_no_luhn/x371413321323331
scenario:redis/obfuscate_redis_string
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. |
be8197d
to
55eec29
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## levi/split-net #766 +/- ##
==================================================
- Coverage 70.55% 70.17% -0.38%
==================================================
Files 297 300 +3
Lines 43547 43849 +302
==================================================
+ Hits 30726 30773 +47
- Misses 12821 13076 +255
|
* Bump rustls from 0.23.16 to 0.23.18 Bumps [rustls](https://github.com/rustls/rustls) from 0.23.16 to 0.23.18. - [Release notes](https://github.com/rustls/rustls/releases) - [Changelog](https://github.com/rustls/rustls/blob/main/CHANGELOG.md) - [Commits](rustls/rustls@v/0.23.16...v/0.23.18) --- updated-dependencies: - dependency-name: rustls dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> * Update LICENSE-3rdparty.yml with package version bump --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Ivo Anjo <[email protected]>
* Fix cmake target directory.
f9b975a
to
b8c5304
Compare
…rlying for TinyBytes::Bytes (#764) Update data-pipeline's send function to accept a tinybytes::Bytes for the payload type. Also update the data-pipeline-ffi ddog_trace_exporter_send function to transmute the incoming ByteSlice to static. The ByteSlice is expected to live for the length of the function call, but it is technically possible to borrow the underlying data for longer, leading to stale references. This will be addressed as part of APMSP-1621.
b8c5304
to
dd97fbf
Compare
fix: install ring CyrptoProvider manually For some reason I do not yet understand, this used to happen automatically but is not happening, and it causes a crashtracker test `crash_tracking_empty_endpoint` to fail because it doesn't have a CryptoProvider.
fd054f7
to
24c38f7
Compare
What does this PR do?
This adds the
ddcommon-net2
crate which uses hyper 1, unlikeddcommon-net1
which uses hyper 0.14. Thedatadog-profiling
anddatadog-profiling-ffi
crates are moved to useddcommon-net2
.Motivation
Hyper migration.
Additional Notes
I was hoping that this specific implementation strategy would get rid of a specific linking issue I hit on Windows in another PR. However,
ddcommon-ffi
depends onddcommon-net1
so both net crates are in play. The relationships are a bit different (better, IMO) but it didn't outright fix the issue. I think all crates built together on Windows need to have the same hyper version dependency :'(How to test the change?
Describe here in detail how the change can be validated.