Skip to content

Commit bb6f650

Browse files
authored
Remove MicrometerCacheMetrics duplicate component (#246)
1 parent c3ff72e commit bb6f650

File tree

9 files changed

+8
-22
lines changed

9 files changed

+8
-22
lines changed

build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ nexusStaging {
2020

2121
allprojects {
2222
group = "ru.tinkoff.kora"
23-
version = System.getenv().getOrDefault("KORA_VERSION", "0.13.1-SNAPSHOT")
23+
version = System.getenv().getOrDefault("KORA_VERSION", "0.14.0-SNAPSHOT")
2424
repositories {
2525
mavenCentral()
2626
}

cache/cache-caffeine/src/main/java/ru/tinkoff/kora/cache/caffeine/AbstractCaffeineCache.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package ru.tinkoff.kora.cache.caffeine;
22

3-
import org.jetbrains.annotations.ApiStatus.Internal;
43
import reactor.core.publisher.Mono;
54

65
import javax.annotation.Nonnull;
@@ -11,7 +10,6 @@
1110
import java.util.Set;
1211
import java.util.function.Function;
1312

14-
@Internal
1513
public abstract class AbstractCaffeineCache<K, V> implements CaffeineCache<K, V> {
1614

1715
private final String name;

cache/cache-caffeine/src/main/java/ru/tinkoff/kora/cache/caffeine/CaffeineCacheTelemetry.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
import javax.annotation.Nonnull;
1010
import javax.annotation.Nullable;
1111

12-
//TODO caffeine metrics???
1312
public final class CaffeineCacheTelemetry {
1413

1514
private static final String ORIGIN = "caffeine";

cache/cache-redis/src/main/java/ru/tinkoff/kora/cache/redis/AbstractRedisCache.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
package ru.tinkoff.kora.cache.redis;
22

3-
import org.jetbrains.annotations.ApiStatus.Internal;
43
import reactor.core.publisher.Flux;
54
import reactor.core.publisher.Mono;
65
import ru.tinkoff.kora.cache.Cache;
@@ -12,7 +11,6 @@
1211
import java.util.function.Function;
1312
import java.util.stream.Collectors;
1413

15-
@Internal
1614
public abstract class AbstractRedisCache<K, V> implements Cache<K, V> {
1715

1816
private final String name;

cache/cache-redis/src/main/java/ru/tinkoff/kora/cache/redis/client/LettuceClientFactory.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,12 @@
55
import io.lettuce.core.cluster.RedisClusterClient;
66
import io.lettuce.core.cluster.RedisClusterURIUtil;
77
import io.lettuce.core.protocol.ProtocolVersion;
8-
import org.jetbrains.annotations.ApiStatus.Internal;
98

109
import javax.annotation.Nonnull;
1110
import java.net.URI;
1211
import java.time.Duration;
1312
import java.util.List;
1413

15-
@Internal
1614
public final class LettuceClientFactory {
1715

1816
@Nonnull

dependencies.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ dependencyResolutionManagement { DependencyResolutionManagement it ->
102102

103103
library('lettuce-core', 'io.lettuce', 'lettuce-core').version('6.2.0.RELEASE')
104104

105-
library('caffeine', 'com.github.ben-manes.caffeine', 'caffeine').version('3.1.1')
105+
library('caffeine', 'com.github.ben-manes.caffeine', 'caffeine').version('3.1.7')
106106

107107
library("awaitility", "org.awaitility", "awaitility").version("4.2.0")
108108
library("junit-jupiter", "org.junit.jupiter", "junit-jupiter").version("5.9.1")

micrometer/micrometer-module/build.gradle

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,9 @@ dependencies {
33
api libs.micrometer.registry.prometheus
44
api libs.prometheus.jmx.collector
55
api libs.opentelemetry.semconv
6+
api(libs.prometheus.collector.caffeine) {
7+
exclude group: "com.github.ben-manes.caffeine"
8+
}
69
annotationProcessor project(':config:config-annotation-processor')
710

811
compileOnly project(':http:http-server-common')
@@ -16,5 +19,4 @@ dependencies {
1619
compileOnly project(':resilient:resilient-kora')
1720
compileOnly project(':cache:cache-common')
1821
compileOnly project(':cache:cache-caffeine')
19-
compileOnly libs.prometheus.collector.caffeine
2022
}

micrometer/micrometer-module/src/main/java/ru/tinkoff/kora/micrometer/module/MetricsModule.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -160,11 +160,6 @@ default MicrometerCacheMetrics micrometerCacheMetrics(MeterRegistry meterRegistr
160160
return new MicrometerCacheMetrics(meterRegistry);
161161
}
162162

163-
@DefaultComponent
164-
default MicrometerCacheMetrics micrometerCaffeineCacheMetrics(MeterRegistry meterRegistry) {
165-
return new MicrometerCacheMetrics(meterRegistry);
166-
}
167-
168163
@DefaultComponent
169164
default CacheMetricsCollector cacheMetricsCollector() {
170165
return new CacheMetricsCollector().register();

mkdocs/docs/features/cache.md

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -81,17 +81,13 @@ public interface DummyCache extends CaffeineCache<CacheKey.Key2<String, BigDecim
8181

8282
Для кэширования и получения значения из кэша для метода *getValue()* следует проаннотировать его аннотацией *@Cacheable*.
8383

84-
Метод проаннотированный *@Cacheable* будет пытаться взять значение по ключу из кэша который указан в *value*, в случае если значение для такого ключа не существует,
85-
будет вызван сам метод и его значение будет закэшированно для последующих операций и возвращено.
86-
Имя кэша из *value* соответствует его имени в конфигурации файла (hocon)
87-
8884
Ключ для кэша составляет из аргументов метода, порядок аргументов имеет значение, в данном случае он будет составляться из значений *arg1* и *arg2*.
8985

9086
```java
9187
@Component
9288
public class CacheExample {
9389

94-
@Cacheable()
90+
@Cacheable(DummyCache.class)
9591
public Long getValue(String arg1, BigDecimal arg2) {
9692
return ThreadLocalRandom.current().nextLong();
9793
}
@@ -200,7 +196,7 @@ public class CacheExample {
200196
// do nothing
201197
}
202198

203-
@CacheInvalidate(DummyCache.class, invalidateAll = true)
199+
@CacheInvalidate(value = DummyCache.class, invalidateAll = true)
204200
public void evictAll() {
205201
// do nothing
206202
}
@@ -427,7 +423,7 @@ public interface ApplicationModules extends RedisCacheModule { }
427423
@Module
428424
interface ApplicationModules {
429425
default LoadableCache<String, String> someEntityLoadCache(DummyCache cache, SomeService someService, ExecutorService executor) {
430-
return LoadableCache.create(cache, CacheLoader.blocking(someService::loadEntity, executor));
426+
return cache.asLoadable(CacheLoader.blocking(someService::loadEntity, executor));
431427
}
432428
}
433429

0 commit comments

Comments
 (0)