EMsnap commented on PR #9021: URL: https://github.com/apache/inlong/pull/9021#issuecomment-1754208655
> > Please add a description on what problem the pr solves, btw why add a class that's not used ? > > The mysql test container existed before this pr and there are some unneed codes, so I remove them. > > As for MSSQL test container, it can help others write sqlserver test container for sqlserver Connector in flink 1.13. Just like the Mysql test container in flink 1.13 end to end test which are not used. > > These container are tested in flink 1.15 end to end test. Thanks for the contribution, but we only submit code that's needed for a feature or so. Maybe you could submit this class when implement sqlserver for flink1.13 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@inlong.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
