Github user HeartSaVioR commented on the issue:

    https://github.com/apache/storm/pull/2433
  
    @JessicaLHartog 
    Thanks for noticing! By the way, I was not aware of the discussion in 
storm-mesos, so don't know which works should be done in Storm side, and how 
these are coupled with this issue. 
    
    As far as I understand in your comment, only concern with this issue is 
specifying Supervisor's thrift port, which shouldn't be random in range but 
just using specified port. If I understand correctly, the patch already does 
that (via configuration), and storm-mesos could launch Supervisor instance with 
overriding supervisor thrift port. Makes sense?
    
    cc. @danny0405 FYI.


---

Reply via email to