You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by "gaozhangmin (via GitHub)" <gi...@apache.org> on 2025/08/11 16:05:56 UTC

Re: [PR] [fix][metadatastore] Support set bk client zkTimeout [pulsar]

gaozhangmin commented on PR #24618:
URL: https://github.com/apache/pulsar/pull/24618#issuecomment-3175612090

   To avoid the path resolution bug in AbstractMetadataDriver, we should remove /ledgers from the bookkeeperMetadataServiceUri configuration .  Since that, this pr wouldn't be needed any more.
   
   But I found there is another problem, the default zktimeout of bk is 10s, but 
   We caculate tickTimeMillis  =  zktimeout/15. 
   
   https://github.com/apache/pulsar/blob/277835a65dd586de941b7d0ad448843b7b5589a4/pulsar-metadata/src/main/java/org/apache/pulsar/metadata/impl/ZKSessionWatcher.java#L63
   
   In below future.get, always timeout.
   https://github.com/apache/pulsar/blob/277835a65dd586de941b7d0ad448843b7b5589a4/pulsar-metadata/src/main/java/org/apache/pulsar/metadata/impl/ZKSessionWatcher.java#L108-L112
   
   We should support set bk zktimeut by 
    bkConf.setZkTimeout((int) conf.getMetadataStoreSessionTimeoutMillis());
   
   


-- 
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: commits-unsubscribe@pulsar.apache.org

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