Thanks Leonard for driving this, and thanks everyone for the discussion. The back-and-force reflects the importance and complexity around this topic. Glad to see we finally reached consensus.
Best, Xintong On Wed, Jul 26, 2023 at 12:42 AM Jing Ge <j...@ververica.com> wrote: > Thanks Leonard for driving it. We are now on the same page. > > Best regards, > Jing > > On Tue, Jul 25, 2023 at 9:19 PM Leonard Xu <xbjt...@gmail.com> wrote: > >> We’ve detailed offline discussions with @Alexander and @Jingsong, about >> “Remove SourceFunction” item, we’ve reached a consensus as following: >> >> 1. Deprecate SourceFunction in 1.18 and implement following improvement >> subtasks of FLINK-28045[1] later is reasonable for all of us. >> >> 2. Deleting SourceFunction API depends on future’s work progress, thus >> “Remove SourceFunction APIs” should be a nice to have item. Alexander has >> volunteered to take these subtasks and would try to finish them next, >> thanks again. >> >> 3. As a nice to have item, and its READY status depends on future’s work >> progress, this won't block release 2.0 must-have item vote. >> >> Thanks again @Alexander, @Jingsong and @Xintong for driving these things >> forward. >> >> Also CC RMs for 1.18 @QingSheng @Jing @Martijn @Konstantin, I’ve >> communicated with Alexander and would like to help review the deprecation >> PR again. >> >> Best, >> Leonard >> >> [1] https://issues.apache.org/jira/browse/FLINK-28045 >> >> >> On Jul 21, 2023, at 6:09 PM, Chesnay Schepler <ches...@apache.org> wrote: >> >> On 21/07/2023 11:45, Leonard Xu wrote: >> >> In this way, the user will see the deprecated API firstly but they can >> not find a candidate if we can not finish all tasks in one minor version . >> >> >> i'm not convinced that this matters. There will be a whole bunch of APIs >> deprecated in 1.18 (that will remain in 1.x!) without a replacement so we >> can remove them in 2.0. >> We already accepted this scenario. >> >> >>