apache / bookkeeper / 242

72%

push

jenkins

<a href="https://github.com/apache/bookkeeper/commit/<a class=hub.com/apache/bookkeeper/commit/345ee95ff5c1431c382ecd4ba8cae3bfed457d6d">345ee95ff<a href="https://github.com/apache/bookkeeper/commit/345ee95ff5c1431c382ecd4ba8cae3bfed457d6d">&quot;&gt;Set ConnectionExpired Listener to MetadataClientDriver in AR


Descriptions of the changes in this PR:

- add setConnectionExpiredListener method to MetadataClientDriver interface.
- add listener to shutdown AR in the case of metadata connection expiry


### Motivation

This commit - </a><a class="double-link" href="https://github.com/apache/bookkeeper/commit/<a class="double-link" href="https://github.com/apache/bookkeeper/commit/4f0d2a195bd9be3788876b47813cee1440cf005c">4f0d2a195</a>">4f0d2a195</a><a href="https://github.com/apache/bookkeeper/commit/345ee95ff5c1431c382ecd4ba8cae3bfed457d6d">, has removed the shutdown logic in AutoRecoveryMain incase of ZK client session expiry, with the following reason &quot;which doesn&#39;t make any sense for current retryable zookeeper&quot;. But if the ZK session has expired then it is not completely correct to let AutoRecoveryMain to continue to run in that state.

Author: Sijie Guo &lt;guosijie@gmail.com&gt;
Author: Andrey Yegorov &lt;dlg99@users.noreply.github.com&gt;
Author: cguttapalem &lt;cguttapalem@salesforce.com&gt;
Author: Charan Reddy Guttapalem &lt;reddycharan18@gmail.com&gt;
Author: Samuel Just &lt;sjust@salesforce.com&gt;

Reviewers: Sijie Guo &lt;sijie@apache.org&gt;, Enrico Olivelli <eolivelli@gmail.com>

This closes #1747 from reddycharan/conexpirylisten

26273 of 36147 relevant lines covered (72.68%)

0.73 hits per line