You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@sedona.apache.org by GitBox <gi...@apache.org> on 2022/08/17 08:17:21 UTC

[GitHub] [incubator-sedona] Elephantusparvus commented on pull request #662: [SEDONA-142] Add ST_Collect to Flink Catalog

Elephantusparvus commented on PR #662:
URL: https://github.com/apache/incubator-sedona/pull/662#issuecomment-1217667268

   > Thanks for your contribution. I have a much neat implementation of spatial aggregation in Flink using the UDAF api. It has been there for a while and I just forgot to create a PR.
   > 
   > I will create a new PR in one or two days. Then you can follow my implementation to tweak your ST_Collect a bit.
   
   Sounds good. As stated earlier this really just is a copy/minimal port of flinks built in CollectAggFunction with a different output type (GeometryCollection instead of a MultiSet<Geometry> ), but using the AggregateFunction Interface instead of BuiltInAggregateFunction.


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org