Thanks Jark for bringing this. +1 to use a unify name: "Interval Join" before 1.10 is release.
I think maybe "Interval Join" was come from SQL world too in [1]. Another candidate is to use "Range Join", But considering DataStream, I am OK with "Interval". [1] https://issues.apache.org/jira/browse/FLINK-8478 Best, Jingsong Lee On Mon, Dec 23, 2019 at 11:42 AM Jark Wu <imj...@gmail.com> wrote: > Hi everyone, > > Currently, in the Table API & SQL documentation[1], we call the joins with > time conditions as "Time-windowed Join". However, the same feature is > called "Interval Join" in DataStream[2]. 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. And "Windowed Join" should be joining > data in the same window, this is also described in DataStream API[3]. > > For Table API & SQL, the "Time-windowed Join" is the "Interval Join" in > DataStream. And the "Windowed Join" feature is missed in Table API & SQL. > > I would propose to correct the terminology in docs before 1.10 is release. > > What do you think? > > Best, > Jark > > [1]: > > https://ci.apache.org/projects/flink/flink-docs-master/dev/table/tableApi.html#joins > [2]: > > https://ci.apache.org/projects/flink/flink-docs-master/dev/stream/operators/joining.html#interval-join > [3]: > > https://ci.apache.org/projects/flink/flink-docs-master/dev/stream/operators/joining.html#window-join > -- Best, Jingsong Lee
