Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/5185
Thanks for the update. Regarding migration, I think the format that we
write to the savepoint must be the same between the two `ListState`
implementations.
@StefanRRichter could you please
Github user je-ik commented on the issue:
https://github.com/apache/flink/pull/5185
@aljoscha I (partly) reworked this PR as you suggest. There are still some
unresolved questions though:
1) I'm not 100% sure how to cleanly support the migration between list
state savepoints,
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/5185
The concept of this looks good. However, if we want to merge it I think we
have to make this a standalone thing and not depend on other states because
this can have unforeseen consequences for
Github user je-ik commented on the issue:
https://github.com/apache/flink/pull/5185
I think that the failed test is not related to this PR.
---
Github user bowenli86 commented on the issue:
https://github.com/apache/flink/pull/5185
This might be a great chance to start discussing how ListState APIs should
evolve - specifically, is it time to consider adding `remove()` to ListState?
---