merge down java retries change (#1058)

CarnegieLearningWeb · Nov 20, 2023 · 553d0cb · 553d0cb
1 parent 19d6f76
commit 553d0cb
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/clientlibs/java/pom.xml b/clientlibs/java/pom.xml
@@ -9,9 +9,9 @@
 		at the same time that happen to rev to the same new version will be caught 
 		by a merge conflict. -->
 
-	<!-- 4.1.12 -> 4.1.13: make 'status' a string in mark payload  -->
+	<!-- 4.1.13 -> 4.1.14: no retries for 4xx, retry other errors once instead of 3 times  -->
 
-	<version>4.1.13</version>
+	<version>4.1.14</version>
 	<build>
 		<plugins>
 			<plugin>

diff --git a/clientlibs/java/src/main/java/org/upgradeplatform/utils/PublishingRetryCallback.java b/clientlibs/java/src/main/java/org/upgradeplatform/utils/PublishingRetryCallback.java
@@ -7,6 +7,7 @@
 import javax.ws.rs.core.Response;
 
 import static javax.ws.rs.core.Response.Status.Family.SUCCESSFUL;
+import static javax.ws.rs.core.Response.Status.Family.CLIENT_ERROR;
 
 public class PublishingRetryCallback<T> implements InvocationCallback<Response> {
 
@@ -35,7 +36,10 @@ public PublishingRetryCallback(AsyncInvoker invoker, Entity<T> message, int retr
 
 	@Override
 	public void completed(Response response) {
-		if (SUCCESSFUL.equals(response.getStatusInfo().getFamily()) || retries <= 0) {
+    boolean isSuccess = SUCCESSFUL.equals(response.getStatusInfo().getFamily());
+    boolean isClientRequestError = CLIENT_ERROR.equals(response.getStatusInfo().getFamily());
+
+		if (isSuccess || isClientRequestError || retries <= 0) {
 			callback.completed(response);
 		} else {
 			retry();

diff --git a/clientlibs/java/src/main/java/org/upgradeplatform/utils/Utils.java b/clientlibs/java/src/main/java/org/upgradeplatform/utils/Utils.java
@@ -33,7 +33,7 @@ public class Utils
 
 	public static final String PATCH = "PATCH";
 
-	public static final int MAX_RETRIES = 3;
+	public static final int MAX_RETRIES = 1;
 
 
 	public static enum RequestType {