mjsax commented on PR #16970: URL: https://github.com/apache/kafka/pull/16970#issuecomment-2316321485
> Given that the goal of the test is to prove onRestoreSuspended get executed, I've removed the race condition for onRestoreSuspended and now check at the end of the test it's been called at least once. Not sure if I see why this would be different? If it's call because restore did not finish yet, the old and new code would pass, and if it's not called before restore was to fast, the old and new code would fail? -- 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]
