Re: [DISCUSS] Merge StreamTableEnvironment.to/fromChangelogStream to release-1.13

2021-04-21 Thread Arvid Heise
+1 from my side. The transition between DataStream and Table should be as smooth as possible. On Wed, Apr 21, 2021 at 5:00 PM Timo Walther wrote: > Thanks for the feedback. > > @Dawid: It should be merged by tomorrow. It has been reviewed already. I > will just wait a few hours to let this discu

Re: [DISCUSS] Merge StreamTableEnvironment.to/fromChangelogStream to release-1.13

2021-04-21 Thread Timo Walther
Thanks for the feedback. @Dawid: It should be merged by tomorrow. It has been reviewed already. I will just wait a few hours to let this discussion settle. Regards, Timo On 21.04.21 14:14, Jark Wu wrote: I also think this is an attractive feature which exposes Flink's CDC engine to wider use

Re: [DISCUSS] Merge StreamTableEnvironment.to/fromChangelogStream to release-1.13

2021-04-21 Thread Jark Wu
I also think this is an attractive feature which exposes Flink's CDC engine to wider users and wider usage scenarios. I'm also fine with merging it to release 1.13. Best, Jark On Wed, 21 Apr 2021 at 17:00, Dawid Wysakowicz wrote: > Hi Timo, > > First of all, thanks for giving a good example of

Re: [DISCUSS] Merge StreamTableEnvironment.to/fromChangelogStream to release-1.13

2021-04-21 Thread Dawid Wysakowicz
Hi Timo, First of all, thanks for giving a good example of following the community agreements ;) Do you have an ETA for when the feature can be merged? Given that as you are saying, it mostly about adding a new API that would be marked as experimental and is rather well isolated from the rest of

[DISCUSS] Merge StreamTableEnvironment.to/fromChangelogStream to release-1.13

2021-04-21 Thread Timo Walther
Hi everyone, sorry for being so late with this request, but fixing a couple of down stream bugs had higher priority than this issue and were also blocking it. Nevertheless, I would like to ask for permission to merge the FLINK-19980[1] to the 1.13 branch as an experimental feature to add the