sunjincheng121 commented on a change in pull request #8359: 
[FLINK-11051][table] Add streaming window FlatAggregate to Table API
URL: https://github.com/apache/flink/pull/8359#discussion_r283174870
 
 

 ##########
 File path: 
flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/runtime/aggregate/AggregateAggFunction.scala
 ##########
 @@ -25,15 +25,19 @@ import org.apache.flink.table.util.Logging
 import org.apache.flink.types.Row
 
 /**
-  * Aggregate Function used for the aggregate operator in
-  * [[org.apache.flink.streaming.api.datastream.WindowedStream]]
+  * Aggregate Function used for the aggregate or table aggregate operator in
+  * [[org.apache.flink.streaming.api.datastream.WindowedStream]].
   *
   * @param genAggregations Generated aggregate helper function
 
 Review comment:
   `aggregate` or `table aggregate`?

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to