This is an automated email from the ASF dual-hosted git repository.
sijie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar-helm-chart.git
The following commit(s) were added to refs/heads/master by this push:
new bf5db57 Make forceSync by default as "yes" (#63)
bf5db57 is described below
commit bf5db574d1cb76cbd0fec2d7dfcd8cc9799bf44d
Author: Naveen Ramanathan <[email protected]>
AuthorDate: Tue Sep 22 20:17:41 2020 +0530
Make forceSync by default as "yes" (#63)
### Motivation
* It's not recommended to run a production zookkeeper cluster with
forceSync as "no". This is also mentioned in the forceSync section in
https://pulsar.apache.org/docs/en/next/reference-configuration/#zookeeper
### Modifications
* Removed ```-Dzookeeper.forceSync=no``` from ```values.yaml``` as default
```forceSync``` is ```yes```.
---
charts/pulsar/values.yaml | 1 -
1 file changed, 1 deletion(-)
diff --git a/charts/pulsar/values.yaml b/charts/pulsar/values.yaml
index 6ec3dc1..eb48d99 100644
--- a/charts/pulsar/values.yaml
+++ b/charts/pulsar/values.yaml
@@ -345,7 +345,6 @@ zookeeper:
-XX:+DoEscapeAnalysis
-XX:+DisableExplicitGC
-XX:+PerfDisableSharedMem
- -Dzookeeper.forceSync=no
## Zookeeper service
## templates/zookeeper-service.yaml
##
