Re: [Bug?] Combiner components don't inherit annotations of source CombineByKey

2023-09-15 Thread Robert Bradshaw via dev
Very clear now :). Thanks for the fix; it looks good. On Fri, Sep 15, 2023 at 5:07 PM Joey Tran wrote: > Ended up just filing a PR [1] > > [1] https://github.com/apache/beam/pull/28489 > > > On Fri, Sep 15, 2023 at 12:51 PM Joey Tran > wrote: > >> While implementing a runner, we tried

Re: [Bug?] Combiner components don't inherit annotations of source CombineByKey

2023-09-15 Thread Joey Tran
Ended up just filing a PR [1] [1] https://github.com/apache/beam/pull/28489 On Fri, Sep 15, 2023 at 12:51 PM Joey Tran wrote: > While implementing a runner, we tried annotating a CombineByKey transform. > I noticed that the annotations for the CBK are then lost in the fusion > optimization

Re: [Bug?] Combiner components don't inherit annotations of source CombineByKey

2023-09-15 Thread Robert Bradshaw via dev
Could you clarify what you mean by annotating the transform? On Fri, Sep 15, 2023 at 9:57 AM Joey Tran wrote: > While implementing a runner, we tried annotating a CombineByKey transform. > I noticed that the annotations for the CBK are then lost in the fusion > optimization stage when the CBK