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_r335438573
##########
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" );
Review comment:
Why not use a real `null`? Or better `none`.
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
