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; }