tomscut commented on a change in pull request #12389:
URL: https://github.com/apache/pulsar/pull/12389#discussion_r730189595
##########
File path:
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/validator/MultipleListenerValidator.java
##########
@@ -39,10 +39,10 @@
/**
* Validate the configuration of `advertisedListeners`,
`internalListenerName`.
- * 2. the listener name in `advertisedListeners` must not duplicate.
- * 3. user can not assign same 'host:port' to different listener.
- * 4. if `internalListenerName` is absent, the first `listener` in the
`advertisedListeners` will be the `internalListenerName`.
- * 5. if pulsar do not specify `brokerServicePortTls`, should only contain
one entry of `pulsar://` per listener name.
+ * 1. the listener name in `advertisedListeners` must not duplicate.
+ * 2. user can not assign same 'host:port' to different listener.
+ * 3. if `internalListenerName` is absent, the first `listener` in the
`advertisedListeners` will be the `internalListenerName`.
+ * 4. if pulsar do not specify `brokerServicePortTls`, should only contain
one entry of `pulsar://` per listener name.
Review comment:
Sorry for not updating the content of the document. Thanks @EronWright
for your suggestion. I fixed it. PTAL. Thanks.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
