Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Improvement] Reorganize the SQL provider package to make it more clear #5008

Closed
jerryshao opened this issue Sep 24, 2024 · 0 comments · Fixed by #5009
Closed

[Improvement] Reorganize the SQL provider package to make it more clear #5008

jerryshao opened this issue Sep 24, 2024 · 0 comments · Fixed by #5009
Assignees
@jerryshao
Labels
0.6.1 Release v0.6.1 0.7.0 Release v0.7.0 improvement Improvements on everything

Comments

@jerryshao
Copy link
Contributor

What would you like to be improved?

Reorganize the SQL provider package to separate into different SQL implementations to make the package more clear.

How should we improve?

No response
@jerryshao jerryshao added the improvement Improvements on everything label Sep 24, 2024
@jerryshao jerryshao self-assigned this Sep 24, 2024
@jerryshao jerryshao mentioned this issue Sep 24, 2024
Merged
@jerryshao jerryshao added 0.6.1 Release v0.6.1 0.7.0 Release v0.7.0 labels Sep 25, 2024
jerryshao added a commit to jerryshao/gravitino that referenced this issue Sep 25, 2024
@jerryshao
[apache#5008] improve(core): reorganize the sql provider package to m…
3ad85c1 
…ake it more clear (apache#5009)

This PR adjust and add some package for SQL provider to make the code
more separately and clear.

Make the code more clear.

Fix: apache#5008

No.

Exising tests.
@jerryshao jerryshao mentioned this issue Sep 25, 2024
Merged
yuqi1129 pushed a commit that referenced this issue Sep 25, 2024
@jerryshao
[#5008] improve(core): reorganize the sql provider package to make it…
eb24835 
… more clear (#5015)

### What changes were proposed in this pull request?

This PR adjusts and adds some packages for SQL provider to make the code
more separately and clear.

### Why are the changes needed?

Make the code more clear.

Fix: #5008

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Existing tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jerryshao jerryshao

Labels
0.6.1 Release v0.6.1 0.7.0 Release v0.7.0 improvement Improvements on everything
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#5008] improve(core): reorganize the sql provider package to make it more clear jerryshao/gravitino
1 participant
@jerryshao