lhotari commented on PR #23977: URL: https://github.com/apache/pulsar/pull/23977#issuecomment-2664834977
> Some test is requried to ensure the notification logic is triggered and works correct. @thetumbled That's right that there should be tests, but this just shows that the original code didn't have proper test coverage if it's currently broken. One possible resolution would be to add an issue report about the missing test coverage and add the tests later. That moment usually never comes, but it's also bad to have this issue around. -- 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]
