-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add an example JDBC connector plugin #15004
Conversation
b89a8ce
to
80da0fb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM % removal from product and let's use h2 instead of psql
plugin/trino-example-jdbc/src/test/java/io/trino/plugin/example/TestExampleQueries.java
Outdated
Show resolved
Hide resolved
@@ -0,0 +1,72 @@ | |||
====================== |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mosabua Can you please review docs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mosabua can you review the docs here or suggest other reviewers? It's ready to go otherwise.
plugin/trino-example-jdbc/src/test/java/io/trino/plugin/example/ExampleQueryRunner.java
Outdated
Show resolved
Hide resolved
plugin/trino-example-jdbc/src/test/java/io/trino/plugin/example/ExampleQueryRunner.java
Show resolved
Hide resolved
55be220
to
a103eda
Compare
a103eda
to
22d750a
Compare
Merged, thanks! |
Description
Add an example JDBC plugin and a new section in the docs. My main intention is to have a docs section where we could start capturing nuances of correct type mapping (rounding, validation, etc.).
This is not related to #2910
Non-technical explanation
An example JDBC plugin can be used as an example when creating a new third-party JDBC connector.
Release notes
(x) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text: