Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/3359
@shixiaogang The changes sound helpful, considering that this PR was
implemented before incremental checkpoints. For a final opinion I would have to
review the code. One concern that I have is a wild grow of more and more
reported state types that we introduce. I am currently working on a feature
that introduces recovery from task-local state, which also looks like it will
add to the states that can be produced in a snapshot. So we might need some
form of consolidation in the future to avoid a "wild-grow" of more and more
state types that make everything more complex and harder to maintain. But
overall, I think the general idea is very good!
---
