Github user gyfora commented on the pull request:
https://github.com/apache/flink/pull/1562#issuecomment-177611308
Hey,
First of all, great work I am looking forward to having this is :)
I think it would be good if we added default implementations of the List,
Reducing state based on the ValueState and those would be the default
implementations returned by the AbstractStateBackend. I think this should go in
before merging this.
Now that are approaching the release we should make the interfaces clean
and future additions easy and I believe this is necessary for this. I think
many backends will not have a very efficient way of implementing List and
Reduce states and they will naturally fall back to the default implementation.
What do you think?
Gyula
---
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.
---
