-
Notifications
You must be signed in to change notification settings - Fork 311
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
feat(oceanbase-catalog): Support schema operations for OceanBase JDBC catalog. #5013
Conversation
@mchades PTAL. |
.../main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseDatabaseOperations.java
Outdated
Show resolved
Hide resolved
...common/src/test/java/org/apache/gravitino/integration/test/container/OceanBaseContainer.java
Show resolved
Hide resolved
...common/src/main/java/org/apache/gravitino/catalog/jdbc/operation/JdbcDatabaseOperations.java
Outdated
Show resolved
Hide resolved
.../main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseDatabaseOperations.java
Outdated
Show resolved
Hide resolved
...common/src/main/java/org/apache/gravitino/catalog/jdbc/operation/JdbcDatabaseOperations.java
Show resolved
Hide resolved
.../main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseDatabaseOperations.java
Outdated
Show resolved
Hide resolved
.../main/java/org/apache/gravitino/catalog/oceanbase/operation/OceanBaseDatabaseOperations.java
Outdated
Show resolved
Hide resolved
@mchades can you please move forward this PR? |
06941f4
to
348bdb7
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. Thanks for your contributions and sorry for my later response.
I think the next PR should be support simple table operations for Oceanbase catalog
I think it might be a better choice to implement the |
ok, plz go ahead |
What changes were proposed in this pull request?
Why are the changes needed?
Fix: #4990
Does this PR introduce any user-facing change?
no
How was this patch tested?
Add unit test: TestOceanBaseDatabaseOperations.