Re: [DISCUSSION] FLIP-457: Improve Table/SQL Configuration for Flink 2.0

2024-05-23 Thread Jane Chan
t;>>>> cc @Ron Liu WDYT? > >>>>>> > >>>>>> 3. Regarding WindowEmitStrategy, IIUC it is currently unsupported > >> on > >>>> TVF > >>>>>> window, so > >>>>>> it's recommended

Re: [DISCUSSION] FLIP-457: Improve Table/SQL Configuration for Flink 2.0

2024-05-22 Thread Leonard Xu
INK-29692[2]. cc @Xuyang >>>>>> >>>>>> [1] >>>>>> >>>>> >>>> >>> >> https://github.com/ververica/flink-sql-benchmark/blob/master/tools/common/flink-conf.yaml >>>>>> [2] https://issues.apache.org/

Re: [DISCUSSION] FLIP-457: Improve Table/SQL Configuration for Flink 2.0

2024-05-22 Thread Jane Chan
:49写道: > > > > > > > > > > > Hi Jane, > > > > > > > > > > > > Thank Jane for driving this proposal ! > > > > > > > > > > > > This makes sense for users, +1 for that. > > > > > > > > > > > > Best

Re: [DISCUSSION] FLIP-457: Improve Table/SQL Configuration for Flink 2.0

2024-05-21 Thread Lincoln Lee
> > > > > > Hi Jane, > > > > > > > > > > > > Thanks for the proposal. +1 from my side. > > > > > > > > > > > > > > > > > > Best, > > > > > > Jark > > > > > > > &

Re: [DISCUSSION] FLIP-457: Improve Table/SQL Configuration for Flink 2.0

2024-05-21 Thread Jane Chan
rote: > > > > > > > > > > > Hi Jane, > > > > > > > > > > > > Thanks for driving this effort! And +1 for the proposed changes. > > > > > > > > > > > > I have one comment on the migration plan. > > >

Re: [DISCUSSION] FLIP-457: Improve Table/SQL Configuration for Flink 2.0

2024-05-19 Thread Ron Liu
r options to be moved to another module/package, I think we have > to > > > > > mark the old option deprecated in 1.20 for it to be removed in 2.0, > > > > > according to the API compatibility guarantees[1]. We can introduce > the > > > > > new optio

Re: [DISCUSSION] FLIP-457: Improve Table/SQL Configuration for Flink 2.0

2024-05-19 Thread Benchao Li
to the API compatibility guarantees[1]. We can introduce the > > > > new option in 1.20 with the same option key in the intended class. > > > > WDYT? > > > > > > > > Best, > > > > Xuannan > > > > > > &g

Re: [DISCUSSION] FLIP-457: Improve Table/SQL Configuration for Flink 2.0

2024-05-19 Thread Lincoln Lee
the intended class. > > > WDYT? > > > > > > Best, > > > Xuannan > > > > > > [1] > > > > https://nightlies.apache.org/flink/flink-docs-master/docs/ops/upgrading/#api-compatibility-guarantees > > > > > > > > >

Re: [DISCUSSION] FLIP-457: Improve Table/SQL Configuration for Flink 2.0

2024-05-16 Thread Yubin Li
> > https://nightlies.apache.org/flink/flink-docs-master/docs/ops/upgrading/#api-compatibility-guarantees > > > > > > > > On Wed, May 15, 2024 at 6:20 PM Jane Chan wrote: > > > > > > Hi all, > > > > > > I'd like to start a discussi

Re: [DISCUSSION] FLIP-457: Improve Table/SQL Configuration for Flink 2.0

2024-05-16 Thread Jark Wu
nan > > [1] > https://nightlies.apache.org/flink/flink-docs-master/docs/ops/upgrading/#api-compatibility-guarantees > > > > On Wed, May 15, 2024 at 6:20 PM Jane Chan wrote: > > > > Hi all, > > > > I'd like to start a discussion on FLIP-457: Imp

Re: [DISCUSSION] FLIP-457: Improve Table/SQL Configuration for Flink 2.0

2024-05-15 Thread Xuannan Su
gt; > I'd like to start a discussion on FLIP-457: Improve Table/SQL Configuration > for Flink 2.0 [1]. This FLIP revisited all Table/SQL configurations to > improve user-friendliness and maintainability as Flink moves toward 2.0. > > I am looking forward to your feedback. > &g

[DISCUSSION] FLIP-457: Improve Table/SQL Configuration for Flink 2.0

2024-05-15 Thread Jane Chan
Hi all, I'd like to start a discussion on FLIP-457: Improve Table/SQL Configuration for Flink 2.0 [1]. This FLIP revisited all Table/SQL configurations to improve user-friendliness and maintainability as Flink moves toward 2.0. I am looking forward to your feedback. Best regards, Jane [1