[GitHub] [pulsar] Anonymitaet commented on a diff in pull request #15943: [feature][doc] Add Pulsar Auth support for Pulsar SQL

[GitBox <gi...@apache.org>]

Anonymitaet commented on code in PR #15943:
URL: https://github.com/apache/pulsar/pull/15943#discussion_r889831732


##########
site2/docs/sql-deployment-configurations.md:
##########
@@ -77,6 +80,9 @@ pulsar.tls-hostname-verification-enable=null
 # path for the trusted TLS certificate file of Pulsar broker
 pulsar.tls-trust-cert-file-path=null
 
+## whether to enable pulsar authorization

Review Comment:
   ```suggestion
   ## whether to enable Pulsar authorization
   ```



##########
site2/docs/sql-deployment-configurations.md:
##########
@@ -17,7 +17,10 @@ connector.name=pulsar
 # the url of Pulsar broker service
 pulsar.web-service-url=http://localhost:8080
 
-# URI of Zookeeper cluster
+# the url of Pulsar broker binary service

Review Comment:
   ```suggestion
   # the URL of Pulsar broker binary service
   ```



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org