[ 
https://issues.apache.org/jira/browse/FLINK-7491?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16143104#comment-16143104
 ] 

Jark Wu commented on FLINK-7491:
--------------------------------

Hi [~suez1224] , what I mean use {{Array}} instead of {{AbstractMultiSet}} is 
the runtime type, not sql type. For sql type, of course we should use the 
{{MultisetSqlType}}. But for the runtime type, you are using a guava 
{{AbstractMultiSet}} (collect elements into the AbstractMultiSet) which I think 
is poor performance, and I think maybe a Java array is enough for this. Calcite 
MultisetSqlType doesn't force us to use which Java type in runtime. 

> Support COLLECT Aggregate function in Flink SQL
> -----------------------------------------------
>
>                 Key: FLINK-7491
>                 URL: https://issues.apache.org/jira/browse/FLINK-7491
>             Project: Flink
>          Issue Type: New Feature
>          Components: Table API & SQL
>            Reporter: Shuyi Chen
>            Assignee: Shuyi Chen
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to