You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by "Luke Chen (Jira)" <ji...@apache.org> on 2021/08/04 01:34:00 UTC

[jira] [Resolved] (KAFKA-13145) Renaming the time interval window for better understanding

     [ https://issues.apache.org/jira/browse/KAFKA-13145?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Luke Chen resolved KAFKA-13145.
-------------------------------
    Resolution: Won't Fix

> 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)