echauchot commented on PR #21035: URL: https://github.com/apache/flink/pull/21035#issuecomment-1327525295
@zentol what do you want that we do with this PR? Having no safe guard in the test waiting loop seems dangerous to me: during the implementation of a source, it could happen that the metrics counter is incorrect leading to infinite wait. IMHO I think we should merge this PR or with an alternative stop condition if we don't want the timeout. -- 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]
