commit 4d00ae7b5e424e3694c2b35dab0b2d6127deb10e
author Patrick Hiesel <hiesel@google.com> Tue Oct 19 07:56:19 2021 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Tue Oct 19 07:56:19 2021 +0000
tree 5cce304ec53a89848d61b27a3c36bfd1c62fc3b8
parent 495a56aa39bfafc4b754e7e2ee4511f96e6dd2aa
parent d67950548b49b271c9954920bdd58ced72052341

Merge "Trace execution time for loading all group names"

java/com/google/gerrit/server/group/db/GroupNameNotes.java

diff --git a/java/com/google/gerrit/server/group/db/GroupNameNotes.java b/java/com/google/gerrit/server/group/db/GroupNameNotes.java
index cdba81f..7c4fb16f9 100644
--- a/java/com/google/gerrit/server/group/db/GroupNameNotes.java
+++ b/java/com/google/gerrit/server/group/db/GroupNameNotes.java
@@ -35,6 +35,9 @@
 import com.google.gerrit.exceptions.DuplicateKeyException;
 import com.google.gerrit.git.ObjectIds;
 import com.google.gerrit.server.git.meta.VersionedMetaData;
+import com.google.gerrit.server.logging.Metadata;
+import com.google.gerrit.server.logging.TraceContext;
+import com.google.gerrit.server.logging.TraceContext.TraceTimer;
 import java.io.IOException;
 import java.util.Collection;
 import java.util.Map;
@@ -210,7 +213,11 @@
     if (ref == null) {
       return ImmutableList.of();
     }
-    try (RevWalk revWalk = new RevWalk(repository);
+    try (TraceTimer ignored =
+            TraceContext.newTimer(
+                "Loading all groups",
+                Metadata.builder().noteDbRefName(RefNames.REFS_GROUPNAMES).build());
+        RevWalk revWalk = new RevWalk(repository);
         ObjectReader reader = revWalk.getObjectReader()) {
       RevCommit notesCommit = revWalk.parseCommit(ref.getObjectId());
       NoteMap noteMap = NoteMap.read(reader, notesCommit);