Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/3508
Looks good in general. Some thoughts on refinement:
- I think we should not really encourage the use of Java Serialization,
so I would suggest to remove the serializable shortcut for broadcast state. We
have it for the default operator state only as a bridge to mimick the previous
behavior (easier migration)
- Since broadcast state already defines its redistribution pattern
(broadcast), the shape of the datastructure is orthogonal. We can for example
also have broadcast map state, broadcast value state, etc. Hence, I would
suggest to take "list" somewhere into the name of the methods.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
