johnwteague commented 4 months ago

Overview

https://github.com/nasa/Common-Metadata-Repository/pull/1978 added several new logs to track transform strategy. This has added too much overhead to our event log creation and egress to the Splunk engine.

What is the feature/fix?

What is the Solution?

Downgrade 3 of the new logs from INFO to DEBUG to reduce log traffic Summarize what you changed.

What areas of the application does this impact?

List impacted areas.

Checklist

[ ] I have updated/added unit and integration tests that prove my fix is effective or that my feature works
[x] New and existing unit and int tests pass locally and remotely with my changes
[ ] I have removed unnecessary/dead code and imports in files I have changed
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

codecov-commenter commented 4 months ago

Codecov Report

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

Project coverage is 57.83%. Comparing base (ecb7153) to head (79e5775). Report is 2 commits behind head on master.

Files	Patch %	Lines
...h/data/metadata_retrieval/metadata_transformer.clj	33.33%	2 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #2105 +/- ## ========================================== - Coverage 57.83% 57.83% -0.01% ========================================== Files 1041 1041 Lines 70223 70223 Branches 1978 1978 ========================================== - Hits 40614 40611 -3 Misses 27746 27746 - Partials 1863 1866 +3 ```

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