EricJoy2048 commented on PR #2083: URL: https://github.com/apache/incubator-seatunnel/pull/2083#issuecomment-1172087028
> LGTM. I find we have moved the new connector to seatunnel-connectors-v2, this is great, so it's better to change all the `new-connector` to `connector-v2`. I find in e2e, the module name is still `new-connector`, we can continue to optimize this after this PR merged. Yes, We should unify the names of each module of the new connector. consist of update update `seatunnel-flink-new-connector-e2e` to `seatunnel-flink-v2-e2e` , update `seatunnel-spark-new-connector-e2e` to `seatunnel-spark-v2-e2e`, update `docs/en/new-connector` to `docs/en/connector-v2`. I will submit a pr to fix this. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
