You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jclouds.apache.org by ad...@apache.org on 2014/10/30 02:39:15 UTC
git commit: JCLOUDS-534 Avoid runtime incompatibility introduced by
Guava's closeQuietly.
Repository: jclouds-chef
Updated Branches:
refs/heads/master 8154e8d22 -> 8d4579d97
JCLOUDS-534 Avoid runtime incompatibility introduced by Guava's closeQuietly.
Project: http://git-wip-us.apache.org/repos/asf/jclouds-chef/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds-chef/commit/8d4579d9
Tree: http://git-wip-us.apache.org/repos/asf/jclouds-chef/tree/8d4579d9
Diff: http://git-wip-us.apache.org/repos/asf/jclouds-chef/diff/8d4579d9
Branch: refs/heads/master
Commit: 8d4579d97381fc18b4b1ecb1f9d5811aa319aad1
Parents: 8154e8d
Author: Adrian Cole <ac...@twitter.com>
Authored: Wed Oct 29 17:14:42 2014 -0700
Committer: Adrian Cole <ac...@twitter.com>
Committed: Wed Oct 29 18:38:59 2014 -0700
----------------------------------------------------------------------
.../org/jclouds/chef/handlers/ChefErrorHandler.java | 14 ++------------
.../jclouds/chef/internal/BaseChefApiLiveTest.java | 11 ++---------
2 files changed, 4 insertions(+), 21 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/jclouds-chef/blob/8d4579d9/core/src/main/java/org/jclouds/chef/handlers/ChefErrorHandler.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/jclouds/chef/handlers/ChefErrorHandler.java b/core/src/main/java/org/jclouds/chef/handlers/ChefErrorHandler.java
index 3376219..85f9129 100644
--- a/core/src/main/java/org/jclouds/chef/handlers/ChefErrorHandler.java
+++ b/core/src/main/java/org/jclouds/chef/handlers/ChefErrorHandler.java
@@ -16,9 +16,7 @@
*/
package org.jclouds.chef.handlers;
-import static com.google.common.base.Throwables.propagate;
-
-import java.io.IOException;
+import static org.jclouds.util.Closeables2.closeQuietly;
import javax.annotation.Resource;
import javax.inject.Inject;
@@ -33,8 +31,6 @@ import org.jclouds.logging.Logger;
import org.jclouds.rest.AuthorizationException;
import org.jclouds.rest.ResourceNotFoundException;
-import com.google.common.io.Closeables;
-
/**
* This will parse and set an appropriate exception on the command object.
*/
@@ -67,13 +63,7 @@ public class ChefErrorHandler implements HttpErrorHandler {
break;
}
} finally {
- if (response.getPayload() != null) {
- try {
- Closeables.close(response.getPayload().getInput(), true);
- } catch (IOException e) {
- throw propagate(e);
- }
- }
+ closeQuietly(response.getPayload());
command.setException(exception);
}
}
http://git-wip-us.apache.org/repos/asf/jclouds-chef/blob/8d4579d9/core/src/test/java/org/jclouds/chef/internal/BaseChefApiLiveTest.java
----------------------------------------------------------------------
diff --git a/core/src/test/java/org/jclouds/chef/internal/BaseChefApiLiveTest.java b/core/src/test/java/org/jclouds/chef/internal/BaseChefApiLiveTest.java
index 225e593..a477be0 100644
--- a/core/src/test/java/org/jclouds/chef/internal/BaseChefApiLiveTest.java
+++ b/core/src/test/java/org/jclouds/chef/internal/BaseChefApiLiveTest.java
@@ -16,11 +16,11 @@
*/
package org.jclouds.chef.internal;
-import static com.google.common.base.Throwables.propagate;
import static com.google.common.collect.Iterables.any;
import static com.google.common.collect.Iterables.isEmpty;
import static com.google.common.hash.Hashing.md5;
import static java.util.concurrent.TimeUnit.MILLISECONDS;
+import static org.jclouds.util.Closeables2.closeQuietly;
import static org.jclouds.util.Predicates2.retry;
import static org.testng.Assert.assertEquals;
import static org.testng.Assert.assertFalse;
@@ -30,7 +30,6 @@ import static org.testng.Assert.assertTrue;
import static org.testng.Assert.fail;
import java.io.File;
-import java.io.IOException;
import java.io.InputStream;
import java.util.Collections;
import java.util.List;
@@ -65,7 +64,6 @@ import com.google.common.base.Predicate;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableSet;
import com.google.common.hash.Hashing;
-import com.google.common.io.Closeables;
import com.google.common.io.Files;
import com.google.common.primitives.Bytes;
@@ -531,12 +529,7 @@ public abstract class BaseChefApiLiveTest<A extends ChefApi> extends BaseChefLiv
Client client = clientApi.getClient(identity);
assertNotNull(client, "Client not found: " + identity);
} finally {
- try {
- Closeables.close(clientApi, true);
- } catch (IOException e) {
- throw propagate(e);
- }
+ closeQuietly(clientApi);
}
}
-
}