[GitHub] [pulsar] yjshen commented on issue #4573: [managed-ledger] create bookkeeper client during ManagedLedger shutdown

[GitBox]

yjshen commented on issue #4573: [managed-ledger] create bookkeeper client 
during ManagedLedger shutdown
URL: https://github.com/apache/pulsar/issues/4573#issuecomment-504258673
 
 
   @rdhabalia could you please take a look at this? It seems it's caused by 
#3933?
   
   ```java
       public void shutdown() throws InterruptedException, 
ManagedLedgerException {
               ......
               ......
   
            if (zookeeper != null) {
               zookeeper.close();
           }
   
           if (isBookkeeperManaged) {
               try {
                   BookKeeper bkFactory = bookkeeperFactory.get();    // this 
create a new bookkeeper using closed zk.
                   if (bkFactory != null) {
                       bkFactory.close();
                   }
               } catch (BKException e) {
                   throw new ManagedLedgerException(e);
               }
           }
   ``` 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services