seatunnel
seatunnel copied to clipboard
[Bug][seatunnel-connector-spark-tidb] Spark tidb Connector Hadoop dependency excluded
Purpose of this pull request
Check list
- [ ] Code changed are covered with tests, or it does not need tests for reason:
- [ ] If any new Jar binary package adding in your PR, please add License Notice according New License Guide
- [ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
#2698
@hailin0 please help to review it
Can you add e2e-testcase(spark)?
reference https://github.com/apache/incubator-seatunnel/tree/dev/seatunnel-e2e/seatunnel-spark-e2e
@hailin0 I didn't find anything about TiDB in E2E, do I need to add a module for this? If so, I can try, and how can this PR review be approved to begin
Hi, Please fix license check and codestyle. Thanks!
hi, thanks for your contribution, dev branch will no longer maintain the V1 version of the connector, so I will close this PR, If you are interested in this, you can contribute the corresponding V2Connector https://github.com/apache/incubator-seatunnel/tree/dev/seatunnel-connectors-v2