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 2021/06/18 21:19:00 UTC

[GitHub] [pulsar] abhilashmandaliya commented on pull request #10961: Fix incorrect port of advertisedListener

abhilashmandaliya commented on pull request #10961:
URL: https://github.com/apache/pulsar/pull/10961#issuecomment-863782575


   I think we should have exposed advertised ports like an advertisedAddress. I am not sure about the impact of this fix but in general, it still looks odd. We may get advertisedAddress and normal broker port. We could have avoided advertisedListner if we had exposed advertisedBrokerPort and adverisedWebPort? If I am not wrong `advertisedListener` 's port is only for broker, not for web.


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