masteryhx commented on PR #23355: URL: https://github.com/apache/flink/pull/23355#issuecomment-1709561911
> > > Thanks for the pr. I have seen you also backported to 1.15. I think it's a minor fix for UT so that it's fine that we just fix it in the master. > > > > > > Ok. Thank you for taking on the MR review > > But I think fixing this test could be added to more than just the master. Because, for example, in our case this test didn't work although it should have failed. IIUC, it's just an issue of the test. And it should only affects the test not the main code path, right ? > Also I think it would be good to add to this test a check after restoring the state Thanks for the advice. This makes sense to me, would you like also to provide such verification? -- 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]
