-
Notifications
You must be signed in to change notification settings - Fork 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
[feature] Introduce flink jdbc catalog store plugin. #3914
Conversation
098ae5d
to
c8c47c1
Compare
hi Mrart: HI, thanks for your contribution. There are a few questions to discuss:
|
...nk/streampark-flink-shims/streampark-flink-catalog-store/src/test/resources/docker/setup.sql
Outdated
Show resolved
Hide resolved
1.In the next pr, I'll provide a full catalog backend management implementation. And help coordinate the front-end resources. |
...k-shims/streampark-flink-catalog-store/src/main/java/org/apache/streampark/JacksonUtils.java
Outdated
Show resolved
Hide resolved
...eampark-flink-catalog-store/src/main/java/org/apache/streampark/JdbcCatalogStoreFactory.java
Outdated
Show resolved
Hide resolved
streampark-flink/streampark-flink-catalog-store/src/test/resources/docker/setup.sql
Outdated
Show resolved
Hide resolved
hi: |
Quality Gate passedIssues Measures |
When moving the module of the project, the load fails, delete the idea directory again and pour it in. Restores.idea/icon.png and idea/vcs.xml |
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
Contribution Checklist
What changes were proposed in this pull request
Issue Number: close #3913
Brief change log
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
jdbc factory load
*jdbc catalog crud test*
This change added tests and can be verified as follows:
Does this pull request potentially affect one of the following parts