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/05/18 07:46:33 UTC

[GitHub] [pulsar] Anonymitaet commented on a change in pull request #10605: [Doc] Add descriptions for broker

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



##########
File path: site2/docs/concepts-architecture-overview.md
##########
@@ -29,6 +29,10 @@ Messages are typically dispatched out of a [managed ledger](#managed-ledgers) ca
 
 Finally, to support geo-replication on global topics, the broker manages replicators that tail the entries published in the local region and republish them to the remote region using the Pulsar [Java client library](client-libraries-java.md).
 
+> **Note**
+> 
+> Broker is thread-safe when running on 64-bit JVM.

Review comment:
       @lhotari many thanks for your details!
   
   How about adding the contents in these two places?
   
   - In [`Deployment > Bare metal > Deploy Pulsar brokers`](https://pulsar.apache.org/docs/en/next/deploy-bare-metal/#deploy-pulsar-brokers)
   ![image](https://user-images.githubusercontent.com/50226895/118612160-15296300-b7f0-11eb-9d91-cb6b51c45791.png)
   
   - In [`Deployment > Bare metal multi-cluster > Deploy Pulsar brokers`](https://pulsar.apache.org/docs/en/next/deploy-bare-metal-multi-cluster/#deploy-brokers)
   ![image](https://user-images.githubusercontent.com/50226895/118611992-f1661d00-b7ef-11eb-8ef9-cf9bca7d226c.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.

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