ldadima commented on PR #23355:
URL: https://github.com/apache/flink/pull/23355#issuecomment-1709680878
> > > > Спасибо за пр. Я видел, что вас также перенесли на версию 1.15. Я
думаю, что это небольшое исправление для UT, так что можно просто исправить это
в мастере.
> > >
> > >
> > > Хорошо. Благодарим вас за отзыв о MR.
> >
> >
> > Но я думаю, что исправление этого теста можно было бы добавить не только
к мастеру. Потому что, например, в нашем случае этот тест не сработал, хотя
должен был провалиться.
>
> IIUC, это всего лишь вопрос теста. И это должно влиять только на тест, а
не на основной путь кода, верно?
>
> > Также думаю было бы неплохо добавить в этот тест проверку после
восстановления состояния
>
> Спасибо за совет. Для меня это имеет смысл, хотели бы вы также
предоставить такую проверку?
> > > > 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 ?
Yes, but this error in the test may affect the correctness of the state
backend improvements in the future (as for us).
>
> > 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?
Yes, no problem
--
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]
