[
https://issues.apache.org/jira/browse/FLINK-8873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16495448#comment-16495448
]
ASF GitHub Bot commented on FLINK-8873:
---------------------------------------
Github user bowenli86 commented on the issue:
https://github.com/apache/flink/pull/5649
When I was developing KeyedProcessFunction, I initially wondered why
there's no tests for KeyedStream, and researched and realized that they were
actually mixed with DataStream tests.
I think having that clarity by separating those tests would be great. Well,
I also agree it doesn't hurt that much to keep them as-is. If you feel strongly
against it, I can close this PR
> move unit tests of KeyedStream from DataStreamTest to KeyedStreamTest
> ---------------------------------------------------------------------
>
> Key: FLINK-8873
> URL: https://issues.apache.org/jira/browse/FLINK-8873
> Project: Flink
> Issue Type: Improvement
> Components: DataStream API, Tests
> Affects Versions: 1.5.0
> Reporter: Bowen Li
> Assignee: Bowen Li
> Priority: Minor
> Fix For: 1.6.0
>
>
> move unit tests of KeyedStream.scala from DataStreamTest.scala to
> KeyedStreamTest.scala, in order to have clearer separation
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
