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



##########
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:
       Aren't temporal joins still supported in the Table API using the old 
method (and listed 
[here](https://ci.apache.org/projects/flink/flink-docs-release-1.12/dev/table/tableApi.html#joins))?
 If so, this sentence is a bit misleading.




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