Transient caches: Include exceeded limit into warning log
This allows to double-check that a configured limit is taking effect.
Release-Notes: skip
Change-Id: Ib5c9b979b06f9b8c4e4c66f274420bea8effbb0b
Signed-off-by: Edwin Kempin <ekempin@google.com>
diff --git a/java/com/google/gerrit/plugins/codeowners/backend/TransientCodeOwnerCache.java b/java/com/google/gerrit/plugins/codeowners/backend/TransientCodeOwnerCache.java
index 0229860..a869df8 100644
--- a/java/com/google/gerrit/plugins/codeowners/backend/TransientCodeOwnerCache.java
+++ b/java/com/google/gerrit/plugins/codeowners/backend/TransientCodeOwnerCache.java
@@ -78,7 +78,7 @@
cache.put(email, codeOwner);
} else if (maxCacheSize.isPresent()) {
logger.atWarning().atMostEvery(1, TimeUnit.DAYS).log(
- "exceeded limit of %s", getClass().getSimpleName());
+ "exceeded limit of %s (limit = %s)", getClass().getSimpleName(), maxCacheSize.get());
}
}
diff --git a/java/com/google/gerrit/plugins/codeowners/backend/TransientCodeOwnerConfigCache.java b/java/com/google/gerrit/plugins/codeowners/backend/TransientCodeOwnerConfigCache.java
index b1a38fa..2ce34dd 100644
--- a/java/com/google/gerrit/plugins/codeowners/backend/TransientCodeOwnerConfigCache.java
+++ b/java/com/google/gerrit/plugins/codeowners/backend/TransientCodeOwnerConfigCache.java
@@ -106,8 +106,8 @@
cache.put(cacheKey, codeOwnerConfig);
} else if (maxCacheSize.isPresent()) {
logger.atWarning().atMostEvery(1, TimeUnit.DAYS).log(
- "exceeded limit of %s (project = %s)",
- getClass().getSimpleName(), cacheKey.codeOwnerConfigKey().project());
+ "exceeded limit of %s (project = %s, limit = %s)",
+ getClass().getSimpleName(), cacheKey.codeOwnerConfigKey().project(), maxCacheSize.get());
}
return codeOwnerConfig;
}
