Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

perf(cache2k): disable statistics and configure eternal#183

Merged
iProdigy merged 1 commit intomainXanthic/cache-api:mainfrom
perf/cache2k-statistics-eternalXanthic/cache-api:perf/cache2k-statistics-eternalCopy head branch name to clipboard
Oct 8, 2023
Merged

perf(cache2k): disable statistics and configure eternal#183
iProdigy merged 1 commit intomainXanthic/cache-api:mainfrom
perf/cache2k-statistics-eternalXanthic/cache-api:perf/cache2k-statistics-eternalCopy head branch name to clipboard

Conversation

@iProdigy
Copy link
Member

@iProdigy iProdigy commented Oct 7, 2023

Cache2k enables statistics by default, and we don't offer an interface to access these statistics, so we can disable statistics for a performance boost.
In addition, when time-based expiry is not enabled, we can specify the eternal hint as another optimization (note: this correctly does not prevent size-based evictions).

@iProdigy iProdigy requested a review from PhilippHeuer October 7, 2023 19:06
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@iProdigy iProdigy enabled auto-merge (squash) October 7, 2023 19:37
@iProdigy iProdigy merged commit a3ede8c into main Oct 8, 2023
@PhilippHeuer PhilippHeuer deleted the perf/cache2k-statistics-eternal branch October 8, 2023 00:53
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.