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

    https://github.com/apache/storm/pull/355#discussion_r22236169
  
    --- Diff: storm-core/src/clj/backtype/storm/daemon/nimbus.clj ---
    @@ -1248,7 +1256,7 @@
                                                    (when-let [owner (:owner 
base)] (.set_owner topo-summ owner))
                                                    (when-let [sched-status 
(.get @(:id->sched-status nimbus) id)] (.set_sched_status topo-summ 
sched-status))
                                                    topo-summ
    -                                          ))]
    +                                          )))]
    --- End diff --
    
    The new ')' is in the wrong place.  It should be up on line 1255 instead.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to