Github user mjsax commented on the pull request:
https://github.com/apache/flink/pull/750#issuecomment-135075011
Thanks for your review. I will add some more answers to the specific
comments (if necessary) and address all others. Do you want me to put a new
commit on top or should I squash directly?
About stop-failure-issue: need to think about it. But the "stop" signal is
intended to be a *request* to the streaming job to shut down. Thus, it's no
"hard" command. Furthermore, "stop" is idempotent -- for the failures case, if
the job is still in "stopping" mode after restart but does not stop, the "stop"
signal can just be sent a second time... Not sure if this is "good enough"?
---
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.
---
