commit 4f070067d1af1775e6f664bc9753d711df4db7ec
author Edwin Kempin <ekempin@google.com> Thu Feb 23 08:00:21 2023 +0100
committer Edwin Kempin <ekempin@google.com> Thu Feb 23 08:00:21 2023 +0100
tree f2efe7cca25b73f353341340f93ba41b6ca491ef
parent 97b9f31595cd6bd1dead714404f79fd203a05e01

BatchUpdate: Fix timer logging for updateRepo step

We intended to include the name of the BatchUpdateOp class into the
timer name, but missed that 'op' was a
Map.Entry<Change.Id, BatchUpdateOp> and by using
'op.getClass().getSimpleName()' we inlcuded the name of the Map.Entry
class instead.

Release-Notes: skip
Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: I5137aa8594e8732534cb7bb19845c3f7e863c0ae

java/com/google/gerrit/server/update/BatchUpdate.java

diff --git a/java/com/google/gerrit/server/update/BatchUpdate.java b/java/com/google/gerrit/server/update/BatchUpdate.java
index 412a8ee..2966fd7 100644
--- a/java/com/google/gerrit/server/update/BatchUpdate.java
+++ b/java/com/google/gerrit/server/update/BatchUpdate.java
@@ -579,7 +579,7 @@
       for (Map.Entry<Change.Id, BatchUpdateOp> op : ops.entries()) {
         try (TraceContext.TraceTimer ignored =
             TraceContext.newTimer(
-                op.getClass().getSimpleName() + "#updateRepo",
+                op.getValue().getClass().getSimpleName() + "#updateRepo",
                 Metadata.builder()
                     .projectName(project.get())
                     .changeId(op.getKey().get())