[GitHub] flink issue #2321: [FLINK-4296] Fixes failure reporting of consumer task sch...

2016-08-01 Thread mxm
Github user mxm commented on the issue: https://github.com/apache/flink/pull/2321 +1 LGTM --- 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

[GitHub] flink issue #2321: [FLINK-4296] Fixes failure reporting of consumer task sch...

2016-08-01 Thread uce
Github user uce commented on the issue: https://github.com/apache/flink/pull/2321 Good catch. The change and test look good to me! This was broken for a long time (since the initial refactoring of the network stack I think). It never surfaced, because most use cases and tests run