commit 694b34084ca623756a9ec7e6a60631ab6027d5c2
author Nasser Grainawi <nasser.grainawi@linaro.org> Sat Aug 19 19:25:21 2023 -0600
committer Nasser Grainawi <nasser.grainawi@linaro.org> Wed Aug 30 09:54:14 2023 -0700
tree 52ca13cf360b9d8f3f4a7ccd11426abe0e8e18f3
parent f447543ba5493bae04cc9a5a2844f2745541e27a

TasksStorage: Remove synchronized from methods

These aren't expected to be necessary for correctness.

Change-Id: Id23d3b385e289ba453638b76d6523016a5aabf2a

src/main/java/com/googlesource/gerrit/plugins/replication/ReplicationTasksStorage.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/replication/ReplicationTasksStorage.java b/src/main/java/com/googlesource/gerrit/plugins/replication/ReplicationTasksStorage.java
index 0d5604c..c847f55 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/replication/ReplicationTasksStorage.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/replication/ReplicationTasksStorage.java
@@ -137,11 +137,11 @@
     return isMultiPrimary;
   }
 
-  public synchronized String create(ReplicateRefUpdate r) {
+  public String create(ReplicateRefUpdate r) {
     return new Task(r).create();
   }
 
-  public synchronized Set<String> start(UriUpdates uriUpdates) {
+  public Set<String> start(UriUpdates uriUpdates) {
     Set<String> startedRefs = new HashSet<>();
     for (ReplicateRefUpdate update : uriUpdates.getReplicateRefUpdates()) {
       Task t = new Task(update);
@@ -152,13 +152,13 @@
     return startedRefs;
   }
 
-  public synchronized void reset(UriUpdates uriUpdates) {
+  public void reset(UriUpdates uriUpdates) {
     for (ReplicateRefUpdate update : uriUpdates.getReplicateRefUpdates()) {
       new Task(update).reset();
     }
   }
 
-  public synchronized void recoverAll() {
+  public void recoverAll() {
     streamRunning().forEach(r -> new Task(r).recover());
   }