lazyfetch opened a new pull request, #11012:
URL: https://github.com/apache/seatunnel/pull/11012
### Purpose of this pull request
#10443
Add Sybase source connector
This pull request implements the Sybase JDBC source connector for SeaTunnel
Connector-V2.
It adds the necessary dialect and type mapping support to allow SeaTunnel to
read data from
Sybase ASE databases using the jTDS JDBC driver. The implementation follows
the existing
dialect pattern (e.g., SQL Server) and is scoped to source-only in this
first version.
### Does this PR introduce _any_ user-facing change?
Yes.
- **New Feature:** Added support for `Sybase` as a source within the `Jdbc`
connector.
- **Documentation:** Added `docs/en/connectors/source/Sybase.md` with type
mappings,
options, and examples.
- **Behavior:** Users can now use the `Jdbc` source connector with
`jdbc:jtds:sybase://`
URLs to read data from Sybase ASE.
### How was this patch tested?
- The dialect, type converter, row converter, and factory compile
successfully.
- An E2E integration test (`JdbcSybaseIT.java`) with full data type coverage
has been
added. However, it cannot be executed because no publicly available Sybase
Docker
image runs stably on modern kernels (the image `nguoianphu/docker-sybase`
starts but
crashes with SIGSEGV before the port becomes available). The test file is
included as
documentation and for future validation if a suitable image becomes
available.
- Code style follows the SeaTunnel conventions; compilation was verified
with:
`./mvnw compile -pl seatunnel-connectors-v2/connector-jdbc -DskipTests
-Dspotless.check.skip=true`
### Check list
* [ ] If any new Jar binary package adding in your PR, please add License
Notice according
to the New License Guide. (No new JARs are added.)
* [x] If necessary, please update the documentation to describe the new
feature.
* [ ] If necessary, please update `incompatible-changes.md` to describe the
incompatibility
caused by this PR. (No incompatible changes.)
* [x] If you are contributing the connector code, please check that the
following files are
updated:
1. `plugin-mapping.properties`: **No change needed.** Sybase is a dialect
within the
existing `connector-jdbc` module, not a separate connector.
2. `seatunnel-dist/pom.xml`: **No change needed.** Sybase is part of
`connector-jdbc`.
3. CI label in `label-scope-conf.yml`: **Already covered** by the existing
`jdbc` label.
4. E2E test case `JdbcSybaseIT.java` added.
5. `config/plugin_config`: **No change needed.** No new connector jar is
introduced.
--
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]
