seatunnel icon indicating copy to clipboard operation
seatunnel copied to clipboard

[Feature][Connector-V2][JDBC] Add presto connector

Open qianmoQ opened this issue 2 years ago • 1 comments

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:
    1. Update change log that in connector document. For more details you can refer to connector-v2
    2. Update plugin-mapping.properties and add new connector information in it
    3. Update the pom file of seatunnel-dist

qianmoQ avatar Dec 20 '22 02:12 qianmoQ

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

qianmoQ avatar Dec 20 '22 02:12 qianmoQ

Hi ,please fix ci first, thanks.

Hisoka-X avatar Jan 18 '23 08:01 Hisoka-X

Hi ,please fix ci first, thanks.

please check it

qianmoQ avatar Jan 29 '23 07:01 qianmoQ

Seem like the jdbc e2e failed.

Hisoka-X avatar Jan 30 '23 02:01 Hisoka-X

Seem like the jdbc e2e failed.

Fixed, please merge

qianmoQ avatar Feb 10 '23 01:02 qianmoQ

when will Presto connector available?, i need to test for my project ,can you please help me

sanjaychitransh avatar Jan 24 '24 06:01 sanjaychitransh

@qianmoQ i tested but connection with presto is failed,please help to close the PR

sanjaychitransh avatar Mar 07 '24 10:03 sanjaychitransh

is the code merged? may i know the update? this PR is closed

sanjaychitransh avatar Mar 07 '24 12:03 sanjaychitransh