[jira] [Commented] (KAFKA-3784) TimeWindows#windowsFor misidentifies some windows if TimeWindows#advanceBy is used

2016-06-02 Thread Tom Rybak (JIRA) 

[ 
https://issues.apache.org/jira/browse/KAFKA-3784?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15313374#comment-15313374
 ] 

Tom Rybak commented on KAFKA-3784:
--

I made a pull request with the fix, plus a unit test that can be used to verify 
that the problem used to exist. 

https://github.com/apache/kafka/pull/1462

> TimeWindows#windowsFor misidentifies some windows if TimeWindows#advanceBy is 
> used
> --
>
> Key: KAFKA-3784
> URL: https://issues.apache.org/jira/browse/KAFKA-3784
> Project: Kafka
>  Issue Type: Bug
>  Components: core
>Affects Versions: 0.10.0.0
>Reporter: Tom Rybak
>Priority: Minor
>
> Using a time window of size 6 minutes with a 5 minute advanceBy results in 
> some of the timestamped data being inserted into the previous overlapping 
> window even though the event's timestamp > that window's end time. 
> The fault lies in TimeWindows#windowsFor which does not check that all the 
> windows it's adding have an endTime > event's timestamp. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Created] (KAFKA-3784) TimeWindows#windowsFor misidentifies some windows if TimeWindows#advanceBy is used

2016-06-02 Thread Tom Rybak (JIRA) 
Tom Rybak created KAFKA-3784:


 Summary: TimeWindows#windowsFor misidentifies some windows if 
TimeWindows#advanceBy is used
 Key: KAFKA-3784
 URL: https://issues.apache.org/jira/browse/KAFKA-3784
 Project: Kafka
  Issue Type: Bug
  Components: core
Affects Versions: 0.10.0.0
Reporter: Tom Rybak
Priority: Minor


Using a time window of size 6 minutes with a 5 minute advanceBy results in some 
of the timestamped data being inserted into the previous overlapping window 
even though the event's timestamp > that window's end time. 

The fault lies in TimeWindows#windowsFor which does not check that all the 
windows it's adding have an endTime > event's timestamp. 




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)