seatunnel
seatunnel copied to clipboard
[Feature][Connector-V2][JDBC] Add presto connector
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
- [ ] If you are contributing the connector code, please check that the following files are updated:
- Update change log that in connector document. For more details you can refer to connector-v2
- Update plugin-mapping.properties and add new connector information in it
- Update the pom file of seatunnel-dist
Should class names use
Trino
as a prefix?
No need, the core is based on the implementation of io.presto
, only to do the jdbc conversion of io.trino
Hi ,please fix ci first, thanks.
Hi ,please fix ci first, thanks.
please check it
Seem like the jdbc e2e failed.
Seem like the jdbc e2e failed.
Fixed, please merge
when will Presto connector available?, i need to test for my project ,can you please help me
@qianmoQ i tested but connection with presto is failed,please help to close the PR
is the code merged? may i know the update? this PR is closed