nasa / Common-Metadata-Repository

Apache License 2.0
340 stars 94 forks source link

CMR-9748 Moving collection metadata cache to its own and separating r… #2069

Closed eereiter closed 8 months ago

eereiter commented 8 months ago

…eaders and writers.

Overview

What is the feature/fix?

Moving collection metadata cache to its own and separating readers and writers.

What is the Solution?

Added some new configuration parameters for the new cache and updated the redis code to use them based on the passed in cache key.

What areas of the application does this impact?

redis

Checklist

codecov-commenter commented 8 months ago

Codecov Report

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

Project coverage is 29.15%. Comparing base (9a7f86e) to head (761e481).

Files Patch % Lines
...utils-lib/src/cmr/redis_utils/redis_hash_cache.clj 0.00% 30 Missing :warning:
.../search/data/metadata_retrieval/metadata_cache.clj 6.66% 28 Missing :warning:
...h/services/humanizers/humanizer_report_service.clj 3.84% 25 Missing :warning:
redis-utils-lib/src/cmr/redis_utils/config.clj 33.33% 15 Missing and 7 partials :warning:
...a/metadata_retrieval/collection_metadata_cache.clj 0.00% 16 Missing :warning:
...edis-utils-lib/src/cmr/redis_utils/redis_cache.clj 0.00% 12 Missing :warning:
redis-utils-lib/src/cmr/redis_utils/redis.clj 42.85% 7 Missing and 1 partial :warning:
...mon-app-lib/src/cmr/common_app/services/search.clj 14.28% 6 Missing :warning:
...execution/has_granules_or_cwic_results_feature.clj 0.00% 5 Missing :warning:
...a/metadata_retrieval/collection_metadata_cache.clj 20.00% 4 Missing :warning:
... and 9 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2069 +/- ## =========================================== - Coverage 58.09% 29.15% -28.95% =========================================== Files 1043 1043 Lines 69655 69749 +94 Branches 1944 1223 -721 =========================================== - Hits 40466 20332 -20134 - Misses 27359 48195 +20836 + Partials 1830 1222 -608 ```

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