Github user StephanEwen commented on the pull request:
https://github.com/apache/flink/pull/1305#issuecomment-158423218
Had an offline chat with @gyfora with the following outcome:
- A deterministic state identifier is needed
- Small change to pass that identifier as a single ID String, initially
internally constructed by state name + operator ID (as in this implementation)
- That way, the streaming runtime can change handling of state names and
operator IDs without breaking state backend implementations
With these changes, looks good to merge.
+1 from my side
---
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.
---
