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] Add e2e test for JDBC entity backend #2678

Closed
yuqi1129 opened this issue Mar 26, 2024 · 0 comments · Fixed by #2686
Closed

[Improvement] Add e2e test for JDBC entity backend #2678

yuqi1129 opened this issue Mar 26, 2024 · 0 comments · Fixed by #2686
Assignees
@yuqi1129
Labels
improvement Improvements on everything
Milestone
Gravitino 0.5.0

Comments

@yuqi1129
Copy link
Contributor

What would you like to be improved?

Currently, all integration tests are running using the RocksDB backend. We need to include the JDBC backend in it.

How should we improve?

No response
@yuqi1129 yuqi1129 added the improvement Improvements on everything label Mar 26, 2024
@yuqi1129 yuqi1129 self-assigned this Mar 26, 2024
@yuqi1129 yuqi1129 mentioned this issue Mar 26, 2024
Merged
@jerryshao jerryshao added this to the Gravitino 0.5.0 milestone Mar 26, 2024
jerryshao pushed a commit that referenced this issue Apr 2, 2024
@yuqi1129
[#2678] improvement(test): Add e2e test for jdbc backend (#2686)
2bfefb9 
### What changes were proposed in this pull request?

Add JDBC entity store in integration test. 

### Why are the changes needed?

To test whether JDBC backends works.

Fix: #2678 

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

N/A.

### How was this patch tested?

Existing ITs
yuqi1129 added a commit to yuqi1129/gravitino that referenced this issue Apr 2, 2024
@yuqi1129
Revert "[apache#2678] improvement(test): Add e2e test for jdbc backend (
056e7c8 
apache#2686)"

This reverts commit 2bfefb9.
yuqi1129 added a commit to yuqi1129/gravitino that referenced this issue Apr 3, 2024
@yuqi1129
Revert "[apache#2678] improvement(test): Add e2e test for jdbc backend (
d93ce3d 
apache#2686)"

This reverts commit 2bfefb9.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yuqi1129 yuqi1129

Labels
improvement Improvements on everything
Projects
None yet
Milestone
Gravitino 0.5.0
Development

Successfully merging a pull request may close this issue.

[#2678] improvement(test): Add e2e test for jdbc backend yuqi1129/gravitino
2 participants
@jerryshao @yuqi1129