[pulsar] branch master updated: Disable sticky read by default (#4526)

[pe...@apache.org]

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

penghui pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/master by this push:
     new d99e95c  Disable sticky read by default (#4526)
d99e95c is described below

commit d99e95c2a65ecb43df06bd742ecf0ff5e05d3afe
Author: Sijie Guo <si...@apache.org>
AuthorDate: Fri Jun 14 09:21:29 2019 +0800

    Disable sticky read by default (#4526)
    
    *Motivation*
    
    2.3.x releases turn on StickyRead by default.
    But StickyRead causes a lot of ArrayIndexOutOfBoundException.
    
    See: apache/bookkeeper#1970 and apache/pulsar#3715
    
    *Modifications*
    
    Disable sticy read by default until the bug is fixed and a new bookkeeper version is released.
---
 conf/broker.conf | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/conf/broker.conf b/conf/broker.conf
index 62a7db3..d231c77 100644
--- a/conf/broker.conf
+++ b/conf/broker.conf
@@ -447,7 +447,9 @@ bookkeeperClientMinAvailableBookiesInIsolationGroups=
 # Enable/disable having read operations for a ledger to be sticky to a single bookie.
 # If this flag is enabled, the client will use one single bookie (by preference) to read
 # all entries for a ledger.
-bookkeeperEnableStickyReads=true
+#
+# Disable Sticy Read until {@link https://github.com/apache/bookkeeper/issues/1970} is fixed
+bookkeeperEnableStickyReads=false
 
 ### --- Managed Ledger --- ###