[pulsar-helm-chart] branch merlimat-patch-1 created (now d5d3e2c)

[mm...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

mmerli pushed a change to branch merlimat-patch-1
in repository https://gitbox.apache.org/repos/asf/pulsar-helm-chart.git.


      at d5d3e2c  Use regular 2-2-2 BK client settings by default

This branch includes the following new commits:

     new d5d3e2c  Use regular 2-2-2 BK client settings by default

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

[pulsar-helm-chart] 01/01: Use regular 2-2-2 BK client settings by default

[mm...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

mmerli pushed a commit to branch merlimat-patch-1
in repository https://gitbox.apache.org/repos/asf/pulsar-helm-chart.git

commit d5d3e2c0105fb54e76f9d63e05c2cf5955374a81
Author: Matteo Merli <mm...@apache.org>
AuthorDate: Thu May 21 15:34:10 2020 -0700

    Use regular 2-2-2 BK client settings by default
    
    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