RuiLi8080 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-522121458 The getBlacklistedSupervisors() call added here is intended to return a list of SupervisorSummaries. I notice that the SupervisorSummary objects are usually referred as supervisors in the thrift file. I guess this method naming could make more sense here? I did some renaming in the BlacklistScheduler to differentiate Host and Id. I hope this helps to address your comments.
---------------------------------------------------------------- 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
