opendev/gerrit
Code Issues Proposed changes

ReceiveCommitsAdvertiseRefsHook: Limit fields during search

Browse Source 
Change-Id: I198c7b9cf53c037903e0d33beb269c3d8a63843a
This commit is contained in:
Dave Borowitz
2016-10-13 11:30:14 -04:00
parent 51341ec530
commit bad8aa355b
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
gerrit-server/src/main/java/com/google/gerrit/server/git/ReceiveCommitsAdvertiseRefsHook.java
View File

@@ -18,11 +18,13 @@ import static org.eclipse.jgit.lib.RefDatabase.ALL;
import com.google.auto.value.AutoValue; import com.google.auto.value.AutoValue;
import com.google.common.annotations.VisibleForTesting; import com.google.common.annotations.VisibleForTesting;
import com.google.common.collect.ImmutableSet;
import com.google.common.collect.Maps; import com.google.common.collect.Maps;
import com.google.common.collect.Sets; import com.google.common.collect.Sets;
import com.google.gerrit.reviewdb.client.PatchSet; import com.google.gerrit.reviewdb.client.PatchSet;
import com.google.gerrit.reviewdb.client.Project; import com.google.gerrit.reviewdb.client.Project;
import com.google.gerrit.reviewdb.client.RefNames; import com.google.gerrit.reviewdb.client.RefNames;
import com.google.gerrit.server.index.change.ChangeField;
import com.google.gerrit.server.query.change.ChangeData; import com.google.gerrit.server.query.change.ChangeData;
import com.google.gerrit.server.query.change.InternalChangeQuery; import com.google.gerrit.server.query.change.InternalChangeQuery;
import com.google.gerrit.server.util.MagicBranch; import com.google.gerrit.server.util.MagicBranch;
@@ -109,6 +111,7 @@ public class ReceiveCommitsAdvertiseRefsHook implements AdvertiseRefsHook {
try { try {
Set<ObjectId> r = Sets.newHashSetWithExpectedSize(limit); Set<ObjectId> r = Sets.newHashSetWithExpectedSize(limit);
for (ChangeData cd : queryProvider.get() for (ChangeData cd : queryProvider.get()
.setRequestedFields(ImmutableSet.of(ChangeField.PATCH_SET.getName()))
.enforceVisibility(true) .enforceVisibility(true)
.setLimit(limit) .setLimit(limit)
.byProjectOpen(projectName)) { .byProjectOpen(projectName)) {