zhannao456 commented on PR #24618: URL: https://github.com/apache/pulsar/pull/24618#issuecomment-3175543738
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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
