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.

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

[Subtask] Integration test with Gravitino Python client #2756

Closed
xunliu opened this issue Apr 1, 2024 · 0 comments · Fixed by #2797
Closed

[Subtask] Integration test with Gravitino Python client #2756

xunliu opened this issue Apr 1, 2024 · 0 comments · Fixed by #2797
Assignees
Labels
subtask Subtasks of umbrella issue

Comments

@xunliu
Copy link
Member

xunliu commented Apr 1, 2024

Describe the subtask

Currently, Gravitino Python client use mock date to test, We consider use real Gravitino Server to integration test it.

Parent issue

#2113

@xunliu xunliu added the subtask Subtasks of umbrella issue label Apr 1, 2024
@xunliu xunliu added this to the Gravitino 0.5.0 milestone Apr 1, 2024
@xunliu xunliu self-assigned this Apr 1, 2024
xunliu added a commit to xunliu/gravitino that referenced this issue Apr 4, 2024
xunliu added a commit to xunliu/gravitino that referenced this issue Apr 7, 2024
xunliu added a commit that referenced this issue Apr 7, 2024
…#2797)

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

Add two Gradle command in the client-python module
1. `./gradlew :clients:client-python:test`
2. `./gradlew :clients:client-python: integrationTest`

### Why are the changes needed?

Proved a real test environment for Gravitino server. 

Fix: #2756

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

N/A

### How was this patch tested?

CI Passed
unknowntpo pushed a commit to unknowntpo/gravitino that referenced this issue Apr 9, 2024
…client (apache#2797)

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

Add two Gradle command in the client-python module
1. `./gradlew :clients:client-python:test`
2. `./gradlew :clients:client-python: integrationTest`

### Why are the changes needed?

Proved a real test environment for Gravitino server. 

Fix: apache#2756

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

N/A

### How was this patch tested?

CI Passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
subtask Subtasks of umbrella issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant