Skip to content

Commit

Permalink
[tests] ManagedLedgerFactoryChangeLedgerPathTest is flaky (apache#5766)
Browse files Browse the repository at this point in the history
*Motivation*

ManagedLedgerFactoryChangeLedgerPathTest sets the zk session timeout to 10 ms.

10ms is too low for a zookeeper client to connect to the zookeeper server.

*Modifications*

Remove zkSessionTimeout and use the default
  • Loading branch information
sijie authored Nov 29, 2019
1 parent ecf7792 commit ae59d55
Showing 1 changed file with 0 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@ public void testChangeZKPath() throws Exception {
configuration.setUseV2WireProtocol(true);
configuration.setEnableDigestTypeAutodetection(true);
configuration.setAllocatorPoolingPolicy(PoolingPolicy.UnpooledHeap);
configuration.setZkTimeout(10);

ManagedLedgerFactory factory = new ManagedLedgerFactoryImpl(configuration, zkConnectString);

Expand Down

0 comments on commit ae59d55

Please sign in to comment.