| commit | 220ce6829d4b8382374b08045dc4a1459db8f001 | [log] [tgz] |
|---|---|---|
| author | David Ostrovsky <david.ostrovsky@gmail.com> | Sat Feb 12 16:16:12 2022 +0000 |
| committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | Sat Feb 12 16:16:12 2022 +0000 |
| tree | c239f5c1470874f25d18324e218225096e0202b9 | |
| parent | ba3a9eb92811feef9c2f47287613badee987a9d2 [diff] | |
| parent | 198984142cb3808d8c9d63c1ce48045627c1ae48 [diff] |
Merge "Format with GJF 1.7"
diff --git a/src/main/java/com/googlesource/gerrit/plugins/replication/Destination.java b/src/main/java/com/googlesource/gerrit/plugins/replication/Destination.java index d1302a5..d2f7213 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/replication/Destination.java +++ b/src/main/java/com/googlesource/gerrit/plugins/replication/Destination.java
@@ -275,6 +275,8 @@ } private void foreachPushOp(Map<URIish, PushOne> opsMap, Function<PushOne, Void> pushOneFunction) { + // Callers may modify the provided opsMap concurrently, hence make a defensive copy of the + // values to loop over them. for (PushOne pushOne : ImmutableList.copyOf(opsMap.values())) { pushOneFunction.apply(pushOne); }