[GitHub] [bookkeeper] StevenLuMT opened a new pull request, #3393: Bugfix for zooKeeper client

[GitBox]

StevenLuMT opened a new pull request, #3393:
URL: https://github.com/apache/bookkeeper/pull/3393

   Descriptions of the changes in this PR:
   
   ### Motivation
   
      zkRetryBackoffMaxRetries is set MAX, 
      when the ZooKeeperClient is already closed, 
      the syncCall can't perception this close action,when the old task is 
fail, new tasks cannot be executed,
      then the ZooWorker.syncCallWithRetries can's exited
   
   The screenshots of these codes are to illustrate that when ZooKeeperClient 
is closed, new tasks cannot be executed
   <img width="956" alt="image" 
src="https://user-images.githubusercontent.com/42990025/177714250-17ed3e4b-1417-45b0-b09a-d28d837bbcf8.png";>
   <img width="687" alt="image" 
src="https://user-images.githubusercontent.com/42990025/177714499-2e48cf82-1070-4003-b6db-e6eba92253d5.png";>
   
   
   
   ### Changes
   
   1. add the judgment of whether to close in ZooWorker.syncCallWithRetries
   2. update testcase's zkRetryBackoffMaxRetries to default 
value(Integer.MAX_VALUE)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org