[ https://issues.apache.org/jira/browse/FLINK-8659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16477648#comment-16477648 ]
ASF GitHub Bot commented on FLINK-8659: --------------------------------------- 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. > Add migration tests for Broadcast state. > ---------------------------------------- > > Key: FLINK-8659 > URL: https://issues.apache.org/jira/browse/FLINK-8659 > Project: Flink > Issue Type: Improvement > Components: Streaming > Affects Versions: 1.5.0 > Reporter: Kostas Kloudas > Assignee: Kostas Kloudas > Priority: Major > Fix For: 1.5.0 > > -- This message was sent by Atlassian JIRA (v7.6.3#76005)