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



##########
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 yes, 64-bit JVM belongs to the system requirements. 
   
   A quick question: for standalone, bare mental, and bare mental 
(multi-cluster), `Note: broker is only supported on 64-bit JVM` is suitable for 
these 3 situations? 
   
   If so, to make the instructions more clear, I think we can add this note to 
the `standalone` chapter as well.
   
![image](https://user-images.githubusercontent.com/50226895/118748330-3ccf0880-b88e-11eb-89c1-6128f46b48f1.png)
   
   @codelipenghui may we know your feedback on this issue? 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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to