[zookeeper] branch master updated: ZOOKEEPER-3863: Fix c client unit test (local session must be enabled for read only observers to work).

[hanm]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 4a2d582  ZOOKEEPER-3863: Fix c client unit test (local session must be 
enabled for read only observers to work).
4a2d582 is described below

commit 4a2d58219b7435c3b8cdf8f7ab04b158c1900223
Author: Michael Han <h...@apache.org>
AuthorDate: Sun Sep 20 08:53:16 2020 -0700

    ZOOKEEPER-3863: Fix c client unit test (local session must be enabled for 
read only observers to work).
    
    Problem
    
    ZOOKEEPER-3863 requires local session to be enabled for read only server to 
work. When merge ZOOKEEPER-3863 (in c47ef905e077184bc5b7f555a3e2dfeb6dc046e1), 
we only updated Java tests, and missed C tests. As a result, Jenkins build is 
broken.
    
    Solution
    
    Enable local session for C client tests.
    
    Notes
    This was not caught by Jenkins when we merge ZOOKEEPER-3863 because 
previous Jenkins was testing master branch (as discussed in 
https://github.com/apache/zookeeper/pull/1460).
    
    Author: Michael Han <l...@twitter.com>
    
    Reviewers: Enrico Olivelli <eolive...@gmail.com>
    
    Closes #1462 from hanm/fixme
---
 zookeeper-client/zookeeper-client-c/tests/zoo.cfg | 1 +
 1 file changed, 1 insertion(+)

diff --git a/zookeeper-client/zookeeper-client-c/tests/zoo.cfg 
b/zookeeper-client/zookeeper-client-c/tests/zoo.cfg
index b8e0b2a..d744423 100644
--- a/zookeeper-client/zookeeper-client-c/tests/zoo.cfg
+++ b/zookeeper-client/zookeeper-client-c/tests/zoo.cfg
@@ -3,6 +3,7 @@ initLimit=10
 syncLimit=5
 dataDir=TMPDIR/zkdata
 maxClientCnxns=MAXCLIENTCONNECTIONS
+localSessionsEnabled=true
 
 clientPort=CLIENTPORT
 secureClientPort=22281