Cache Metrics Broken on JDK > 11

[broadcom-tony]

Expected Behavior

All caching metrics should work cache_size, cache_puts, hits, etc.

Actual Behaviour

cache_size metrics work but cache_puts, hits, etc do not increment. This issue describes the behavior:

#94

I believe the issue is that javax was pulled out of the JDK in version 11 and greater however the JCacheMetricsBinder.java and potentially other places @require javax.cache.CacheManager to load their bean.

See:

https://github.com/micronaut-projects/micronaut-cache/blob/master/cache-core/src/main/java/io/micronaut/cache/jcache/metrics/JCacheMetricsBinder.java#L42

Steps To Reproduce

No response

Environment Information

• Linux/MacOs
• JDK 17

`
id "io.micronaut.minimal.application" version "3.7.8"
micronaut {
version = "3.9.1"
}
implementation "io.micronaut.micrometer:micronaut-micrometer-core"
implementation "io.micronaut.micrometer:micronaut-micrometer-registry-prometheus"
implementation "io.micronaut.cache:micronaut-cache-core"
implementation "io.micronaut.cache:micronaut-cache-caffeine"

`

Example Application

No response

Version

3.9.1

[sdelamo]

does it work if you add javax.cache:cache-api:1.1.1to your application?

[Richardson-e]

I was able to reproduce this issue with micronaut 4.0.0
I added implementation ('javax.cache:cache-api:1.1.1') to the build.gradle file, checked it was in my classpath and it did not resolve the problem. Metric cache.size works(as it did previously), but cache.gets nor cache.puts increment as it should.

[sdelamo]

do you have a sample application @Richardson-e ?

[ben-manes]

fyi, in JCache you have to enable stats as part of the cache configuration. For Caffeine, if using the config file,

caffeine.jcache.default.monitoring.statistics = true

where default is the base template, or can be a specific cache name.

Since JCache requires JMX (management = true) you can enable and verify in jconsole separately from other reporters.

[Richardson-e]

do you have a sample application @Richardson-e ?

https://github.com/Richardson-e/hello-world-caffeine-cache

I'll take a look at explicitly enabling stats for jcache as part of the cache config.