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

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2019/08/30 23:36:10 UTC

[GitHub] [pulsar] sijie opened a new pull request #5084: [admin] DeleteBookieRack should remove the rack info from zookeeper

sijie opened a new pull request #5084: [admin] DeleteBookieRack should remove the rack info from zookeeper 
URL: https://github.com/apache/pulsar/pull/5084
 
 
   *Motivation*
   
   Currently deleteBookieRackInfo doesn't actually remove rack info from zookeeper.
   So if a broker restarts, you still can get the *deleted* bookie rack info.
   
   *Modification*
   
   - Fix the deleteBbookieInfo logic to update rack configuration in zookeeper
   - Remove a group from the map if the group is empty
   

----------------------------------------------------------------
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