Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/3834
I was already familiar with the high level design of this PR from my
discussions with @tzulitai. As the feature freeze is already tomorrow, I had to
focus my review on the functionality of the design (i.e. if all cases in
backwards compatibility that we want to cover are possible and future plans
like state transformation can base upon this work). I also had a look into a
couple of important tests, but could not go through all the implementation
details for now.
Overall, I think this is very good work and improves a ton of things w.r.t.
everything that has to do with serialization format updates and backwards
compatibility. I suggest that we proceed to merge this, so that the code makes
it into the release. We can still do potential refinements and fixes in the QA
phase, in case they are needed.
@tzulitai if you agree, please feel free to merge this. +1
---
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 and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
