Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/4896
Thanks for your review @GJL. I've addressed all your comments. I rebased
this PR onto #4890. Once Travis gives green light, I'll merge it.
---
Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/4896
Yes, this can wait until after the fork. Moreover, I will do what
@StephanEwen suggested and count the number of executed tests before and after
of this change to make sure that we don't introdu
Github user zentol commented on the issue:
https://github.com/apache/flink/pull/4896
Do you intend to merge this for 1.4? (I would prefer merging it after the
fork).
---
Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/4896
It compiles now @zentol.
---
Github user zentol commented on the issue:
https://github.com/apache/flink/pull/4896
Doesn't compile:
```[INFO] flink-hadoop-compatibility . FAILURE [
5.449 s]
[INFO] flink-avro . SKIPPED
[INFO] flink-tests ..
Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/4896
The diffs looks good, but what I cannot judge in a final manner is whether
some tests now get not executed any more (accidentally).
What would be good is to take the Travis output from th
