-
Notifications
You must be signed in to change notification settings - Fork 378
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
[Improvement] Add a framework to support multiple JDBC storage backend. #4832
Labels
Comments
jerryshao
pushed a commit
that referenced
this issue
Sep 5, 2024
…end. (#4833) ### What changes were proposed in this pull request? 1. Add a framework to support multiple JDBC backend 2. Modify all the mapper classes related to metalake, catalog, schema, filesetset, table, topic and use new framework, others will use another PR to avoid a large PR. ### Why are the changes needed? To support more JDBC storage backend. Fix: #4832 ### Does this PR introduce _any_ user-facing change? N/A ### How was this patch tested? Existing tests.
jerryshao
pushed a commit
that referenced
this issue
Sep 6, 2024
…o use the new framework to support multiple JDBC backend. (#4869) ### What changes were proposed in this pull request? Change tag, user, and role-related code to support the new framework introduced by #4832 ### Why are the changes needed? To support multiple JDBC backend. Fix: #4868 ### Does this PR introduce _any_ user-facing change? N/A. ### How was this patch tested? Existing UTs and ITs.
yuqi1129
added a commit
to yuqi1129/gravitino
that referenced
this issue
Sep 24, 2024
…C backend. (apache#4833) ### What changes were proposed in this pull request? 1. Add a framework to support multiple JDBC backend 2. Modify all the mapper classes related to metalake, catalog, schema, filesetset, table, topic and use new framework, others will use another PR to avoid a large PR. ### Why are the changes needed? To support more JDBC storage backend. Fix: apache#4832 ### Does this PR introduce _any_ user-facing change? N/A ### How was this patch tested? Existing tests.
jerqi
pushed a commit
to qqqttt123/gravitino
that referenced
this issue
Sep 24, 2024
…lass to use the new framework to support multiple JDBC backend. (apache#4869) ### What changes were proposed in this pull request? Change tag, user, and role-related code to support the new framework introduced by apache#4832 ### Why are the changes needed? To support multiple JDBC backend. Fix: apache#4868 ### Does this PR introduce _any_ user-facing change? N/A. ### How was this patch tested? Existing UTs and ITs.
diqiu50
pushed a commit
that referenced
this issue
Sep 24, 2024
…end (#4998) ### What changes were proposed in this pull request? 1. Add a framework to support multiple JDBC backend 2. Modify all the mapper classes related to metalake, catalog, schema, filesetset, table, topic and use new framework, others will use another PR to avoid a large PR. ### Why are the changes needed? To support more JDBC storage backend. Fix: #4832 ### Does this PR introduce _any_ user-facing change? N/A ### How was this patch tested? Existing tests.
jerryshao
pushed a commit
that referenced
this issue
Sep 24, 2024
…o use the new framework to support multiple JDBC backend. (#5001) ### What changes were proposed in this pull request? Change tag, user, and role-related code to support the new framework introduced by #4832 ### Why are the changes needed? To support multiple JDBC backend. Fix: #4868 ### Does this PR introduce _any_ user-facing change? N/A. ### How was this patch tested? Existing UTs and ITs. Co-authored-by: Qi Yu <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
What would you like to be improved?
Currently, we use hard code to support MySQL and H2 backend, all SQLs in mappers are constant, and it's hard to support other JDBC databases, so this PR is to add a framework so that Gravitino can use multiple JDBC database as backend.
How should we improve?
No response
The text was updated successfully, but these errors were encountered: