Github user mjsax commented on the pull request:
https://github.com/apache/flink/pull/750#issuecomment-138587744
Yes. That is what I assumed writing the code for this PR. The question is,
if you still simply assume that `cancel` behaves this way or try to enforce it
somehow. Furthermore, the assumption about non-blocking behavior is not well
documented. Should we at least update the JavaDoc for `cancel`?
This question is blocking this PR to get merged. So we need to get to a
conclusion if there is an issue or not (and if I need to change my code or this
PR can just get merged).
I personally would only update the JavaDoc.
---
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.
---
