commit 4f1cda223d5c960e1ae3b6fa93c4b78633fd69a6
author Han-Wen Nienhuys <hanwen@google.com> Tue Jul 13 14:02:29 2021 +0200
committer Han-Wen Nienhuys <hanwen@google.com> Tue Jul 13 15:10:15 2021 +0200
tree 95e1c8ce45ef40a546921f74a3cc76507ee4be42
parent 3e4a797470b9e39555f3b16b8040c1eaa757c253

ReceiveCommits: combine multiple log statements

Change-Id: Iff90578720bd48cdabaf3cfff0dca0c3cfe50c5e

java/com/google/gerrit/server/git/receive/ReceiveCommits.java

diff --git a/java/com/google/gerrit/server/git/receive/ReceiveCommits.java b/java/com/google/gerrit/server/git/receive/ReceiveCommits.java
index 005aebb..454df66 100644
--- a/java/com/google/gerrit/server/git/receive/ReceiveCommits.java
+++ b/java/com/google/gerrit/server/git/receive/ReceiveCommits.java
@@ -659,7 +659,9 @@
   // Process as many commands as possible, but may leave some commands in state NOT_ATTEMPTED.
   private void processCommandsUnsafe(
       Collection<ReceiveCommand> commands, MultiProgressMonitor progress) {
-    logger.atFine().log("Calling user: %s", user.getLoggableName());
+    logger.atFine().log("Calling user: %s, commands: %d", user.getLoggableName(), commands.size());
+
+    // If the list of groups is large, the log entry may get dropped, so separate out.
     logger.atFine().log("Groups: %s", lazy(() -> user.getEffectiveGroups().getKnownGroups()));
 
     if (!projectState.getProject().getState().permitsWrite()) {
@@ -669,8 +671,6 @@
       return;
     }
 
-    logger.atFine().log("Parsing %d commands", commands.size());
-
     List<ReceiveCommand> magicCommands = new ArrayList<>();
     List<ReceiveCommand> regularCommands = new ArrayList<>();