Savonitar commented on PR #229: URL: https://github.com/apache/flink-connector-kafka/pull/229#issuecomment-3907973351
Hi @Efrat19 ! Thank you for the review. 1. I agree, that the stability is important, however, this rule has not been used for a long time (years), the removing does not change the current stability level. If we need retry logic somewhere, it would be better to apply it directly on the specific test classes that need it. 2. Good point. I have removed the unused annotations, methods, test class (`KafkaTestBaseWithFlink`), and updated the Javadoc. -- 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]
