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

Jark Wu updated FLINK-15339:
----------------------------
    Description: 
Currently, in the docuementation, we call the joins with time conditions as 
"Time-windowed Join". However, it is called "Interval Join" in DataStream. We 
should align the terminology in Flink project. 

>From my point of view, "Interval Join" is more suitable, because it joins a 
>time interval range of right stream[1]. And "Windowed Join" should be joins 
>data in the same window, this is also described in DataStream API. 

For Table API & SQL, the "Time-windowed Join" is the "Interval Join" in 
DataStream. And we miss the new feature "Windowed Join" in Table API & SQL.

I propose to correct the terminology in docs before 1.10 is release. 


[1]: 
https://ci.apache.org/projects/flink/flink-docs-master/dev/stream/operators/joining.html#interval-join
[2]: 
https://ci.apache.org/projects/flink/flink-docs-master/dev/stream/operators/joining.html#window-join

Discussion thread: 
http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/DISCUSS-Correct-the-terminology-of-quot-Time-windowed-Join-quot-to-quot-Interval-Join-quot-in-Table-L-td36202.html#a36208
Voting thread: 
http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/VOTE-Rename-terminology-quot-Time-windowed-Join-quot-to-quot-Interval-Join-quot-in-Table-API-amp-SQL-td36370.html

  was:
Currently, in the docuementation, we call the joins with time conditions as 
"Time-windowed Join". However, it is called "Interval Join" in DataStream. We 
should align the terminology in Flink project. 

>From my point of view, "Interval Join" is more suitable, because it joins a 
>time interval range of right stream[1]. And "Windowed Join" should be joins 
>data in the same window, this is also described in DataStream API. 

For Table API & SQL, the "Time-windowed Join" is the "Interval Join" in 
DataStream. And we miss the new feature "Windowed Join" in Table API & SQL.

I propose to correct the terminology in docs before 1.10 is release. 


[1]: 
https://ci.apache.org/projects/flink/flink-docs-master/dev/stream/operators/joining.html#interval-join
[2]: 
https://ci.apache.org/projects/flink/flink-docs-master/dev/stream/operators/joining.html#window-join

Discussion thread: 
http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/DISCUSS-Correct-the-terminology-of-quot-Time-windowed-Join-quot-to-quot-Interval-Join-quot-in-Table-L-td36202.html#a36208


> Correct the terminology of "Time-windowed Join" to "Interval Join" in Table 
> API & SQL
> -------------------------------------------------------------------------------------
>
>                 Key: FLINK-15339
>                 URL: https://issues.apache.org/jira/browse/FLINK-15339
>             Project: Flink
>          Issue Type: Task
>          Components: Documentation, Table SQL / API
>            Reporter: Jark Wu
>            Assignee: Jark Wu
>            Priority: Blocker
>             Fix For: 1.11.0
>
>
> Currently, in the docuementation, we call the joins with time conditions as 
> "Time-windowed Join". However, it is called "Interval Join" in DataStream. We 
> should align the terminology in Flink project. 
> From my point of view, "Interval Join" is more suitable, because it joins a 
> time interval range of right stream[1]. And "Windowed Join" should be joins 
> data in the same window, this is also described in DataStream API. 
> For Table API & SQL, the "Time-windowed Join" is the "Interval Join" in 
> DataStream. And we miss the new feature "Windowed Join" in Table API & SQL.
> I propose to correct the terminology in docs before 1.10 is release. 
> [1]: 
> https://ci.apache.org/projects/flink/flink-docs-master/dev/stream/operators/joining.html#interval-join
> [2]: 
> https://ci.apache.org/projects/flink/flink-docs-master/dev/stream/operators/joining.html#window-join
> Discussion thread: 
> http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/DISCUSS-Correct-the-terminology-of-quot-Time-windowed-Join-quot-to-quot-Interval-Join-quot-in-Table-L-td36202.html#a36208
> Voting thread: 
> http://apache-flink-mailing-list-archive.1008284.n3.nabble.com/VOTE-Rename-terminology-quot-Time-windowed-Join-quot-to-quot-Interval-Join-quot-in-Table-API-amp-SQL-td36370.html



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

Reply via email to