eolivelli commented on pull request #10899: URL: https://github.com/apache/pulsar/pull/10899#issuecomment-861261329
`This feature has been included in ManagedLedgerMetricsTest.testManagedLedgerMetrics().` Sorry, I do not understand your answer. If you change some line in the code, this should be an effect on tests: - make them fail , and in this case you usually have to modify the tests - in case the code you touch is not covered by a test, then you have to add a test it looks strange to me that you are changing a constant and there is no change in the tests. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
