Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/2962
CC @aljoscha @StephanEwen
---
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 have this feature
enabled an
Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/2962
CC @aljoscha @kl0u @StephanEwen
---
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 have this feature
enab
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/2962
Thanks for changing! I just realised, however, that we already have tests
in `StateBackendTestBase` that check for restoring from wrong serialisers so we
don't actually need those.
I will a
Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/2962
Yes, the commits you mentioned are actually meant as fixups for my previous
PR. I think it would be good to separate this a bit if possible. Thanks! :-)
---
If your project is set up for it,
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/2962
I merged this.
Thanks a lot for your work. ð Could you please close this PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel