Github user kl0u commented on the issue:
https://github.com/apache/flink/pull/5955
Hi @tzulitai ! Thanks for the review. I integrated most of your comments.
The only one I left out is the one about merging the checkpointing and the
checking. I am not against that. It is just that the way it is now, it is
aligned with the `StatefulJobSavepointMigrationITCase`. If it were to make the
change, then we should change both and I would prefer to do it in a separate
commit.
Let me know what do you think about the current changes and if you are ok,
I can merge.
---
