Thanks Lucifer for do this. I did some review on this PR, the change only affects few source connector and flink translation layer. So I think it’s fine if we can make sure this PR be merged ASAP.
________________________ Jia Fan > 2023年12月2日 16:34，JUN GAO <[email protected]> 写道： > > @hailin0 <mailto:[email protected]> @Jia Fan <mailto:[email protected]> > What do you think about this? > > Lucifer Tyrant <[email protected] <mailto:[email protected]>> > 于2023年11月29日周三 22:51写道： >> Hi developers, >> >> After several major iterations in the community, the Source/Link v2 >> API has stabilized, and the community has recently been promoting the >> development of multi table read/write functionality. However, when I >> was refactoring the Flink translation layer[1], I found that the >> current implementation of the Source connector did not strictly follow >> the pull push model originally designed. Therefore, I propose to pause >> the merging of multi table read PR and adjust the implementation of >> all Source connectors before proceeding with the related >> functionality. >> >> What do you think? If you have any questions, please let me know! >> >> [1] https://github.com/apache/seatunnel/pull/5100 >> >> Chao Tian > > > -- > Best Regards > ------------ > EricJoy2048 > > [email protected] <mailto:[email protected]> >
