[GitHub] flink issue #5849: [FLINK-8986][e2e-test][WIP] Flink end to end test REST AP...

2018-05-22 Thread walterddr 
Github user walterddr commented on the issue:

https://github.com/apache/flink/pull/5849
  
Thanks @tzulitai for the suggestion. I will close this and continue with 
the new PR in #6054 .


---

[GitHub] flink issue #5849: [FLINK-8986][e2e-test][WIP] Flink end to end test REST AP...

2018-05-21 Thread tzulitai 
Github user tzulitai commented on the issue:

https://github.com/apache/flink/pull/5849
  
@walterddr yes, you can do that. In the description, just leave some notice 
that the PR is based on another, and which of the commits are relevant.


---

[GitHub] flink issue #5849: [FLINK-8986][e2e-test][WIP] Flink end to end test REST AP...

2018-05-18 Thread walterddr 
Github user walterddr commented on the issue:

https://github.com/apache/flink/pull/5849
  
Hi @tzulitai . I've actually created a new version of the test based on 
@zentol 's comment on this PR: 
https://github.com/walterddr/flink/compare/FLINK-8985...walterddr:FLINK-8986-test

But it actually depends on https://github.com/apache/flink/pull/5863 as I 
reused the periodic stream job for testing. Is it possible to create a PR on 
top of another currently pending PR?


---

[GitHub] flink issue #5849: [FLINK-8986][e2e-test][WIP] Flink end to end test REST AP...

2018-05-18 Thread tzulitai 
Github user tzulitai commented on the issue:

https://github.com/apache/flink/pull/5849
  
Hi @walterddr, what is the status of this PR? Would be nice if we can move 
forward with this PR (and also the CLI e2e test PR that also you opened.)


---