-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Feature][connector-v2]Support opengauss jdbc connnector using opengauss driver. #7622
Conversation
f0c21bb
to
62a9b84
Compare
.../java/org/apache/seatunnel/connectors/seatunnel/jdbc/catalog/opengauss/OpenGaussCatalog.java
Outdated
Show resolved
Hide resolved
.../seatunnel/connectors/seatunnel/jdbc/internal/dialect/opengauss/OpenGaussDialectFactory.java
Outdated
Show resolved
Hide resolved
...eatunnel/connectors/seatunnel/jdbc/internal/dialect/opengauss/OpenGaussJdbcRowConverter.java
Outdated
Show resolved
Hide resolved
...e/seatunnel/connectors/seatunnel/jdbc/internal/dialect/opengauss/OpenGaussTypeConverter.java
Outdated
Show resolved
Hide resolved
This implementation code is almost identical to jdbc-postgresql, so I suggest you just rely on jdbc-postgresql's implementation and override only the implementation of specific methods, such as those involving jdbc-url and the drivers in the pom. |
Make changes to the code as suggested. |
.../seatunnel/connectors/seatunnel/jdbc/internal/dialect/opengauss/OpenGaussDialectFactory.java
Outdated
Show resolved
Hide resolved
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 if ci passed
…uss driver. 获取最新代码 获取最新代码 获取最新代码 [Feature][connector-v2]Support opengauss jdbc connnector using opengauss driver.
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.
Thanks @luckyLJY !
@hawk9821 would you mind take a look again? |
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 . thanks @luckyLJY
…uss driver. (apache#7622) Co-authored-by: lucky_ljy <[email protected]>
Use opengauss driver to implement opengauss jdbc sink and source