From bf5db574d1cb76cbd0fec2d7dfcd8cc9799bf44d Mon Sep 17 00:00:00 2001
From: Naveen Ramanathan <45779883+naveen1100@users.noreply.github.com>
Date: Tue, 22 Sep 2020 20:17:41 +0530
Subject: [PATCH] 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
   ##