Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/5439
Thanks for the reviews! Will merge.
---
Github user tzulitai commented on the issue:
https://github.com/apache/flink/pull/5439
LGTM once Travis is green.
We should also remove the redundant mention of FLINK-8571 in
`ReinterpretAsKeyedStreamITCase` while merging.
---
Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/5439
Added the test.
---
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/5439
The changes look good! But as @tzulitai mentioned the missing Scala tests
seem strange.
---
Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/5439
CC @aljoscha @tzulitai
---