Github user zentol commented on the issue:
https://github.com/apache/flink/pull/2939
It should be part of this PR since this case simply didn't exist before the
new interface. The equivalent to no state was the state being null, at which
point restore was never called in the first place. Now, if the state is null we
get an empty list, afaik.
Here is what is confusing me: Every single function checks whether the
state is empty. Every one. So, there is apparently the possibility that it's
empty. But the behavior for that case does not seem well-defined.
According to the code receiving an empty state list is not a reason to fail
for any of these tests.
If this is the case we don't need to actually implement `restoreState` in
the first place since it is irrelevant to the result of the test
If this is not the case we should try to fail as early as possible by
adding a failure condition.
---
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.
---
