prashantbh commented on PR #28201: URL: https://github.com/apache/flink/pull/28201#issuecomment-4933640808
Thanks for the review @XComp, and sorry for the slow turnaround. I was on vacation and then caught up with regular work. Added an IT — `JobMasterServiceLeadershipRunnerITCase` with a reusable `JobMasterServiceLeadershipRunnerTestHarness`, that runs the real runner over a real `DefaultJobMasterServiceProcess` and covers both event orderings deterministically. Confirmed it fails with the fix reverted, so it guards the regression. PTAL. -- 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]
