lhotari commented on PR #23354: URL: https://github.com/apache/pulsar/pull/23354#issuecomment-2410666125
> These changes not cover any issue. Changes are covered by existing tests. This PR just improves code. @nikam14 That doesn't make much sense. In the motivation there is "So, checking it in getSchema(String schemaId, SchemaVersion version) can improve code, prevent from any future issue.". Please add a test case for the potential future issue. -- 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]
