[ 
https://issues.apache.org/jira/browse/FLINK-15841?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17029628#comment-17029628
 ] 

Aljoscha Krettek commented on FLINK-15841:
------------------------------------------

Which tests are failing? I will try and take a look.

I think failing tests would indicate that we shouldn't change the behaviour 
now, even though it's wrong according to the Javadoc. Mostly because of what I 
mentioned above about things having been like this for so long.

> TimeWindow.intersects return true for consecutive windows
> ---------------------------------------------------------
>
>                 Key: FLINK-15841
>                 URL: https://issues.apache.org/jira/browse/FLINK-15841
>             Project: Flink
>          Issue Type: Bug
>          Components: API / DataStream
>            Reporter: Jörn Kottmann
>            Assignee: Jörn Kottmann
>            Priority: Trivial
>
> The TimeWindow JavaDoc explains that the start index is inclusive and the end 
> index is exclusive, therefore two windows T0 to T1 and T1 to T2 are next to 
> each other without overlapping.
> To fix this the intersects comparison should be changed to: {{this.start < 
> other.end && this.end > other.start}}
> Also a test should be added to verify the methods works correctly.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to