BewareMyPower commented on PR #24947: URL: https://github.com/apache/pulsar/pull/24947#issuecomment-3490853238
Well, there has been a debate before in https://github.com/apache/pulsar/pull/24500/files#r2199584548 The timeout failure usually represents a unknown state. Removing it from the cache cannot cancel the ongoing topic loading process in background. There could be two concurrent topic loading tasks if we added the logic, some tests might fail (you can see the changes in #24500). We have to take care of this case. Topic loading for non-persistent topics is much more simple than persistent topics, so actually the timeout exception hardly happen. -- 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]
