[ https://issues.apache.org/jira/browse/FLINK-3396?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15147615#comment-15147615 ]
ASF GitHub Bot commented on FLINK-3396: --------------------------------------- Github user tillrohrmann commented on the pull request: https://github.com/apache/flink/pull/1633#issuecomment-184313039 Changes look good to me @uce. I had only one inline question concerning a semantic change. Apart from that +1 for merging. > Job submission Savepoint restore logic flawed > --------------------------------------------- > > Key: FLINK-3396 > URL: https://issues.apache.org/jira/browse/FLINK-3396 > Project: Flink > Issue Type: Bug > Reporter: Ufuk Celebi > Assignee: Ufuk Celebi > Fix For: 1.0.0 > > > When savepoint restoring fails, the thrown Exception fails the execution > graph, but the client is not informed about the failure. > The expected behaviour is that the submission should be acked with success or > failure in any case. With savepoint restore failures, the ack message will be > skipped. -- This message was sent by Atlassian JIRA (v6.3.4#6332)