[GitHub] flink issue #5890: [FLINK-8999] [e2e] Ensure the job has an operator with op...

2018-05-07 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/5890 @tzulitai Yes. Thanks ~ . I totally agree. I have been so busy these days. I will close this PR. ---

[GitHub] flink issue #5890: [FLINK-8999] [e2e] Ensure the job has an operator with op...

2018-05-06 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/5890 Hi @zhangminglei, I think the current general purpose DataStream job already subsumes this PR; for example, the sequence source generator in the job already uses operator state. Moreover,

[GitHub] flink issue #5890: [FLINK-8999] [e2e] Ensure the job has an operator with op...

2018-04-23 Thread StefanRRichter
Github user StefanRRichter commented on the issue: https://github.com/apache/flink/pull/5890 Hi @zhangminglei . I think there might unfortunately be a small misunderstanding: this issue was a subtask for the task of creating a single general purpose test job, i.e. a single job that

[GitHub] flink issue #5890: [FLINK-8999] [e2e] Ensure the job has an operator with op...

2018-04-23 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/5890 Hello, @StefanRRichter . Could you review this code please ? ---