summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHugo Arès <hugo.ares@ericsson.com>2017-04-28 09:42:49 +0200
committerHugo Arès <hugo.ares@ericsson.com>2017-04-28 09:42:49 +0200
commitdc18cb665eb452d71602e1a980d7669a67265dfc (patch)
tree1fe9dbc18cfc45f2fa1d5dd7290cb163a9b2a63f
parent47c2691a109ffedf8737f3697a80eef82dd5a4a9 (diff)
Fix replication retries when maxRetries is set to 0v2.14.1
maxRetries defaults to 0 which means retry indefinitely but it was not retrying. Change-Id: I6c8d8d6b1511d67b647f25193dcfe860687eea76
-rw-r--r--src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java b/src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java
index 473ebc5..525c990 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java
@@ -201,7 +201,7 @@ class PushOne implements ProjectRunnable, CanceledWhileRunning {
boolean setToRetry() {
retrying = true;
retryCount++;
- return retryCount <= maxRetries;
+ return maxRetries == 0 || retryCount <= maxRetries;
}
void canceledByReplication() {