Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/4585#discussion_r141111367 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/runtime/aggregate/AggregateUtil.scala --- @@ -1414,8 +1414,29 @@ object AggregateUtil { aggregates(index) = udagg.getFunction accTypes(index) = udagg.accType - case unSupported: SqlAggFunction => --- End diff -- Since we add a dedicated case for `COLLECT`, this case should not be remain at the end of this `match`.
---