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

[#3319] fix(PyClient): ./gradlew test -PskipITs failed in python-client #3338

Conversation

jerryshao
Copy link
Collaborator

What changes were proposed in this pull request?

  1. Add skipITs build params in the Python Client Gradle script.
  2. Use different directory storage integration test and unit tests.

Why are the changes needed?

When running this command ./gradlew test -PskipITs directly. the Gradle python test task will be failed because of lacking compileDistribution step. we should fix it to make all the commands work.

Fix: #3319

Does this PR introduce any user-facing change?

N/A

How was this patch tested?

CI Passed

…nt (#3322)

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

1. Add `skipITs` build params in the Python Client Gradle script.
2. Use different directory storage integration test and unit tests.

### Why are the changes needed?

When running this command ./gradlew test -PskipITs directly. the Gradle
python test task will be failed because of lacking compileDistribution
step. we should fix it to make all the commands work.

Fix: #3319

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

N/A

### How was this patch tested?

CI Passed
@jerryshao jerryshao merged commit 5626a1b into branch-0.5 May 13, 2024
22 checks passed
@jerryshao jerryshao deleted the cherry-pick-branch-0.5-224d6a5069fe0b988e8a51963af9bfd9b13a77bf branch May 13, 2024 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants