[GitHub] flink pull request: [FLINK-2619] [tests] Fix for some unexecuted S...

2015-09-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/1103 --- 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 enab

[GitHub] flink pull request: [FLINK-2619] [tests] Fix for some unexecuted S...

2015-09-09 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1103#issuecomment-138922680 Testing my patch, then merging this... --- 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 projec

[GitHub] flink pull request: [FLINK-2619] [tests] Fix for some unexecuted S...

2015-09-09 Thread chiwanpark
Github user chiwanpark commented on the pull request: https://github.com/apache/flink/pull/1103#issuecomment-138868908 Thanks for clarifying. I added a commit for the manual 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-2619] [tests] Fix for some unexecuted S...

2015-09-09 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1103#issuecomment-138868121 Okay, the issue is that this mock execution never reaches a "FAILED" state in time, because the canceling of the other vertices (after one failed) is never acknowled

[GitHub] flink pull request: [FLINK-2619] [tests] Fix for some unexecuted S...

2015-09-09 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1103#issuecomment-138862613 Okay, let me merge your code and try to fix this test on the way... --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] flink pull request: [FLINK-2619] [tests] Fix for some unexecuted S...

2015-09-08 Thread chiwanpark
Github user chiwanpark commented on the pull request: https://github.com/apache/flink/pull/1103#issuecomment-138497195 @StephanEwen Do you mean `ExecutionGraphRestartTest`? When I increased slots in test instance, I got another exception. Exception log is following: ``` or

[GitHub] flink pull request: [FLINK-2619] [tests] Fix for some unexecuted S...

2015-09-08 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1103#issuecomment-138489873 Looks, good, but one of the tests seems to be actually in error. I think that the instance created in the test simply does not have enough slots, so the test executi

[GitHub] flink pull request: [FLINK-2619] [tests] Fix for some unexecuted S...

2015-09-08 Thread StephanEwen
Github user StephanEwen commented on a diff in the pull request: https://github.com/apache/flink/pull/1103#discussion_r38904456 --- Diff: flink-tests/src/test/scala/org/apache/flink/api/scala/misc/MassiveCaseClassSortingITCase.scala --- @@ -35,11 +35,13 @@ import org.apache.flink

[GitHub] flink pull request: [FLINK-2619] [tests] Fix for some unexecuted S...

2015-09-07 Thread chiwanpark
GitHub user chiwanpark opened a pull request: https://github.com/apache/flink/pull/1103 [FLINK-2619] [tests] Fix for some unexecuted Scala tests This PR includes to resolve FLINK-2619. The changes are following: * Add JUnitRunner(`@RunWith(classOf[JUnitRunner])`) for unexecu