You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jclouds.apache.org by ga...@apache.org on 2013/09/05 05:10:26 UTC

git commit: Upgrade to Guava 15.0-rc1

Updated Branches:
  refs/heads/master 0d6ef0607 -> 310a898bc


Upgrade to Guava 15.0-rc1

Release notes:

https://code.google.com/p/guava-libraries/wiki/Release15


Project: http://git-wip-us.apache.org/repos/asf/incubator-jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-jclouds/commit/310a898b
Tree: http://git-wip-us.apache.org/repos/asf/incubator-jclouds/tree/310a898b
Diff: http://git-wip-us.apache.org/repos/asf/incubator-jclouds/diff/310a898b

Branch: refs/heads/master
Commit: 310a898bc3beb3fcff01f020bd3377b27ab98d56
Parents: 0d6ef06
Author: Andrew Gaul <ga...@apache.org>
Authored: Tue Aug 27 20:49:05 2013 -0700
Committer: Andrew Gaul <ga...@apache.org>
Committed: Wed Sep 4 19:56:00 2013 -0700

----------------------------------------------------------------------
 core/pom.xml                                    |  2 +-
 .../concurrent/config/WithSubmissionTrace.java  | 31 ++++++++++++++------
 ...ctorAndReflectiveTypeAdapterFactoryTest.java |  3 +-
 3 files changed, 25 insertions(+), 11 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-jclouds/blob/310a898b/core/pom.xml
----------------------------------------------------------------------
diff --git a/core/pom.xml b/core/pom.xml
index cea171c..8883e1f 100644
--- a/core/pom.xml
+++ b/core/pom.xml
@@ -90,7 +90,7 @@
     <dependency>
       <groupId>com.google.guava</groupId>
       <artifactId>guava</artifactId>
-      <version>14.0.1</version>
+      <version>15.0-rc1</version>
     </dependency>
     <dependency>
       <groupId>org.osgi</groupId>

http://git-wip-us.apache.org/repos/asf/incubator-jclouds/blob/310a898b/core/src/main/java/org/jclouds/concurrent/config/WithSubmissionTrace.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/jclouds/concurrent/config/WithSubmissionTrace.java b/core/src/main/java/org/jclouds/concurrent/config/WithSubmissionTrace.java
index 0facd97..ff2ebd6 100644
--- a/core/src/main/java/org/jclouds/concurrent/config/WithSubmissionTrace.java
+++ b/core/src/main/java/org/jclouds/concurrent/config/WithSubmissionTrace.java
@@ -26,6 +26,7 @@ import java.util.Set;
 import java.util.concurrent.Callable;
 import java.util.concurrent.Delayed;
 import java.util.concurrent.ExecutionException;
+import java.util.concurrent.Executor;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.TimeoutException;
 
@@ -113,7 +114,8 @@ public class WithSubmissionTrace {
 
    private static final Set<String> stackTracesToTrim = ImmutableSet.of(WithSubmissionTrace.class.getName(),
          ListeningExecutorService.class.getName(), ListenableFuture.class.getName(),
-         ListeningScheduledExecutorService.class.getName(), ScheduledFuture.class.getName());
+         ListeningScheduledExecutorService.class.getName(), ScheduledFuture.class.getName(),
+         ListenableScheduledFuture.class.getName());
 
    /** returns the stack trace at the caller */
    private static StackTraceElement[] getStackTraceHere() {
@@ -182,25 +184,25 @@ public class WithSubmissionTrace {
 
       @SuppressWarnings({ "rawtypes", "unchecked" })
       @Override
-      public ScheduledFuture<?> schedule(Runnable command, long delay, TimeUnit unit) {
-         return new ScheduledFuture(delegate().schedule(command, delay, unit));
+      public ListenableScheduledFuture<?> schedule(Runnable command, long delay, TimeUnit unit) {
+         return new ListenableScheduledFuture(delegate().schedule(command, delay, unit));
       }
 
       @Override
-      public <V> ScheduledFuture<V> schedule(Callable<V> callable, long delay, TimeUnit unit) {
-         return new ScheduledFuture<V>(delegate().schedule(callable, delay, unit));
+      public <V> ListenableScheduledFuture<V> schedule(Callable<V> callable, long delay, TimeUnit unit) {
+         return new ListenableScheduledFuture(delegate().schedule(callable, delay, unit));
       }
 
       @SuppressWarnings({ "rawtypes", "unchecked" })
       @Override
-      public ScheduledFuture<?> scheduleAtFixedRate(Runnable command, long initialDelay, long period, TimeUnit unit) {
-         return new ScheduledFuture(delegate().scheduleAtFixedRate(command, initialDelay, period, unit));
+      public ListenableScheduledFuture<?> scheduleAtFixedRate(Runnable command, long initialDelay, long period, TimeUnit unit) {
+         return new ListenableScheduledFuture(delegate().scheduleAtFixedRate(command, initialDelay, period, unit));
       }
 
       @SuppressWarnings({ "rawtypes", "unchecked" })
       @Override
-      public ScheduledFuture<?> scheduleWithFixedDelay(Runnable command, long initialDelay, long delay, TimeUnit unit) {
-         return new ScheduledFuture(delegate().scheduleWithFixedDelay(command, initialDelay, delay, unit));
+      public ListenableScheduledFuture<?> scheduleWithFixedDelay(Runnable command, long initialDelay, long delay, TimeUnit unit) {
+         return new ListenableScheduledFuture(delegate().scheduleWithFixedDelay(command, initialDelay, delay, unit));
       }
 
    }
@@ -250,4 +252,15 @@ public class WithSubmissionTrace {
       }
    }
 
+   private static class ListenableScheduledFuture<T> extends ScheduledFuture<T>
+         implements com.google.common.util.concurrent.ListenableScheduledFuture<T> {
+      private ListenableScheduledFuture(com.google.common.util.concurrent.ListenableScheduledFuture<T> delegate) {
+         super(delegate);
+      }
+
+      @Override
+      public void addListener(Runnable listener, Executor executor) {
+         ((com.google.common.util.concurrent.ListenableScheduledFuture<T>) delegate()).addListener(listener, executor);
+      }
+   }
 }

http://git-wip-us.apache.org/repos/asf/incubator-jclouds/blob/310a898b/core/src/test/java/org/jclouds/json/internal/DeserializationConstructorAndReflectiveTypeAdapterFactoryTest.java
----------------------------------------------------------------------
diff --git a/core/src/test/java/org/jclouds/json/internal/DeserializationConstructorAndReflectiveTypeAdapterFactoryTest.java b/core/src/test/java/org/jclouds/json/internal/DeserializationConstructorAndReflectiveTypeAdapterFactoryTest.java
index 0c57cef..8a3e62c 100644
--- a/core/src/test/java/org/jclouds/json/internal/DeserializationConstructorAndReflectiveTypeAdapterFactoryTest.java
+++ b/core/src/test/java/org/jclouds/json/internal/DeserializationConstructorAndReflectiveTypeAdapterFactoryTest.java
@@ -116,7 +116,8 @@ public final class DeserializationConstructorAndReflectiveTypeAdapterFactoryTest
       }
    }
 
-   @Test(expectedExceptions = IllegalArgumentException.class, expectedExceptionsMessageRegExp = "duplicate key: foo")
+   @Test(expectedExceptions = IllegalArgumentException.class,
+         expectedExceptionsMessageRegExp = "Multiple entries with same key: foo.*")
    public void testNoDuplicateSerializedNamesRequiredOnAllParameters() {
       parameterizedCtorFactory.create(gson, TypeToken.get(DuplicateSerializedNames.class));
    }