You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@storm.apache.org by GitBox <gi...@apache.org> on 2019/08/16 19:16:22 UTC

[GitHub] [storm] RuiLi8080 commented on issue #3104: [STORM-3484] Add Cluster Blacklist to UI to show what supervisors are…

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:
users@infra.apache.org


With regards,
Apache Git Services