[GitHub] [maven-wagon] michael-o commented on a change in pull request #57: WAGON-567: support retry on server side errors

[GitBox <gi...@apache.org>]

michael-o commented on a change in pull request #57: WAGON-567: support retry on server side errors
URL: https://github.com/apache/maven-wagon/pull/57#discussion_r335438381
 
 

 ##########
 File path: wagon-providers/wagon-http-shared/src/main/java/org/apache/maven/wagon/shared/http/AbstractHttpClientWagon.java
 ##########
 @@ -451,6 +453,54 @@ private static HttpRequestRetryHandler createRetryHandler()
         }
     }
 
+    /**
+     * The type of the serviceUnavailableRetryStrategy, defaults to {@code null}.
+     * Values can be {@link default DefaultServiceUnavailableRetryStrategy},
+     * or {@link standard SimpleHttpRequestRetryHandler},
+     * a fully qualified name class with a no-arg or null to not use a ServiceUnavailableRetryStrategy.
+     */
+    private static final String SERVICE_UNAVAILABLE_RETRY_STRATEGY_CLASS =
+            System.getProperty( "maven.wagon.http.serviceUnavailableRetryStrategy.class", "null" );
+
+    /**
+     * Interval in milliseconds between retries when using a serviceUnavailableRetryStrategy.
+     * <b>1000 by default</b>
+     */
+    private static final int SERVICE_UNAVAILABLE_RETRY_STRATEGY_RETRY_INTERVAL =
+        Integer.parseInt( System.getProperty( "maven.wagon.http.serviceUnavailableRetryStrategy.retryInterval", "1000" ) );
 
 Review comment:
   Use `Integer.getIntger()` for that.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services