Skip to content

added a memory-based Caffeine fetcher cache#347

Merged
jdhayhurst merged 1 commit into
masterfrom
fetcher_cache
Jun 2, 2026
Merged

added a memory-based Caffeine fetcher cache#347
jdhayhurst merged 1 commit into
masterfrom
fetcher_cache

Conversation

@jdhayhurst

Copy link
Copy Markdown
Contributor
  • Changed the fetcher cache from the simple cache (a hashmap) to a Caffeine implementation which prevents the cache from filling indefinitely.
  • An upper global cache pool size is configurable which is then divided equally between the many (~20) fetcher caches - this means we will be over and underspending the cache allocation, but it keeps the implementation simple.

separate cache for each fetcher

make cache method public so we can use in tests
@jdhayhurst jdhayhurst requested review from javfg and remo87 June 2, 2026 11:02
@jdhayhurst jdhayhurst merged commit be2167d into master Jun 2, 2026
2 checks passed
@jdhayhurst jdhayhurst deleted the fetcher_cache branch June 2, 2026 12:40
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.

2 participants