Github user HeartSaVioR commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2433#discussion_r158737793
  
    --- Diff: storm-core/test/clj/org/apache/storm/nimbus_test.clj ---
    @@ -792,15 +795,18 @@
           (bind _ (log-message "ass1, t76, pre beat: " (pr-str ass1)))
           (bind _ (log-message "ass2, t76, pre beat: " (pr-str ass2)))
           (is (= ass1 (executor-assignment cluster storm-id executor-id1)))
    +      ;TODO: fix local test of worker reports
    --- End diff --
    
    I pulled the latest update of PR to local but they're still existing in 
nimbus_test.clj. Please check again.


---

Reply via email to