> + this.initialWait = initialWait;
> + this.timeBetweenPolls = timeBetweenPolls;
> + this.client = checkNotNull(client, "client");
> + }
> +
> + @Inject
> + public BasePollingStrategy(GlacierClient client) {
> + this(DEFAULT_INITIAL_WAIT, DEFAULT_TIME_BETWEEN_POLLS, client);
> + }
> +
> + private boolean inProgress(String job, String vault) {
> + JobMetadata jobMetadata = client.describeJob(vault, job);
> + return (jobMetadata != null) && (jobMetadata.getStatusCode() ==
> JobStatus.IN_PROGRESS);
> + }
> +
> + private void waitForJob(String job, String vault) throws
> InterruptedException {
More of a style question, but is there any reason not to inline this method?
It's pretty short and seems to be called from one place only?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-aws/pull/44/files#r15726023
