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