[pulsar-helm-chart] branch master updated: Use regular 2-2-2 BK client settings by default (#13)

[si...@apache.org]

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 6e9ad25  Use regular 2-2-2 BK client settings by default (#13)
6e9ad25 is described below

commit 6e9ad25ba322f6f0fc7c11c66fb88faa6d0218db
Author: Matteo Merli <mm...@apache.org>
AuthorDate: Thu May 21 21:52:53 2020 -0700

    Use regular 2-2-2 BK client settings by default (#13)
    
    Using write=3 and ack=2 leads to unbound memory usage in BK client when one bookie is slow or failing, so we should avoid it by default.
---
 charts/pulsar/values.yaml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/charts/pulsar/values.yaml b/charts/pulsar/values.yaml
index 8822896..e2b7703 100644
--- a/charts/pulsar/values.yaml
+++ b/charts/pulsar/values.yaml
@@ -599,8 +599,8 @@ broker:
       -XX:+UseG1GC
       -XX:MaxGCPauseMillis=10
       "
-    managedLedgerDefaultEnsembleSize: "3"
-    managedLedgerDefaultWriteQuorum: "3"
+    managedLedgerDefaultEnsembleSize: "2"
+    managedLedgerDefaultWriteQuorum: "2"
     managedLedgerDefaultAckQuorum: "2"
   ## Broker service
   ## templates/broker-service.yaml