newrelic / newrelic-android-agent

SDK to enable instrumentation of Android mobile apps in New Relic
Apache License 2.0
15 stars 15 forks source link

[NR-293384] Implement AEI trace reporting #302

Closed cthomas-newrelic closed 2 weeks ago

cthomas-newrelic commented 3 weeks ago

Following the existing store-and-forward pattern used by log reporting, add a new reporter and sender component to manage AEI trace uploads. Traces are cached to disk, uploaded during harvest cycles and removed from disk once the upload is successful or has expired.

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 58.41584% with 84 lines in your changes missing coverage. Please review.

Project coverage is 57.56%. Comparing base (bb25f63) to head (9bc3a8c).

Files with missing lines Patch % Lines
...m/newrelic/agent/android/aei/AEITraceReporter.java 48.14% 49 Missing and 21 partials :warning:
...elic/agent/android/aei/ApplicationExitMonitor.java 61.90% 3 Missing and 5 partials :warning:
...com/newrelic/agent/android/aei/AEITraceSender.java 92.68% 3 Missing :warning:
.../com/newrelic/agent/android/aei/SessionMapper.java 50.00% 2 Missing :warning:
...relic/agent/android/payload/FileBackedPayload.java 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #302 +/- ## ============================================= + Coverage 57.50% 57.56% +0.05% - Complexity 3398 3442 +44 ============================================= Files 328 330 +2 Lines 16803 16989 +186 Branches 1835 1858 +23 ============================================= + Hits 9663 9779 +116 - Misses 6105 6153 +48 - Partials 1035 1057 +22 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

cthomas-newrelic commented 2 weeks ago

Hmm, flakey tests. Hold off on review until resolved.

cthomas-newrelic commented 2 weeks ago

Do -Over: Closed to clean up and rebase branch.