get rid of unassignbyexecutor method

Nathan Marz · Nathan Marz · commit 7929a1331ba2 · 2012-06-04T13:09:59.000-07:00
diff --git a/src/jvm/backtype/storm/scheduler/SchedulerAssignment.java b/src/jvm/backtype/storm/scheduler/SchedulerAssignment.java
@@ -35,19 +35,6 @@ public void assign(WorkerSlot slot, Collection<ExecutorDetails> executors) {
             this.executorToSlot.put(executor, slot);
         }
     }
- 
-    /**
-     * Unassign the assignment for executor.
-     */
-    public void unassignByExecutor(ExecutorDetails executor) {
-        this.executorToSlot.remove(executor);
-    }
-    
-    public void unassignByExecutors(Collection<ExecutorDetails> executors) {
-        for (ExecutorDetails executor : executors) {
-            this.unassignByExecutor(executor);
-        }
-    }
     
     /**
      * Release the slot occupied by this assignment.
diff --git a/test/clj/backtype/storm/scheduler_test.clj b/test/clj/backtype/storm/scheduler_test.clj
@@ -47,10 +47,6 @@
     (is (= {[6 10] ["supervisor2" 2]}
            (clojurify-executor->slot (.getExecutorToSlot assignment))))
 
-    ;; test unassignByExecutor
-    (.unassignByExecutor assignment (ExecutorDetails. (int 6) (int 10)))
-    (is (= true
-           (empty? (.getExecutorToSlot assignment))))
     ))
 
 (deftest test-topologies
