[GitHub] flink issue #5547: [FLINK-8704] [tests] Port AccumulatorLiveITCase to MiniCl...

2018-02-21 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/5547 @aljoscha Okay. thanks. I will follow those issues. ---

[GitHub] flink issue #5547: [FLINK-8704] [tests] Port AccumulatorLiveITCase to MiniCl...

2018-02-21 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/5547 @zhangminglei I suggest you follow those issues and check out the solutions in the end. To be honest, I don't yet know how these difficult tests are going to be ported. ---

[GitHub] flink issue #5547: [FLINK-8704] [tests] Port AccumulatorLiveITCase to MiniCl...

2018-02-21 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/5547 I see ```verifyResults``` and I found it is too scared for me at this moment. ---

[GitHub] flink issue #5547: [FLINK-8704] [tests] Port AccumulatorLiveITCase to MiniCl...

2018-02-21 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/5547 Thanks @zentol Yes. I also find this issue is more difficult than what I thought before. I have found those test are really difficult when I keep working on this, as there are many internal

[GitHub] flink issue #5547: [FLINK-8704] [tests] Port AccumulatorLiveITCase to MiniCl...

2018-02-21 Thread zentol
Github user zentol commented on the issue: https://github.com/apache/flink/pull/5547 I would ask you to stop working on this issue. You do not know what changes are required to properly port these tests. Adding a `MiniClusterResource` (or extend `AbstractTestBase`) is