[GitHub] flink pull request: [FLINK-2804] [client, runtime] Add blocking jo...

2015-10-08 Thread uce
Github user uce closed the pull request at: https://github.com/apache/flink/pull/1230 --- 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

[GitHub] flink pull request: [FLINK-2804] [client, runtime] Add blocking jo...

2015-10-08 Thread uce
Github user uce commented on the pull request: https://github.com/apache/flink/pull/1230#issuecomment-146507072 Closing this in favour of Tills upcoming PR. --- 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

[GitHub] flink pull request: [FLINK-2804] [client, runtime] Add blocking jo...

2015-10-06 Thread uce
GitHub user uce opened a pull request: https://github.com/apache/flink/pull/1230 [FLINK-2804] [client, runtime] Add blocking job submission support in… … … case of JobManager recovery This is based on some other PRs. Only review the last commit. The client submit and

[GitHub] flink pull request: [FLINK-2804] [client, runtime] Add blocking jo...

2015-10-06 Thread tillrohrmann
Github user tillrohrmann commented on the pull request: https://github.com/apache/flink/pull/1230#issuecomment-145812474 I'm on it :-) --- 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

[GitHub] flink pull request: [FLINK-2804] [client, runtime] Add blocking jo...

2015-10-06 Thread tillrohrmann
Github user tillrohrmann commented on the pull request: https://github.com/apache/flink/pull/1230#issuecomment-145846664 For the sake of completeness I repost what I've just commented on the last commit: I think it would be better to move the `JobManager` retrieval logic into

[GitHub] flink pull request: [FLINK-2804] [client, runtime] Add blocking jo...

2015-10-06 Thread uce
Github user uce commented on the pull request: https://github.com/apache/flink/pull/1230#issuecomment-145847936 OK, I like your idea! And thanks for finishing it. It depends on the job graph recovery for the added IT cases to work. I think you can work your way around the tests, but