Anonymitaet commented on a change in pull request #11266:
URL: https://github.com/apache/pulsar/pull/11266#discussion_r666618388



##########
File path: site2/docs/reference-configuration.md
##########
@@ -708,12 +708,6 @@ The [Pulsar 
proxy](concepts-architecture-overview.md#pulsar-proxy) can be config
 | brokerWebServiceURLTLS | The TLS Web service URL pointing to the broker 
cluster | |
 | functionWorkerWebServiceURL | The Web service URL pointing to the function 
worker cluster. It is only configured when you setup function workers in a 
separate cluster. | |
 | functionWorkerWebServiceURLTLS | The TLS Web service URL pointing to the 
function worker cluster. It is only configured when you setup function workers 
in a separate cluster. | |
-|brokerServiceURL|If service discovery is disabled, this url should point to 
the discovery service provider.|N/A|
-|brokerServiceURLTLS|If service discovery is disabled, this url should point 
to the discovery service provider.|N/A|
-|brokerWebServiceURL|This settings are unnecessary if `zookeeperServers` is 
specified.|N/A|
-|brokerWebServiceURLTLS|This settings are unnecessary if `zookeeperServers` is 
specified.|N/A|
-|functionWorkerWebServiceURL|If function workers are setup in a separate 
cluster, configure the this setting to point to the function workers 
cluster.|N/A|

Review comment:
       The descriptions are not the same and should be added to the 1st 
property?
   If so, pls check all similar cases.
   
   
![image](https://user-images.githubusercontent.com/50226895/125011302-c8009980-e09a-11eb-8a88-2e512ce694f3.png)
   




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


Reply via email to