[GitHub] flink pull request: [FLINK-2180] [streaming] Iteration test fix

2015-07-17 Thread gaborhermann
Github user gaborhermann closed the pull request at: https://github.com/apache/flink/pull/802 --- 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

[GitHub] flink pull request: [FLINK-2180] [streaming] Iteration test fix

2015-07-17 Thread mbalassi
Github user mbalassi commented on the pull request: https://github.com/apache/flink/pull/802#issuecomment-15823 I think this has been merged manually by @gyfora. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] flink pull request: [FLINK-2180] [streaming] Iteration test fix

2015-07-13 Thread gaborhermann
Github user gaborhermann commented on the pull request: https://github.com/apache/flink/pull/802#issuecomment-120875478 Okay, thanks :) --- 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-2180] [streaming] Iteration test fix

2015-07-10 Thread gyfora
Github user gyfora commented on the pull request: https://github.com/apache/flink/pull/802#issuecomment-120276079 I am currently preparing a new PR with a rework of how the iterations are constructed in the StreamGraph it will be ready today. It will solve many issues that we

[GitHub] flink pull request: [FLINK-2180] [streaming] Iteration test fix

2015-07-07 Thread gaborhermann
Github user gaborhermann commented on the pull request: https://github.com/apache/flink/pull/802#issuecomment-119321451 The main purpose of this is to enable a deterministic (non-time based) test for iteration as it sometimes failed for some reason on Travis. (See [here]

[GitHub] flink pull request: [FLINK-2180] [streaming] Iteration test fix

2015-06-23 Thread gaborhermann
Github user gaborhermann commented on the pull request: https://github.com/apache/flink/pull/802#issuecomment-114525454 Thanks for the heads up, I removed the aggregation test. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] flink pull request: [FLINK-2180] [streaming] Iteration test fix

2015-06-17 Thread gaborhermann
Github user gaborhermann commented on the pull request: https://github.com/apache/flink/pull/802#issuecomment-112968975 I updated the PR. Added the fix for the iteration test. Probably not getting a new element within the maximum wait time caused the failing of the iteration tests.