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 18:19:34 UTC

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

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


With regards,
Apache Git Services