+1 to fix it first. I also agree to deprecate it if there are few people using it, but this should be another discussion thread within dev+user ML.
In the future, we are planning to introduce user-defined-operator based on the TVF functionality which I think can fully subsume the UDTAG, cc @Timo Walther <[email protected]>. Best, Jark On Thu, 7 Sept 2023 at 11:44, Jane Chan <[email protected]> wrote: > Hi devs, > > Recently, we noticed an issue regarding a feature regression related to > Table API. `org.apache.flink.table.functions.TableAggregateFunction` > provides an API `emitUpdateWithRetract` [1] to cope with updated values, > but it's not being called in the code generator. As a result, even if users > override this method, it does not work as intended. > > This issue has been present since version 1.15 (when the old planner was > deprecated), but surprisingly, only two users have raised concerns about it > [2][3]. > > So, I would like to initiate a discussion to bring it back. Of course, if > few users use it, we can also consider deprecating it. > > [1] > > https://nightlies.apache.org/flink/flink-docs-master/docs/dev/table/functions/udfs/#retraction-example > [2] https://lists.apache.org/thread/rnvw8k3636dqhdttpmf1c9colbpw9svp > [3] https://www.mail-archive.com/[email protected]/msg15230.html > > Best, > Jane >
