BewareMyPower commented on PR #25707: URL: https://github.com/apache/pulsar/pull/25707#issuecomment-4477227142
@nodece Let me merge it first because this topic is out of the scope of the topic policies service itself. I agree that your comments [1] and [2] make sense, but it's more like an issue that should be resolved by structured concurrency, which needs careful design. It would be better add a common abstraction for listeners and skip all of them after closing the topic policies service. [1] https://github.com/apache/pulsar/pull/25707#discussion_r3258091432 [2] https://github.com/apache/pulsar/pull/25707#discussion_r3258114750 -- 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]
