[
https://issues.apache.org/jira/browse/STORM-1232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15196194#comment-15196194
]
ASF GitHub Bot commented on STORM-1232:
---------------------------------------
Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/1108#discussion_r56235924
--- Diff: storm-core/test/clj/org/apache/storm/scheduler_test.clj ---
@@ -262,20 +262,15 @@
(deftest test-sort-slots
;; test supervisor2 has more free slots
- (is (= '(["supervisor2" 6700] ["supervisor1" 6700]
- ["supervisor2" 6701] ["supervisor1" 6701]
- ["supervisor2" 6702])
- (sort-slots [["supervisor1" 6700] ["supervisor1" 6701]
- ["supervisor2" 6700] ["supervisor2" 6701]
["supervisor2" 6702]
- ])))
+ (is (= "[supervisor2:6700, supervisor1:6700, supervisor2:6701,
supervisor1:6701, supervisor2:6702]"
+ (.toString (EvenScheduler/sortSlots [(WorkerSlot. "supervisor1"
6700) (WorkerSlot. "supervisor1" 6701)
--- End diff --
Can we remove the .toString and compare against a List of WorkerSlots? We
might need a clojurifyStructure in there too?
i.e
```
(is (= [(WorkerSlot. "supervisor2" 6700) (WorkerSlot. "supervisor1" 6700)
(WorkerSlot. "supervisor2" 6701) (WorkerSlot. "supervisor1" 6701)
(WorkerSlot. "supervisor2" 6702)]
(clojurify-structure (EvenScheduler/sortSlots [(WorkerSlot.
"supervisor1" 6700) (WorkerSlot. "supervisor1" 6701)...
```
> port backtype.storm.scheduler.DefaultScheduler to java
> -------------------------------------------------------
>
> Key: STORM-1232
> URL: https://issues.apache.org/jira/browse/STORM-1232
> Project: Apache Storm
> Issue Type: New Feature
> Components: storm-core
> Reporter: Robert Joseph Evans
> Assignee: Xin Wang
> Labels: java-migration, jstorm-merger
>
> port the DefaultScheduler to java
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
