commit a9eaeb1a9891d1d2effe5039bb82ed6338d9ebdc
author Edwin Kempin <ekempin@google.com> Tue Dec 04 11:14:02 2018 +0100
committer Edwin Kempin <ekempin@google.com> Fri Dec 07 12:53:07 2018 +0100
tree 9a1d51656cba21004ebd082f13dc42c5526e1823
parent 48c55620eff2338112861c33e2266c2a0800e997

ReceiveCommits: Log results of ReceiveCommands if tracing is enabled

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

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 ba11985..65b4f73 100644
--- a/java/com/google/gerrit/server/git/receive/ReceiveCommits.java
+++ b/java/com/google/gerrit/server/git/receive/ReceiveCommits.java
@@ -18,6 +18,7 @@
 import static com.google.common.base.Preconditions.checkArgument;
 import static com.google.common.base.Preconditions.checkState;
 import static com.google.common.collect.ImmutableSet.toImmutableSet;
+import static com.google.common.flogger.LazyArgs.lazy;
 import static com.google.gerrit.common.FooterConstants.CHANGE_ID;
 import static com.google.gerrit.reviewdb.client.RefNames.REFS_CHANGES;
 import static com.google.gerrit.reviewdb.client.RefNames.isConfigRef;
@@ -611,6 +612,10 @@
       replaceProgress.end();
       queueSuccessMessages(newChanges);
       refsPublishDeprecationWarning();
+
+      logger.atFine().log(
+          "Command results: %s",
+          lazy(() -> commands.stream().map(ReceiveCommits::commandToString).collect(joining(","))));
     }
   }
 
@@ -3277,4 +3282,15 @@
   private static boolean isConfig(ReceiveCommand cmd) {
     return cmd.getRefName().equals(RefNames.REFS_CONFIG);
   }
+
+  private static String commandToString(ReceiveCommand cmd) {
+    StringBuilder b = new StringBuilder();
+    b.append(cmd);
+    b.append("  (").append(cmd.getResult());
+    if (cmd.getMessage() != null) {
+      b.append(": ").append(cmd.getMessage());
+    }
+    b.append(")\n");
+    return b.toString();
+  }
 }