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 2020/05/08 07:46:55 UTC

[GitHub] [pulsar] sijie opened a new pull request #6917: Make ZkBookieRackAffinityMapping work as expected

sijie opened a new pull request #6917:
URL: https://github.com/apache/pulsar/pull/6917


   ### Motivation
   
   The current bookie rack affinity logic is problematic. The rack map is expecting a `host:port` pair but the bookkeeper DNS resolver is expecting a hostname or a host address. 
   
   ### Modification
   
   - Introduce a new HashMap to keep a mapping between hostname and bookie info. It maintains the mapping for bookkeeper DNS resolver to lookup the network location.
   
   ### Verification
   
   I have verified it worked well in setting up rackware placement policy.
   


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

[GitHub] [pulsar] jiazhai commented on pull request #6917: Make ZkBookieRackAffinityMapping work as expected

Posted by GitBox <gi...@apache.org>.
jiazhai commented on pull request #6917:
URL: https://github.com/apache/pulsar/pull/6917#issuecomment-626086715


   /pulsarbot run-failure-checks
   
   


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

[GitHub] [pulsar] sijie merged pull request #6917: Make ZkBookieRackAffinityMapping work as expected

Posted by GitBox <gi...@apache.org>.
sijie merged pull request #6917:
URL: https://github.com/apache/pulsar/pull/6917


   


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

[GitHub] [pulsar] codelipenghui commented on pull request #6917: Make ZkBookieRackAffinityMapping work as expected

Posted by GitBox <gi...@apache.org>.
codelipenghui commented on pull request #6917:
URL: https://github.com/apache/pulsar/pull/6917#issuecomment-625825434


   /pulsarbot run-failure-checks


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

[GitHub] [pulsar] sijie commented on pull request #6917: Make ZkBookieRackAffinityMapping work as expected

Posted by GitBox <gi...@apache.org>.
sijie commented on pull request #6917:
URL: https://github.com/apache/pulsar/pull/6917#issuecomment-630457011


   @jiazhai: fix the comment. the BookiesRackConfiguration expects the key to be `hostname:port`. We used `bookieInfoMap` for resolving the rack information.  


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

[GitHub] [pulsar] jiazhai commented on pull request #6917: Make ZkBookieRackAffinityMapping work as expected

Posted by GitBox <gi...@apache.org>.
jiazhai commented on pull request #6917:
URL: https://github.com/apache/pulsar/pull/6917#issuecomment-626520934


   /pulsarbot run-failure-checks
   
   


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

[GitHub] [pulsar] sijie commented on pull request #6917: Make ZkBookieRackAffinityMapping work as expected

Posted by GitBox <gi...@apache.org>.
sijie commented on pull request #6917:
URL: https://github.com/apache/pulsar/pull/6917#issuecomment-625976394


   /pulsarbot run-failure-checks


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