commit 935b313dc9ba4dd57e2583218992436d0a1c7f7e
author Edwin Kempin <ekempin@google.com> Tue Mar 15 11:50:34 2022 +0100
committer Edwin Kempin <ekempin@google.com> Tue Mar 15 11:50:34 2022 +0100
tree 7183812f9239d9318611750ffdbc5b6dcef6bd56
parent b08a7ed5043b36613ccd027b34f08ae9d497ed6e

Use Flogger for writing logs

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: Ia60c0062b318ebef5037d37a70c52ec24b358626

src/main/java/com/googlesource/gerrit/plugins/simplesubmitrules/rules/NoUnresolvedCommentsRule.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/simplesubmitrules/rules/NoUnresolvedCommentsRule.java b/src/main/java/com/googlesource/gerrit/plugins/simplesubmitrules/rules/NoUnresolvedCommentsRule.java
index 25d6e31..099ca45 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/simplesubmitrules/rules/NoUnresolvedCommentsRule.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/simplesubmitrules/rules/NoUnresolvedCommentsRule.java
@@ -14,6 +14,7 @@
 
 package com.googlesource.gerrit.plugins.simplesubmitrules.rules;
 
+import com.google.common.flogger.FluentLogger;
 import com.google.gerrit.entities.LegacySubmitRequirement;
 import com.google.gerrit.entities.SubmitRecord;
 import com.google.gerrit.exceptions.StorageException;
@@ -28,13 +29,11 @@
 import com.googlesource.gerrit.plugins.simplesubmitrules.SimpleSubmitRulesConfig;
 import java.util.Collections;
 import java.util.Optional;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 /** Simple rule: block submission when unresolved comments are present. */
 @Singleton
 public class NoUnresolvedCommentsRule implements SubmitRule {
-  private static final Logger log = LoggerFactory.getLogger(NoUnresolvedCommentsRule.class);
+  private static final FluentLogger logger = FluentLogger.forEnclosingClass();
   public static final String RULE_NAME = "No-Unresolved-Comments";
   private static final LegacySubmitRequirement REQUIREMENT =
       LegacySubmitRequirement.builder()
@@ -57,7 +56,8 @@
     try {
       config = pluginConfigFactory.getFromProjectConfig(cd.project(), pluginName);
     } catch (NoSuchProjectException e) {
-      log.error("Error when fetching config of change {}'s project", cd.getId(), e);
+      logger.atSevere().withCause(e).log(
+          "Error when fetching config of change %s's project", cd.getId());
 
       return error("Error when fetching configuration");
     }
@@ -73,7 +73,8 @@
     try {
       unresolvedComments = cd.unresolvedCommentCount();
     } catch (StorageException e) {
-      log.error("Error when counting unresolved comments for change {}", cd.getId(), e);
+      logger.atSevere().withCause(e).log(
+          "Error when counting unresolved comments for change %s", cd.getId());
 
       return error("Error when counting unresolved comments");
     }