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