Github user wuchong commented on the issue:
https://github.com/apache/flink/pull/3338
Hi @fhueske , yes, Calcite forces an Calc after each aggregate that only
renames fields, because we rename every aggregates in Table API which is not
necessary. I changed the logic of getting projections on aggregates to only
rename the duplicate aggregates. And that works good, no more Calc appended.
Hi @haohui , the ArrayRelDataType is still NOT NULL. I reverted [that
line](https://github.com/apache/flink/blob/master/flink-libraries/flink-table/src/main/scala/org/apache/flink/table/calcite/FlinkTypeFactory.scala#L121)
which is not need to be changed in this PR.
Cheers,
Jark Wu
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
