You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jclouds.apache.org by an...@apache.org on 2014/06/25 23:51:34 UTC

git commit: Aligning overrides in ComputeServiceEventProxy with ComputeService

Repository: jclouds-karaf
Updated Branches:
  refs/heads/fix-karaf [created] d94b19c21


Aligning overrides in ComputeServiceEventProxy with ComputeService

See https://git-wip-us.apache.org/repos/asf?p=jclouds.git;h=8598ee8


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

Branch: refs/heads/fix-karaf
Commit: d94b19c21ad7d91358b8aabc240fcceb3a3a3a2d
Parents: 20bb5f0
Author: Andrew Phillips <an...@apache.org>
Authored: Wed Jun 25 23:50:20 2014 +0200
Committer: Andrew Phillips <an...@apache.org>
Committed: Wed Jun 25 23:50:20 2014 +0200

----------------------------------------------------------------------
 .../karaf/core/ComputeServiceEventProxy.java      | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jclouds-karaf/blob/d94b19c2/core/src/main/java/org/jclouds/karaf/core/ComputeServiceEventProxy.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/jclouds/karaf/core/ComputeServiceEventProxy.java b/core/src/main/java/org/jclouds/karaf/core/ComputeServiceEventProxy.java
index e31af7e..1330f74 100644
--- a/core/src/main/java/org/jclouds/karaf/core/ComputeServiceEventProxy.java
+++ b/core/src/main/java/org/jclouds/karaf/core/ComputeServiceEventProxy.java
@@ -268,13 +268,15 @@ public class ComputeServiceEventProxy implements ComputeService {
      * <p/>
      * affected nodes may not resume with the same IP address(es)
      *
+     * @return list of nodes resumed
+     *
      * @throws UnsupportedOperationException if the underlying provider doesn't support suspend/resume
      * @throws java.util.NoSuchElementException
      *                                       if no nodes matched the predicate specified
      */
     @Override
-    public void resumeNodesMatching(Predicate<NodeMetadata> filter) {
-        computeService.resumeNodesMatching(filter);
+    public Set<? extends NodeMetadata> resumeNodesMatching(Predicate<NodeMetadata> filter) {
+        return computeService.resumeNodesMatching(filter);
     }
 
     /**
@@ -300,13 +302,15 @@ public class ComputeServiceEventProxy implements ComputeService {
      * <p/>
      * affected nodes may not resume with the same IP address(es)
      *
+     * @return list of nodes suspended
+     *
      * @throws UnsupportedOperationException if the underlying provider doesn't support suspend/resume
      * @throws java.util.NoSuchElementException
      *                                       if no nodes matched the predicate specified
      */
     @Override
-    public void suspendNodesMatching(Predicate<NodeMetadata> filter) {
-        computeService.suspendNodesMatching(filter);
+    public Set<? extends NodeMetadata> suspendNodesMatching(Predicate<NodeMetadata> filter) {
+        return computeService.suspendNodesMatching(filter);
     }
 
     /**
@@ -366,12 +370,14 @@ public class ComputeServiceEventProxy implements ComputeService {
      * nodes matching the filter are treated as a logical set. Using this command, you can save time
      * by rebooting the nodes in parallel.
      *
+     * @return list of nodes rebooted
+     *
      * @throws java.util.NoSuchElementException
      *          if no nodes matched the predicate specified
      */
     @Override
-    public void rebootNodesMatching(Predicate<NodeMetadata> filter) {
-        computeService.rebootNodesMatching(filter);
+    public Set<? extends NodeMetadata> rebootNodesMatching(Predicate<NodeMetadata> filter) {
+        return computeService.rebootNodesMatching(filter);
     }
 
     /**