> +
> + @Inject
> + public AtmosServerErrorRetryHandler(BackoffLimitedRetryHandler
> backoffHandler,
> + AtmosUtils utils) {
> + this.backoffHandler = backoffHandler;
> + this.utils = utils;
> + }
> +
> + @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) {
Check for `command.getFailureCount() > retryCountLimit ||
response.getStatusCode() != 500` here? Then we can skip the outermost `if`
below and lose one level of indenting?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/285/files#r9651174
