Github user danny0405 commented on the issue:
https://github.com/apache/storm/pull/2433
@HeartSaVioR @revans2
Based on your comments, i thought there are mainly 3 TODOs for this patch:
1. Supervisor container-blity promotion, like support multiple
supervisor instances on one machine.
2. Assignments security should be guaranteed.
3. Backwards compatibility for old version storm workers.
For TODO 1 I can make supervisor thrift ports picked in a range for a
machine node, and nimbus aware the port-info from SupervisorHeartbeats. Also i
will passed the port as an start up argument so that workers will know its
parent supervisor port.
For TODO 2 i understood i should wait for @revans2's token authentication
right?
For TODO 3 i don't know how much worker would be taken, if @HeartSaVioR can
help to contribute it will very appreciate it.
Also the supervisor local disk worker heartbeats can also be removed
actually, and it's easy to achieve for the patch, should i also do this?
---
