rdhabalia commented on PR #17872: URL: https://github.com/apache/pulsar/pull/17872#issuecomment-1277076090
how can we miss such ClassCastException in the Pulsar code? didn't we add any unit-test case for the PR which caused this failure? certainly, not good practice in code have such RuntImeException. I am seeing a lot of such instances lately in Pulsar which should be avoided. Because of such practice, it takes a lot of minor releases to have a stable release. I am seeing a lot of necessary PRs without reviews and merging such PRs which keeps creating instability in code. -- 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]
