xxsc0529 commented on PR #7311:
URL: https://github.com/apache/seatunnel/pull/7311#issuecomment-2268580907
> ### Purpose of this pull request
> The original oceanbase catalog directly inherits the mysql catalog, which
results in the need to add an additional mysql driver to the lib directory when
using oceanbase alone. This submission solves this problem.
>
> ### Does this PR introduce _any_ user-facing change?
> yes，colse #7289
>
> ### How was this patch tested?
> Here is a simple test example env { job.mode = "BATCH" }
>
> # 配置数据源
> source { jdbc { url = "jdbc:oceanbase://localhost:2881/demo" driver =
"com.oceanbase.jdbc.Driver" user = "xxxx@tt1" password = "123456"
generate_sink_sql = true compatible_mode="mysql" database = "demo" table =
"t_ds_user" query = "select * from t_ds_user" } }
>
> transform { }
>
> # 配置目标库
> sink { jdbc { url = "jdbc:oceanbase://localhost:2881/test" driver =
"com.oceanbase.jdbc.Driver" user = "xxxx@tt1" password = "123456"
compatible_mode="mysql" generate_sink_sql = true database = "test" table =
"t_ds_user" } Before the code was modified, because the oceanbase catalog
inherited the mysql catalog, some classes could not be found when the mysql
dependency package was not introduced, such as
It is also impossible to automatically create a table After making the
modification, the table can be created successfully without error.
>
> ### Check list
> * [ ] If any new Jar binary package adding in your PR, please add License
Notice according
> [New License
Guide](https://github.com/apache/seatunnel/blob/dev/docs/en/contribution/new-license.md)
> * [ ] If necessary, please update the documentation to describe the new
feature. https://github.com/apache/seatunnel/tree/dev/docs
> * [ ] If you are contributing the connector code, please check that the
following files are updated:
>
> 1. Update change log that in connector document. For more details you
can refer to
[connector-v2](https://github.com/apache/seatunnel/tree/dev/docs/en/connector-v2)
> 2. Update
[plugin-mapping.properties](https://github.com/apache/seatunnel/blob/dev/plugin-mapping.properties)
and add new connector information in it
> 3. Update the pom file of
[seatunnel-dist](https://github.com/apache/seatunnel/blob/dev/seatunnel-dist/pom.xml)
> * [ ] Update the
[`release-note`](https://github.com/apache/seatunnel/blob/dev/release-note.md).
--
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]
