prestodb / presto

The official home of the Presto distributed SQL query engine for big data
http://prestodb.io
Apache License 2.0
16.05k stars 5.38k forks source link

Use @TypeParameter to simplify multimap_agg aggregation function #21364

Open ZacBlanco opened 1 year ago

ZacBlanco commented 1 year ago

With the merge of #21316 we can now simplify this aggregation function implementation by using @TypeParameter annotations in the serializer and state factory constructors, which means the function can be entirely converted to using the function annotation framework. This will simplify the implementation and improve maintainability.

weimingdiit commented 10 months ago

@ZacBlanco i would like to work on this.