Skip to content

CMR-9979 Remove Memory cache stats and JVM stats logging. - #2496

Merged
eereiter merged 4 commits into
masterfrom
CMR-9979
Sep 16, 2026
Merged

eereiter merged 4 commits into
masterfrom
CMR-9979

Conversation

@eereiter

Copy link
Copy Markdown
Contributor

Overview

What is the objective?

Remove the memory cache size log generation
Remove the CMR app jvm stats log generation

No one uses these logs.

What are the changes?

Remove the memory cache size log generation
Remove the CMR app jvm stats log generation

What areas of the application does this impact?

Removed the jobs that create the logs.

Required Checklist

  • New and existing unit and int tests pass locally and remotely
  • clj-kondo has been run locally and all errors in changed files are corrected
  • I have commented my code, particularly in hard-to-understand areas
  • I have made changes to the documentation (if necessary)
  • My changes generate no new warnings

Additional Checklist

  • I have removed unnecessary/dead code and imports in files I have changed
  • I have cleaned up integration tests by doing one or more of the following:
    • migrated any are2 tests to are3 in files I have changed
    • de-duped, consolidated, removed dead int tests
    • transformed applicable int tests into unit tests
    • reduced number of system state resets by updating fixtures. Ex) (use-fixtures :each (ingest/reset-fixture {})) to be :once instead of :each

@codecov-commenter

codecov-commenter commented Sep 15, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 37.50000% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 29.29%. Comparing base (a3f19a2) to head (3736c64).

Files with missing lines Patch % Lines
bootstrap-app/src/cmr/bootstrap/system.clj 50.00% 1 Missing ⚠️
indexer-app/src/cmr/indexer/system.clj 0.00% 1 Missing ⚠️
ingest-app/src/cmr/ingest/system.clj 0.00% 1 Missing ⚠️
search-app/src/cmr/search/system.clj 0.00% 1 Missing ⚠️
...ual-product-app/src/cmr/virtual_product/system.clj 0.00% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (a3f19a2) and HEAD (3736c64). Click for more details.

HEAD has 8 uploads less than BASE
Flag BASE (a3f19a2) HEAD (3736c64)
12 4
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #2496       +/-   ##
===========================================
- Coverage   57.92%   29.29%   -28.64%     
===========================================
  Files        1075     1013       -62     
  Lines       74666    71134     -3532     
  Branches     2178     1202      -976     
===========================================
- Hits        43250    20838    -22412     
- Misses      29381    49156    +19775     
+ Partials     2035     1140      -895     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@eereiter
eereiter merged commit 571e64b into master Sep 16, 2026
6 checks passed
@eereiter
eereiter deleted the CMR-9979 branch September 16, 2026 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants