jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Phillips <notificati...@github.com>
Subject Re: [jclouds-labs-aws] JCLOUDS-457: Add polling strategy (#44)
Date Sat, 02 Aug 2014 01:15:07 GMT
> +import org.jclouds.glacier.GlacierClient;
> +import org.jclouds.glacier.blobstore.strategy.PollingStrategy;
> +import org.jclouds.glacier.domain.JobMetadata;
> +import org.jclouds.glacier.domain.JobStatus;
> +
> +import com.google.inject.Inject;
> +
> +public class BasePollingStrategy implements PollingStrategy {
> +   public static final long DEFAULT_INITIAL_WAIT = TimeUnit.HOURS.toMillis(3);
> +   public static final long DEFAULT_TIME_BETWEEN_POLLS = TimeUnit.MINUTES.toMillis(15);
> +
> +   private final GlacierClient client;
> +   private final long initialWait;
> +   private final long timeBetweenPolls;
> +
> +   public BasePollingStrategy(long initialWait, long timeBetweenPolls, GlacierClient
client) {

[minor] Change the order to match the overloaded constructor and field declarations, i.e.
`(client, initialWait, timeBetweenPolls)`?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-aws/pull/44/files#r15725952
Mime
View raw message