leonardBang commented on a change in pull request #14152:
URL: https://github.com/apache/flink/pull/14152#discussion_r531342302



##########
File path: docs/dev/table/streaming/joins.md
##########
@@ -65,174 +65,147 @@ WHERE o.id = s.orderId AND
 
 Compared to a regular join operation, this kind of join only supports 
append-only tables with time attributes. Since time attributes are 
quasi-monotonic increasing, Flink can remove old values from its state without 
affecting the correctness of the result.
 
-Join with a Temporal Table Function
---------------------------
-
-A join with a temporal table function joins an append-only table (left 
input/probe side) with a temporal table (right input/build side),
-i.e., a table that changes over time and tracks its changes. Please check the 
corresponding page for more information about [temporal 
tables](temporal_tables.html).
+Temporal Joins
+--------------
+<span class="label label-danger">Attention</span> This feature is only 
supported in Blink planner.
+<span class="label label-danger">Attention</span> Attention it is only 
supported in SQL, and not supported in Table API yet.

Review comment:
       I agree your point, @morsapaes . Old temporal joins(using temporal table 
function + `lateral`) is still supported in Table API, the newborn temporal 
join(using temporal table DDL + temporal join syntax) is supported in SQL.
   How about :
   ```
   Temporal table has ways to define in Flink, i.e. [temporal table 
function]({% link dev/table/streaming/temporal_tables.md 
%}#temporal-table-function) ) and [temporal table DDL](({% link 
dev/table/streaming/temporal_tables.md %}#temporal-table-ddl)).
   Temporal join that using temporal table function is only supported in Table 
API,  temporal join that using temporal table DDL is only supported in SQL, 
please see the page [temporal table]({% link 
dev/table/streaming/temporal_tables.md %}) for more information about the 
differences between temporal table function and temporal table DDL.
   ```
    




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to