opendev/gerrit
Code Issues Proposed changes

HackPushNegotiateHook: Avoid RefDatabase#getRefs

Browse Source 
Change-Id: I7682d55fa72674f3ebdf1e5758f6100f0016d4a3
This commit is contained in:
Dave Borowitz
2019-01-04 16:23:50 -08:00
parent adc087bf8e
commit 1acab02049
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
java/com/google/gerrit/server/git/receive/HackPushNegotiateHook.java
View File

@@ -14,7 +14,7 @@
package com.google.gerrit.server.git.receive; package com.google.gerrit.server.git.receive;
import static org.eclipse.jgit.lib.RefDatabase.ALL; import static java.util.stream.Collectors.toMap;
import com.google.common.collect.Sets; import com.google.common.collect.Sets;
import com.google.common.flogger.FluentLogger; import com.google.common.flogger.FluentLogger;
@@ -78,7 +78,12 @@ public class HackPushNegotiateHook implements AdvertiseRefsHook {
Map<String, Ref> r = rp.getAdvertisedRefs(); Map<String, Ref> r = rp.getAdvertisedRefs();
if (r == null) { if (r == null) {
try { try {
r = rp.getRepository().getRefDatabase().getRefs(ALL); r =
rp.getRepository()
.getRefDatabase()
.getRefs()
.stream()
.collect(toMap(Ref::getName, x -> x));
} catch (ServiceMayNotContinueException e) { } catch (ServiceMayNotContinueException e) {
throw e; throw e;
} catch (IOException e) { } catch (IOException e) {