sijie commented on a change in pull request #2362:  [documentation][deploy] 
Update deployment instructions for deploying pulsar to minikube
URL: https://github.com/apache/incubator-pulsar/pull/2362#discussion_r209694791
 
 

 ##########
 File path: deployment/kubernetes/generic/bookie.yaml
 ##########
 @@ -23,10 +23,10 @@ kind: ConfigMap
 metadata:
     name: bookie-config
 data:
-    PULSAR_MEM: "\" -Xms512m -Xmx512m -XX:MaxDirectMemorySize=1g\""
-    dbStorage_writeCacheMaxSizeMb: "256" # Write cache size (direct memory)
-    dbStorage_readAheadCacheMaxSizeMb: "256" # Read cache size (direct memory)
-    zkServers: zk-0.zookeeper,zk-1.zookeeper,zk-2.zookeeper
+    PULSAR_MEM: "\" -Xms64m -Xmx256m -XX:MaxDirectMemorySize=256m\""
 
 Review comment:
   the changes are I made here are reducing the memory usage for people to 
start. so they are still valid for a "custom" K8S cluster, we don't need to 
differentiate them. because in general, for a "custom" K8S cluster, we don't 
know exactly the resources people have, so starting from smaller values would 
be better for most people.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to