Merge pull request #847 from andrewgaul/delete-all-keys-in-list-futures-get-unchecked

Avoid Futures.getUnchecked in DeleteAllKeysInList
This commit is contained in:
Adrian Cole 2012-09-19 18:38:13 -07:00
commit 3deb94e0dd
1 changed files with 6 additions and 2 deletions

View File

@ -22,6 +22,7 @@ import static org.jclouds.blobstore.options.ListContainerOptions.Builder.recursi
import static org.jclouds.concurrent.FutureIterables.awaitCompletion; import static org.jclouds.concurrent.FutureIterables.awaitCompletion;
import java.util.Map; import java.util.Map;
import java.util.concurrent.ExecutionException;
import java.util.concurrent.ExecutorService; import java.util.concurrent.ExecutorService;
import java.util.concurrent.Future; import java.util.concurrent.Future;
@ -95,8 +96,11 @@ public class DeleteAllKeysInList implements ClearListStrategy, ClearContainerStr
for (int i = 0; i < maxErrors; ) { for (int i = 0; i < maxErrors; ) {
// fetch partial directory listing // fetch partial directory listing
try { try {
listing = Futures.getUnchecked(connection.list(containerName, options)); listing = connection.list(containerName, options).get();
} catch (RuntimeException ee) { } catch (InterruptedException ie) {
Thread.currentThread().interrupt();
break;
} catch (ExecutionException ee) {
++i; ++i;
if (i == maxErrors) { if (i == maxErrors) {
throw Throwables.propagate(ee.getCause()); throw Throwables.propagate(ee.getCause());