[GitHub] flink issue #3531: [FLINK-6034][checkpoint] Add KeyedStateHandle for the sna...

2017-03-28 Thread shixiaogang
Github user shixiaogang commented on the issue: https://github.com/apache/flink/pull/3531 @StefanRRichter Thanks for your work. I will close the PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] flink issue #3531: [FLINK-6034][checkpoint] Add KeyedStateHandle for the sna...

2017-03-28 Thread StefanRRichter
Github user StefanRRichter commented on the issue: https://github.com/apache/flink/pull/3531 Merged in cd5527417a1cae57073a8855c6c3b88c88c780aa. @shixiaogang can you please close the PR? --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] flink issue #3531: [FLINK-6034][checkpoint] Add KeyedStateHandle for the sna...

2017-03-28 Thread StefanRRichter
Github user StefanRRichter commented on the issue: https://github.com/apache/flink/pull/3531 Thanks for this very nice contribution @shixiaogang! I will merge this now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If y

[GitHub] flink issue #3531: [FLINK-6034][checkpoint] Add KeyedStateHandle for the sna...

2017-03-28 Thread shixiaogang
Github user shixiaogang commented on the issue: https://github.com/apache/flink/pull/3531 @StefanRRichter I updated the PR as suggested. Very appreciated for your hard work. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] flink issue #3531: [FLINK-6034][checkpoint] Add KeyedStateHandle for the sna...

2017-03-24 Thread StefanRRichter
Github user StefanRRichter commented on the issue: https://github.com/apache/flink/pull/3531 @shixiaogang I had a few more comments on the updated PR. When they are resolved, I think this can be merged immediately. --- If your project is set up for it, you can reply to this email and

[GitHub] flink issue #3531: [FLINK-6034][checkpoint] Add KeyedStateHandle for the sna...

2017-03-23 Thread shixiaogang
Github user shixiaogang commented on the issue: https://github.com/apache/flink/pull/3531 @StefanRRichter Thanks a lot for your comments. I have updated the pull request as suggested, making the type of raw keyed states to be `KeyedStateHandle` as well. --- If your project is set up