Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-10-31 Thread Jing Ge
so needs to happen there) > > > > > > e. It's up to the maintainers for DynamoDB and Opensearch to > decide > > > > this > > > > > > themselves, given that I believe they've only been introduced > recently > > > > > and > > > > >

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-10-30 Thread Yun Gao
> > asap. > > > > > > > > > > Best regards, > > > > > > > > > > Martijn > > > > > > > > > > [1] https://issues.apache.org/jira/browse/FLINK-25726 > > > > > > > > > > > > > &g

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-10-30 Thread Martijn Visser
; > > > > transactions, there is no consensus on solving that problem in > > SinkV2. > > > > Will > > > > > it require any breaking change in SinkV2? > > > > > > > > > > Just want to reiterate my earlier question. Wh

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-09-28 Thread Tzu-Li (Gordon) Tai
> > > deprecating SinkFunction at this point. Dong Lin has raised the > > blocker > > > > > issues of migration multiple times in this thread which I think we > > > should > > > > > address first. I don't know why we rush to deprecate Si

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-13 Thread Jing Ge
bility > > > of > > > > Flink connector APIs. > > > > > > > > We are hoping any issues with the APIs for Iceberg connector will > > surface > > > > > sooner and get more attention from the Flink community when the > > > conne

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-11 Thread Dong Lin
rather than in Iceberg repo. > > > > > > > > > The connector externalizing is a big step for building a mechanism to > > > guarantee Flink connector API is stable and workable. The follow-up > step > > > should be trying the new APIs in externalized connectors and g

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-11 Thread Dong Lin
t; > Iceberg community raised concerns[1] about the workability and > > stability > > > of > > > > Flink connector APIs. > > > > > > > > We are hoping any issues with the APIs for Iceberg connector will > > surface > > > > > sooner

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-11 Thread Galen Warren
to > > > guarantee Flink connector API is stable and workable. The follow-up > step > > > should be trying the new APIs in externalized connectors and giving > users > > > the confidence to migrate. > > > > > > > > > Best, > > >

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-10 Thread Martijn Visser
migrate. > > > > > > Best, > > Jark > > > > [1]: https://lists.apache.org/thread/r5zqnkt01x2c611brkjmxxnt3bfcgl1b > > > > On Tue, 7 Feb 2023 at 09:53, yuxia wrote: > > > > > Hi Konstantin, > > > Just FYI, the FileSy

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-06 Thread Steven Wu
xia > > > > ----- 原始邮件 - > > 发件人: "Dong Lin" > > 收件人: "dev" > > 抄送: "Jing Ge" , "Yun Tang" > > 发送时间: 星期二, 2023年 2 月 07日 上午 9:41:07 > > 主题: Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction &

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-06 Thread Jark Wu
> > Best regards, > Yuxia > > - 原始邮件 - > 发件人: "Dong Lin" > 收件人: "dev" > 抄送: "Jing Ge" , "Yun Tang" > 发送时间: 星期二, 2023年 2 月 07日 上午 9:41:07 > 主题: Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction > > Hi

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-06 Thread yuxia
Hi Konstantin, Just FYI, the FileSystemTableSink are still using SinkFunction. Best regards, Yuxia - 原始邮件 - 发件人: "Dong Lin" 收件人: "dev" 抄送: "Jing Ge" , "Yun Tang" 发送时间: 星期二, 2023年 2 月 07日 上午 9:41:07 主题: Re: [DISCUSS] Promote SinkV2 to @Public a

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-06 Thread Dong Lin
Hi Konstantin, Thanks for the reply. Please see my comments inline. On Mon, Feb 6, 2023 at 9:48 PM Konstantin Knauf wrote: > Hi Steven, > > Sink is already deprecated. It was deprecated at the moment where we > introduced SinkV2. > > Hi Jark, Hi Dong, > > My understanding is the SinkV2 is a wor

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-06 Thread Jing Ge
Hi, We have the same goal to let users migrate to SinkV2. Just marking those interfaces as deprecated or public won't achieve this goal. Frankly speaking, users will start to migrate not because those interfaces are marked as deprecated. They will do it because there is a workable implementation w

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-06 Thread Galen Warren
Recently, a critical bug with the Unified Sink committer was reported: [FLINK-30238] Unified Sink committer does not clean up state on final savepoint - ASF JIRA (apache.org) . Fabian Paul reported: Hi folks, > > I did some initial investigation

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-06 Thread Konstantin Knauf
Hi Steven, Sink is already deprecated. It was deprecated at the moment where we introduced SinkV2. Hi Jark, Hi Dong, My understanding is the SinkV2 is a workable interface. The most important connectors have been migrated (Kafka, Filesystem) and more connectors (OpenSearch, ElasticSearch, Kinesi

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-06 Thread Jark Wu
I agree with Dong Lin. Oracle explains how to use Deprecate API [1]: You are strongly recommended to use the Javadoc @deprecated tag with > appropriate comments explaining how to use the new API. This ensures > developers will *have a workable migration path from the old API to the > new API*.

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-05 Thread Steven Wu
Regarding the discussion on global committer [1] for sinks with global transactions, there is no consensus on solving that problem in SinkV2. Will it require any breaking change in SinkV2? Also will SinkV1 be deprecated too? or it should happen sometime after SinkFunction deprecation? [1] https:/

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-05 Thread Dong Lin
Hi Konstantin, Thanks for the comment! Please see my comment inline. Cheers, Dong On Sat, Feb 4, 2023 at 2:06 AM Konstantin Knauf wrote: > Hi everyone, > > sorry for joining the discussion late. > > 1) Is there an option to deprecate SinkFunction in Flink 1.17 while leaving > SinkV2 @PublicEvo

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-02-03 Thread Konstantin Knauf
Hi everyone, sorry for joining the discussion late. 1) Is there an option to deprecate SinkFunction in Flink 1.17 while leaving SinkV2 @PublicEvolving in Flink 1.17. We then aim to make SinkV2 @Public in and remove SinkFunction in Flink 1.18. @PublicEvolving are intended for public use. So, I don

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-01-30 Thread Dong Lin
Hi Martijn, Thanks for driving this effort to clean-up the Flink codebase! I like the idea to cleanup Flink codebase to avoid having two Sinks. On the other hand, I also thing the concern mentioned by Jing makes sense. In addition to thinking in terms of the rule proposed in FLIP-197

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-01-28 Thread yuxia
, Yuxia - 原始邮件 - 发件人: "Jing Ge" 收件人: "dev" 抄送: "Yun Gao" , "Qingsheng Ren" , "Jark Wu" 发送时间: 星期二, 2023年 1 月 24日 下午 7:25:59 主题: Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction Hi Martijn, Thanks for sharing more info

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-01-24 Thread Jing Ge
t; > On Thu, Jan 19, 2023 at 3:26 AM Lijie Wang > > > wrote: > > > > > >> Hi Martijn, > > >> > > >> Thanks for driving this. I have a only concern about the > > Sink.InitContext. > > >> > > >> Does the Sink.InitContext will also be changed to @Public ? As

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-01-19 Thread Martijn Visser
sinks. If it is marked > >> as > >> public/stable, we can no longer modify it in the future(since most > >> connectors are not migrated to SinkV2 currently, we may find we need > more > >> information via InitContext in the future migrations). > >> > >&g

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-01-18 Thread Jing Ge
lready existed with two releases. >> > And I didn't find a FLIP to discuss the process to deprecate APIs, >> > considering the SinkFunction has actually been stale for some time, I >> think >> > we can deprecate it with the @Public SinkV2. >> > >>

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-01-18 Thread Jing Ge
nkV2. > > > > Thus, +1 (binding) for this proposal. > > > > [1] https://issues.apache.org/jira/browse/FLINK-2 > > > > Best > > Yun Tang > > > > ____________ > > From: Martijn Visser > > Sent: Wednesda

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-01-18 Thread Biao Liu
____ > > From: Martijn Visser > > Sent: Wednesday, January 18, 2023 18:50 > > To: dev ; Jing Ge ; Yun Tang < > > myas...@live.com> > > Subject: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction > > > > Hi all, > > > > Whi

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-01-18 Thread Lijie Wang
y 18, 2023 18:50 > To: dev ; Jing Ge ; Yun Tang < > myas...@live.com> > Subject: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction > > Hi all, > > While discussing FLIP-281 [1] the discussion also turned to the > SinkFunction and the SinkV2 API. For a broader d

Re: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-01-18 Thread Yun Tang
ng Ge ; Yun Tang Subject: [DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction Hi all, While discussing FLIP-281 [1] the discussion also turned to the SinkFunction and the SinkV2 API. For a broader discussion I'm opening up a separate discussion thread. As Yun Tang has mention

[DISCUSS] Promote SinkV2 to @Public and deprecate SinkFunction

2023-01-18 Thread Martijn Visser
Hi all, While discussing FLIP-281 [1] the discussion also turned to the SinkFunction and the SinkV2 API. For a broader discussion I'm opening up a separate discussion thread. As Yun Tang has mentioned in that discussion thread, it would be a good time to deprecate the SinkFunction to avoid the ne