commit 2574a736a384c901298af89f3fbc7420280125be
author Christian Aistleitner <christian@quelltextlich.at> Sat May 09 09:17:46 2020 +0200
committer Christian Aistleitner <christian@quelltextlich.at> Mon Jun 29 08:29:10 2020 +0200
tree 58ce12b4381b19bb58f5a9ad4f013a9252f56d40
parent f5063d0bdae244eb7ed1354c6ca9f198f4ed7b57

Switch remaining parts to Flogger

The switch to Flogger on stable-3.1 covered most ground. We now switch
also the remaining parts to Flogger.

Change-Id: I7fb3cdf32b89c5d849c88e822cc0da425152ee50

src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/FireEventOnCommitsParametersExtractor.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/FireEventOnCommitsParametersExtractor.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/FireEventOnCommitsParametersExtractor.java
index d7ae586..feb29b9 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/FireEventOnCommitsParametersExtractor.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/FireEventOnCommitsParametersExtractor.java
@@ -15,19 +15,16 @@
 package com.googlesource.gerrit.plugins.its.base.workflow;
 
 import com.google.common.base.Strings;
+import com.google.common.flogger.FluentLogger;
 import com.googlesource.gerrit.plugins.its.base.workflow.commit_collector.CommitCollector;
 import com.googlesource.gerrit.plugins.its.base.workflow.commit_collector.SinceLastTagCommitCollector;
 import java.util.Arrays;
 import java.util.Map;
 import java.util.Optional;
 import javax.inject.Inject;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class FireEventOnCommitsParametersExtractor {
-
-  private static final Logger log =
-      LoggerFactory.getLogger(FireEventOnCommitsParametersExtractor.class);
+  private static final FluentLogger logger = FluentLogger.forEnclosingClass();
 
   private final SinceLastTagCommitCollector.Factory sinceLastTagCommitCollectorFactory;
 
@@ -54,21 +51,21 @@
       ActionRequest actionRequest, Map<String, String> properties) {
     String[] parameters = actionRequest.getParameters();
     if (parameters.length != 1) {
-      log.error(
-          "Wrong number of received parameters. Received parameters are {}. Only one parameter is expected, the collector name.",
+      logger.atSevere().log(
+          "Wrong number of received parameters. Received parameters are %s. Only one parameter is expected, the collector name.",
           Arrays.toString(parameters));
       return Optional.empty();
     }
 
     String collectorName = parameters[0];
     if (Strings.isNullOrEmpty(collectorName)) {
-      log.error("Received collector name is blank");
+      logger.atSevere().log("Received collector name is blank");
       return Optional.empty();
     }
 
     CommitCollector commitCollector = getCommitCollector(collectorName);
     if (commitCollector == null) {
-      log.error("No commit collector found for name {}", collectorName);
+      logger.atSevere().log("No commit collector found for name %s", collectorName);
       return Optional.empty();
     }