luoyuxia commented on PR #21257: URL: https://github.com/apache/flink/pull/21257#issuecomment-1366633565
> @beyond1920 Thanks for reviewing. I prefer to append a commit with tests to mock the speculative execution to validate these operators. Actaully, I don't think we need to add tests to mock peculative execution for the test will be just like create multiple operators and make operator to process element and make sure it'll generate different paths for different attempts. Do you think we still need to add these test. @beyond1920 -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
