search

newrelic / newrelic-java-agent

The New Relic Java agent
Apache License 2.0
202 stars 143 forks source link

1757 log sender sends too much #1784

Closed jbedell-newrelic closed 7 months ago

jbedell-newrelic commented 8 months ago

Store off the reportPeriodInMillis from the backend, and use it to calculate the proper maxSamplesStored on config file change.

codecov-commenter commented 8 months ago

Codecov Report

Attention: Patch coverage is 85.71429% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 70.86%. Comparing base (44d1dd5) to head (b9d8a13).

Files Patch % Lines
...ic/agent/service/logging/LogSenderServiceImpl.java 80.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1784 +/- ## ============================================ + Coverage 70.85% 70.86% +0.01% - Complexity 9944 9948 +4 ============================================ Files 826 827 +1 Lines 39868 39873 +5 Branches 6040 6040 ============================================ + Hits 28248 28257 +9 + Misses 8892 8891 -1 + Partials 2728 2725 -3 ```

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