kezhuw commented on PR #1917: URL: https://github.com/apache/zookeeper/pull/1917#issuecomment-2310330257
> https://github.com/netty/netty/pull/13849 probably "fixed" this. I confirmed this with https://github.com/kezhuw/zookeeper/commit/0f2610b2f4116a4cd4e1b7c3ce86f4ab14aee029. After netty/netty#13849, `connectFuture.cancel(false)` will fire cancellation completion callback before next connection in netty event group. So there will be no ongoing connecting future in next iteration. But I am not sure whether there are other paths/conditions leading to uncancellable connecting future. Though, I am still prefer to fix this in ZooKeeper side. -- 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: notifications-unsubscr...@zookeeper.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
