[ 
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)

Reply via email to