agresch commented on issue #3104: [STORM-3484] Add Cluster Blacklist to UI to show what supervisors are… URL: https://github.com/apache/storm/pull/3104#issuecomment-522104443 We still have a change to push back that allows multiple supervisor ids on a single host for use on NUMA. While doing this, we ran into some bugs where host/node ids were not necessarily synonymous. For clarity, could you update the naming from of methods/variables something like: getBlacklistedSupervisors() to getBlacklistedSupervisorsHosts()? This will help prevent issues going forward. We still have an open task to push this NUMA code back.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
