DataDog / dd-sdk-android

Datadog SDK for Android (Compatible with Kotlin and Java)
Apache License 2.0
141 stars 57 forks source link

Remove the dd-sketches dependency and related logic #2062

Closed mariusc83 closed 1 month ago

mariusc83 commented 1 month ago

What does this PR do?

A brief description of the change being made with this pull request.

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

Review checklist (to be filled by reviewers)

codecov-commenter commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.30%. Comparing base (1dd0f18) to head (868fd0a).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## feature/otel-support #2062 +/- ## ======================================================== + Coverage 66.91% 67.30% +0.39% ======================================================== Files 737 733 -4 Lines 27244 27027 -217 Branches 4598 4573 -25 ======================================================== - Hits 18228 18188 -40 + Misses 7798 7616 -182 - Partials 1218 1223 +5 ``` | [Files](https://app.codecov.io/gh/DataDog/dd-sdk-android/pull/2062?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=DataDog) | Coverage Δ | | |---|---|---| | [...ace/bootstrap/instrumentation/api/AgentTracer.java](https://app.codecov.io/gh/DataDog/dd-sdk-android/pull/2062?src=pr&el=tree&filepath=features%2Fdd-sdk-android-trace%2Fsrc%2Fmain%2Fjava%2Fcom%2Fdatadog%2Ftrace%2Fbootstrap%2Finstrumentation%2Fapi%2FAgentTracer.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=DataDog#diff-ZmVhdHVyZXMvZGQtc2RrLWFuZHJvaWQtdHJhY2Uvc3JjL21haW4vamF2YS9jb20vZGF0YWRvZy90cmFjZS9ib290c3RyYXAvaW5zdHJ1bWVudGF0aW9uL2FwaS9BZ2VudFRyYWNlci5qYXZh) | `7.64% <ø> (+0.14%)` | :arrow_up: | | [...c/main/java/com/datadog/trace/core/CoreTracer.java](https://app.codecov.io/gh/DataDog/dd-sdk-android/pull/2062?src=pr&el=tree&filepath=features%2Fdd-sdk-android-trace%2Fsrc%2Fmain%2Fjava%2Fcom%2Fdatadog%2Ftrace%2Fcore%2FCoreTracer.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=DataDog#diff-ZmVhdHVyZXMvZGQtc2RrLWFuZHJvaWQtdHJhY2Uvc3JjL21haW4vamF2YS9jb20vZGF0YWRvZy90cmFjZS9jb3JlL0NvcmVUcmFjZXIuamF2YQ==) | `71.67% <ø> (+0.10%)` | :arrow_up: | | [...ace/core/datastreams/NoOpDataStreamMonitoring.java](https://app.codecov.io/gh/DataDog/dd-sdk-android/pull/2062?src=pr&el=tree&filepath=features%2Fdd-sdk-android-trace%2Fsrc%2Fmain%2Fjava%2Fcom%2Fdatadog%2Ftrace%2Fcore%2Fdatastreams%2FNoOpDataStreamMonitoring.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=DataDog#diff-ZmVhdHVyZXMvZGQtc2RrLWFuZHJvaWQtdHJhY2Uvc3JjL21haW4vamF2YS9jb20vZGF0YWRvZy90cmFjZS9jb3JlL2RhdGFzdHJlYW1zL05vT3BEYXRhU3RyZWFtTW9uaXRvcmluZy5qYXZh) | `23.08% <ø> (-5.49%)` | :arrow_down: | | [...atadog/trace/core/propagation/CorePropagation.java](https://app.codecov.io/gh/DataDog/dd-sdk-android/pull/2062?src=pr&el=tree&filepath=features%2Fdd-sdk-android-trace%2Fsrc%2Fmain%2Fjava%2Fcom%2Fdatadog%2Ftrace%2Fcore%2Fpropagation%2FCorePropagation.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=DataDog#diff-ZmVhdHVyZXMvZGQtc2RrLWFuZHJvaWQtdHJhY2Uvc3JjL21haW4vamF2YS9jb20vZGF0YWRvZy90cmFjZS9jb3JlL3Byb3BhZ2F0aW9uL0NvcmVQcm9wYWdhdGlvbi5qYXZh) | `28.57% <100.00%> (+0.79%)` | :arrow_up: | | [...og/trace/core/propagation/NoOpCorePropagation.java](https://app.codecov.io/gh/DataDog/dd-sdk-android/pull/2062?src=pr&el=tree&filepath=features%2Fdd-sdk-android-trace%2Fsrc%2Fmain%2Fjava%2Fcom%2Fdatadog%2Ftrace%2Fcore%2Fpropagation%2FNoOpCorePropagation.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=DataDog#diff-ZmVhdHVyZXMvZGQtc2RrLWFuZHJvaWQtdHJhY2Uvc3JjL21haW4vamF2YS9jb20vZGF0YWRvZy90cmFjZS9jb3JlL3Byb3BhZ2F0aW9uL05vT3BDb3JlUHJvcGFnYXRpb24uamF2YQ==) | `0.00% <ø> (ø)` | | ... and [26 files with indirect coverage changes](https://app.codecov.io/gh/DataDog/dd-sdk-android/pull/2062/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=DataDog)