twalthr commented on a change in pull request #9545: [FLINK-13355][docs] Add
documentation for Temporal Table Join in blink planner
URL: https://github.com/apache/flink/pull/9545#discussion_r319475250
##########
File path: docs/dev/table/sql.md
##########
@@ -647,6 +647,29 @@ WHERE
<p>For more information please check the more detailed <a
href="streaming/temporal_tables.html">temporal tables concept
description</a>.</p>
</td>
</tr>
+ <tr>
+ <td>
+ <strong>Join with Temporal Table</strong><br>
+ <span class="label label-primary">Batch</span> <span class="label
label-primary">Streaming</span>
+ </td>
+ <td>
+ <p><a href="streaming/temporal_tables.html">Temporal Tables</a> are
tables that track changes over time.
+ A <a href="streaming/temporal_tables.html#temporal-table">Temporal
Table</a> provides access to the versions of a temporal table at a specific
point in time.</p>
+
+ <p>Only inner and left joins with processing-time temporal tables are
supported.</p>
+ <p>The following example assumes that <strong>LatestRates</strong> is
a <a href="streaming/temporal_tables.html#temporal-table">Temporal Table</a>
which is materialized with the latest rate.</p>
+{% highlight sql %}
+SELECT
+ o.amout, o.currency, r.rate, o.amount * r.rate
+FROM
+ Orders AS o
+ JOIN LatestRates FOR SYSTEM_TIME AS OF o.proctime AS r
+ ON r.currency = o.currency
+{% endhighlight %}
+ <p>For more information please check the more detailed <a
href="streaming/temporal_tables.html">Temporal Tables</a> concept
description.</p>
+ <p>Only supported in blink planner.</p>
Review comment:
nit: could we agree on writing `Blink` in capital letters? Similar to
`Flink` is kind of brand name.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
