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]


Reply via email to