+1 from my side, thanks Martijn for the effort. Best, Leonard
> 2024年5月4日 下午7:41,Ahmed Hamdy <hamdy10...@gmail.com> 写道: > > Hi Martijn > Thanks for the proposal +1 from me. > Should this change take place in 1.20, what are the planned release steps > for connectors that only offer a deprecated interface in this case (i.e. > RabbitMQ, Cassandra, pusbub, Hbase)? Are we going to refrain from releases > that support 1.20+ till the blockers are implemented? > Best Regards > Ahmed Hamdy > > > On Fri, 3 May 2024 at 14:32, Péter Váry <peter.vary.apa...@gmail.com> wrote: > >>> With regards to FLINK-35149, the fix version indicates a change at Flink >> CDC; is that indeed correct, or does it require a change in the SinkV2 >> interface? >> >> The fix doesn't need change in SinkV2, so we are good there. >> The issue is that the new SinkV2 SupportsCommitter/SupportsPreWriteTopology >> doesn't work with the CDC yet. >> >> Martijn Visser <martijnvis...@apache.org> ezt írta (időpont: 2024. máj. >> 3., >> P, 14:06): >> >>> Hi Ferenc, >>> >>> You're right, 1.20 it is :) >>> >>> I've assigned the HBase one to you! >>> >>> Thanks, >>> >>> Martijn >>> >>> On Fri, May 3, 2024 at 1:55 PM Ferenc Csaky <ferenc.cs...@pm.me.invalid> >>> wrote: >>> >>>> Hi Martijn, >>>> >>>> +1 for the proposal. >>>> >>>>> targeted for Flink 1.19 >>>> >>>> I guess you meant Flink 1.20 here. >>>> >>>> Also, I volunteer to take updating the HBase sink, feel free to assign >>>> that task to me. >>>> >>>> Best, >>>> Ferenc >>>> >>>> >>>> >>>> >>>> On Friday, May 3rd, 2024 at 10:20, Martijn Visser < >>>> martijnvis...@apache.org> wrote: >>>> >>>>> >>>>> >>>>> Hi Peter, >>>>> >>>>> I'll add it for completeness, thanks! >>>>> With regards to FLINK-35149, the fix version indicates a change at >>> Flink >>>>> CDC; is that indeed correct, or does it require a change in the >> SinkV2 >>>>> interface? >>>>> >>>>> Best regards, >>>>> >>>>> Martijn >>>>> >>>>> >>>>> On Fri, May 3, 2024 at 7:47 AM Péter Váry >> peter.vary.apa...@gmail.com >>>>> >>>>> wrote: >>>>> >>>>>> Hi Martijn, >>>>>> >>>>>> We might want to add FLIP-371 [1] to the list. (Or we aim only for >>>> higher >>>>>> level FLIPs?) >>>>>> >>>>>> We are in the process of using the new API in Iceberg connector >> [2] - >>>> so >>>>>> far, so good. >>>>>> >>>>>> I know of one minor known issue about the sink [3], which should be >>>> ready >>>>>> for the release. >>>>>> >>>>>> All-in-all, I think we are in good shape, and we could move forward >>>> with >>>>>> the promotion. >>>>>> >>>>>> Thanks, >>>>>> Peter >>>>>> >>>>>> [1] - >>>>>> >>>>>> >>>> >>> >> https://cwiki.apache.org/confluence/plugins/servlet/mobile?contentId=263430387 >>>>>> [2] - https://github.com/apache/iceberg/pull/10179 >>>>>> [3] - https://issues.apache.org/jira/browse/FLINK-35149 >>>>>> >>>>>> On Thu, May 2, 2024, 09:47 Muhammet Orazov >>>> mor+fl...@morazow.com.invalid >>>>>> wrote: >>>>>> >>>>>>> Got it, thanks! >>>>>>> >>>>>>> On 2024-05-02 06:53, Martijn Visser wrote: >>>>>>> >>>>>>>> Hi Muhammet, >>>>>>>> >>>>>>>> Thanks for joining the discussion! The changes in this FLIP >> would >>>> be >>>>>>>> targeted for Flink 1.19, since it's only a matter of changing >> the >>>>>>>> annotation. >>>>>>>> >>>>>>>> Best regards, >>>>>>>> >>>>>>>> Martijn >>>>>>>> >>>>>>>> On Thu, May 2, 2024 at 7:26 AM Muhammet Orazov >>>> mor+fl...@morazow.com >>>>>>>> wrote: >>>>>>>> >>>>>>>>> Hello Martijn, >>>>>>>>> >>>>>>>>> Thanks for the FLIP and detailed history of changes, +1. >>>>>>>>> >>>>>>>>> Would FLIP changes target for 2.0? I think it would be good >>>>>>>>> to have clear APIs on 2.0 release. >>>>>>>>> >>>>>>>>> Best, >>>>>>>>> Muhammet >>>>>>>>> >>>>>>>>> On 2024-05-01 15:30, Martijn Visser wrote: >>>>>>>>> >>>>>>>>>> Hi everyone, >>>>>>>>>> >>>>>>>>>> I would like to start a discussion on FLIP-453: Promote >>>> Unified Sink >>>>>>>>>> API V2 >>>>>>>>>> to Public and Deprecate SinkFunction >>>>>>>>>> https://cwiki.apache.org/confluence/x/rIobEg >>>>>>>>>> >>>>>>>>>> This FLIP proposes to promote the Unified Sink API V2 from >>>>>>>>>> PublicEvolving >>>>>>>>>> to Public and to mark the SinkFunction as Deprecated. >>>>>>>>>> >>>>>>>>>> I'm looking forward to your thoughts. >>>>>>>>>> >>>>>>>>>> Best regards, >>>>>>>>>> >>>>>>>>>> Martijn >>>> >>> >>