diff options
author | Kaushik Lingarkar <kaushik.lingarkar@linaro.org> | 2022-05-19 17:12:47 -0700 |
---|---|---|
committer | Kaushik Lingarkar <kaushik.lingarkar@linaro.org> | 2022-05-24 20:56:33 +0000 |
commit | 3d9c2666dec61135acfe7d93a2a6c62b9f97605a (patch) | |
tree | d06c0c200cb44904a097037af447064239a14198 | |
parent | 5adecafa69feae2d61710d855180019d9933a739 (diff) |
Avoid loading change notes to obtain hashtags in ssh queries
Hash tags are already available in ChangeData, use it, instead of
unnecessarily loading change notes to obtain them.
Bug: Issue 15942
Release-Notes: skip
Change-Id: Iad00c43adfe303c8e10777ce4744cb0709d9be30
-rw-r--r-- | java/com/google/gerrit/server/query/change/OutputStreamQuery.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/java/com/google/gerrit/server/query/change/OutputStreamQuery.java b/java/com/google/gerrit/server/query/change/OutputStreamQuery.java index b931457223..7b71b76671 100644 --- a/java/com/google/gerrit/server/query/change/OutputStreamQuery.java +++ b/java/com/google/gerrit/server/query/change/OutputStreamQuery.java @@ -18,6 +18,7 @@ import static com.google.common.base.Preconditions.checkState; import static java.nio.charset.StandardCharsets.UTF_8; import com.google.common.collect.ImmutableListMultimap; +import com.google.common.collect.Lists; import com.google.common.flogger.FluentLogger; import com.google.gerrit.entities.Change; import com.google.gerrit.entities.LabelTypes; @@ -250,7 +251,8 @@ public class OutputStreamQuery { ChangeData d, Map<Project.NameKey, Repository> repos, Map<Project.NameKey, RevWalk> revWalks) throws IOException { LabelTypes labelTypes = d.getLabelTypes(); - ChangeAttribute c = eventFactory.asChangeAttribute(d.change(), d.notes()); + ChangeAttribute c = eventFactory.asChangeAttribute(d.change()); + c.hashtags = Lists.newArrayList(d.hashtags()); eventFactory.extend(c, d.change()); if (!trackingFooters.isEmpty()) { |