> + }
> +
> + @Inject(optional = true)
> + @Named(Constants.PROPERTY_MAX_RETRIES)
> + private int retryCountLimit = 5;
> + @Resource
> + protected Logger logger = Logger.NULL;
> +
> + public boolean shouldRetryRequest(HttpCommand command, HttpResponse
> response) {
> + if (command.getFailureCount() > retryCountLimit) {
> + return false;
> + }
> + if (response.getStatusCode() == 500) {
> + byte[] content =
> HttpUtils.closeClientButKeepContentStream(response);
> + // Content can be null in the case of HEAD requests
> + if (content != null) {
Invert this to a `if (content == null) { return false; }` block to lose another
level of indenting?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/285/files#r9651204
