RocMarshal commented on a change in pull request #13791:
URL: https://github.com/apache/flink/pull/13791#discussion_r513201162
##########
File path: docs/dev/table/tableApi.zh.md
##########
@@ -3172,7 +3176,7 @@ Table table = input
</td>
<td>
<p>Similar to a <b>GroupBy Aggregation</b>. Groups the rows on the
grouping keys with the following running table aggregation operator to
aggregate rows group-wise. The difference from an AggregateFunction is that
TableAggregateFunction may return 0 or more records for a group. You have to
close the "flatAggregate" with a select statement. And the select statement
does not support aggregate functions.</p>
- <p>Instead of using <code>emitValue</code> to output results, you can
also use the <code>emitUpdateWithRetract</code> method. Different from
<code>emitValue</code>, <code>emitUpdateWithRetract</code> is used to emit
values that have been updated. This method outputs data incrementally in
retract mode, i.e., once there is an update, we have to retract old records
before sending new updated ones. The <code>emitUpdateWithRetract</code> method
will be used in preference to the <code>emitValue</code> method if both methods
are defined in the table aggregate function, because the method is treated to
be more efficient than <code>emitValue</code> as it can output values
incrementally. See <a href="{{ site.baseurl
}}/dev/table/functions/udfs.html#table-aggregation-functions">Table Aggregation
Functions</a> for details.</p>
+ <p>Instead of using <code>emitValue</code> to output results, you can
also use the <code>emitUpdateWithRetract</code> method. Different from
<code>emitValue</code>, <code>emitUpdateWithRetract</code> is used to emit
values that have been updated. This method outputs data incrementally in
retract mode, i.e., once there is an update, we have to retract old records
before sending new updated ones. The <code>emitUpdateWithRetract</code> method
will be used in preference to the <code>emitValue</code> method if both methods
are defined in the table aggregate function, because the method is treated to
be more efficient than <code>emitValue</code> as it can output values
incrementally. See <a href="{% link dev/table/functions/udfs.md
%}#table-aggregation-functions">Table Aggregation Functions</a> for details.</p>
Review comment:
```suggestion
<p>Instead of using <code>emitValue</code> to output results, you
can also use the <code>emitUpdateWithRetract</code> method. Different from
<code>emitValue</code>, <code>emitUpdateWithRetract</code> is used to emit
values that have been updated. This method outputs data incrementally in
retract mode, i.e., once there is an update, we have to retract old records
before sending new updated ones. The <code>emitUpdateWithRetract</code> method
will be used in preference to the <code>emitValue</code> method if both methods
are defined in the table aggregate function, because the method is treated to
be more efficient than <code>emitValue</code> as it can output values
incrementally. See <a href="{% link dev/table/functions/udfs.zh.md
%}#table-aggregation-functions">Table Aggregation Functions</a> for details.</p>
```
##########
File path: docs/dev/table/tableApi.zh.md
##########
@@ -3571,9 +3575,9 @@ Please see the dedicated page about [data
types](types.html).
Generic types and (nested) composite types (e.g., POJOs, tuples, rows, Scala
case classes) can be fields of a row as well.
-Fields of composite types with arbitrary nesting can be accessed with [value
access functions]({{ site.baseurl
}}/dev/table/functions/systemFunctions.html#value-access-functions).
+Fields of composite types with arbitrary nesting can be accessed with [value
access functions]({% link dev/table/functions/systemFunctions.md
%}#value-access-functions).
-Generic types are treated as a black box and can be passed on or processed by
[user-defined functions]({{ site.baseurl }}/dev/table/functions/udfs.html).
+Generic types are treated as a black box and can be passed on or processed by
[user-defined functions]({% link dev/table/functions/udfs.md %}).
Review comment:
```suggestion
Generic types are treated as a black box and can be passed on or processed
by [user-defined functions]({% link dev/table/functions/udfs.zh.md %}).
```
##########
File path: docs/dev/table/tableApi.zh.md
##########
@@ -3571,9 +3575,9 @@ Please see the dedicated page about [data
types](types.html).
Generic types and (nested) composite types (e.g., POJOs, tuples, rows, Scala
case classes) can be fields of a row as well.
-Fields of composite types with arbitrary nesting can be accessed with [value
access functions]({{ site.baseurl
}}/dev/table/functions/systemFunctions.html#value-access-functions).
+Fields of composite types with arbitrary nesting can be accessed with [value
access functions]({% link dev/table/functions/systemFunctions.md
%}#value-access-functions).
Review comment:
```suggestion
Fields of composite types with arbitrary nesting can be accessed with [value
access functions]({% link dev/table/functions/systemFunctions.zh.md
%}#value-access-functions).
```
##########
File path: docs/dev/table/tableApi.zh.md
##########
@@ -3398,7 +3402,7 @@ val table = input
</td>
<td>
<p>Similar to a <b>GroupBy Aggregation</b>. Groups the rows on the
grouping keys with the following running table aggregation operator to
aggregate rows group-wise. The difference from an AggregateFunction is that
TableAggregateFunction may return 0 or more records for a group. You have to
close the "flatAggregate" with a select statement. And the select statement
does not support aggregate functions.</p>
- <p>Instead of using <code>emitValue</code> to output results, you can
also use the <code>emitUpdateWithRetract</code> method. Different from
<code>emitValue</code>, <code>emitUpdateWithRetract</code> is used to emit
values that have been updated. This method outputs data incrementally in
retract mode, i.e., once there is an update, we have to retract old records
before sending new updated ones. The <code>emitUpdateWithRetract</code> method
will be used in preference to the <code>emitValue</code> method if both methods
are defined in the table aggregate function, because the method is treated to
be more efficient than <code>emitValue</code> as it can output values
incrementally. See <a href="{{ site.baseurl
}}/dev/table/functions/udfs.html#table-aggregation-functions">Table Aggregation
Functions</a> for details.</p>
+ <p>Instead of using <code>emitValue</code> to output results, you can
also use the <code>emitUpdateWithRetract</code> method. Different from
<code>emitValue</code>, <code>emitUpdateWithRetract</code> is used to emit
values that have been updated. This method outputs data incrementally in
retract mode, i.e., once there is an update, we have to retract old records
before sending new updated ones. The <code>emitUpdateWithRetract</code> method
will be used in preference to the <code>emitValue</code> method if both methods
are defined in the table aggregate function, because the method is treated to
be more efficient than <code>emitValue</code> as it can output values
incrementally. See <a href="{% link dev/table/functions/udfs.md
%}#table-aggregation-functions">Table Aggregation Functions</a> for details.</p>
Review comment:
```suggestion
<p>Instead of using <code>emitValue</code> to output results, you
can also use the <code>emitUpdateWithRetract</code> method. Different from
<code>emitValue</code>, <code>emitUpdateWithRetract</code> is used to emit
values that have been updated. This method outputs data incrementally in
retract mode, i.e., once there is an update, we have to retract old records
before sending new updated ones. The <code>emitUpdateWithRetract</code> method
will be used in preference to the <code>emitValue</code> method if both methods
are defined in the table aggregate function, because the method is treated to
be more efficient than <code>emitValue</code> as it can output values
incrementally. See <a href="{% link dev/table/functions/udfs.zh.md
%}#table-aggregation-functions">Table Aggregation Functions</a> for details.</p>
```
----------------------------------------------------------------
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]