[
https://issues.apache.org/jira/browse/KAFKA-13145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17392615#comment-17392615
]
A. Sophie Blee-Goldman commented on KAFKA-13145:
------------------------------------------------
Fine with me. FWIW the "InclusiveExclusiveWindow" name was my idea, but that
was just to avoid using something called "SessionWindow" in the _Sliding_
window processor – making a new SlidingWindow class works too.
> Renaming the time interval window for better understanding
> ----------------------------------------------------------
>
> Key: KAFKA-13145
> URL: https://issues.apache.org/jira/browse/KAFKA-13145
> Project: Kafka
> Issue Type: Improvement
> Components: streams
> Reporter: Luke Chen
> Assignee: Luke Chen
> Priority: Major
>
>
> I have another thought, which is to rename the time interval related windows.
> Currently, we have 3 types of time interval window:
> {{TimeWindow}} -> to have {{[start,end)}} time interval
> {{SessionWindow}} -> to have {{[start,end]}} time interval
> {{UnlimitedWindow}} -> to have {{[start, MAX_VALUE)}} time interval
> I think the name {{SessionWindow}} is definitely not good here, especially we
> want to use it in {{SlidingWindows}} now, although it is only used for
> {{SessionWindows}} before. We should name them with time interval meaning,
> not the streaming window functions meaning. {{}}Because these 3 window types
> are internal use only, it is safe to rename them.
>
> {{TimeWindow}} --> {{InclusiveExclusiveWindow}}
> {{SessionWindow}} / {{SlidingWindow}} --> {{InclusiveInclusiveWindow}}
> {{UnlimitedWindow}} --> {{InclusiveUnboundedWindow}}
> {{}}
> See the discussion here{{:
> [https://github.com/apache/kafka/pull/11124#issuecomment-887989639]}}
> {{}}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
