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