From 3e883a7b78b6a22a788e9804171d45080cd54559 Mon Sep 17 00:00:00 2001 From: Hector Oswaldo Caballero Date: Thu, 8 Feb 2018 12:28:41 -0500 Subject: [PATCH] LuceneVersionManager: Fix log error messages The built-in log formatter uses curly braces instead of '%s'. Without this change, log messages look like: WARN com.google.gerrit.lucene.LuceneVersionManager: Not a directory: %s" Change-Id: I60b4c0a788c01f0a48aa8c0f7f9c82d7c380e632 --- .../java/com/google/gerrit/lucene/LuceneVersionManager.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gerrit-lucene/src/main/java/com/google/gerrit/lucene/LuceneVersionManager.java b/gerrit-lucene/src/main/java/com/google/gerrit/lucene/LuceneVersionManager.java index b46f1f604a..f05f87953b 100644 --- a/gerrit-lucene/src/main/java/com/google/gerrit/lucene/LuceneVersionManager.java +++ b/gerrit-lucene/src/main/java/com/google/gerrit/lucene/LuceneVersionManager.java @@ -111,7 +111,7 @@ public class LuceneVersionManager implements LifecycleListener { if (!Files.exists(sitePaths.index_dir)) { throw new ProvisionException(runReindexMsg); } else if (!Files.exists(sitePaths.index_dir)) { - log.warn("Not a directory: %s", sitePaths.index_dir.toAbsolutePath()); + log.warn("Not a directory: {}", sitePaths.index_dir.toAbsolutePath()); throw new ProvisionException(runReindexMsg); } @@ -233,7 +233,7 @@ public class LuceneVersionManager implements LifecycleListener { Path p = getDir(sitePaths, def.getName(), schema); boolean isDir = Files.isDirectory(p); if (Files.exists(p) && !isDir) { - log.warn("Not a directory: %s", p.toAbsolutePath()); + log.warn("Not a directory: {}", p.toAbsolutePath()); } int v = schema.getVersion(); versions.put(v, new Version<>(