-
Notifications
You must be signed in to change notification settings - Fork 28.3k
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
[MINOR][INFRA] Add -Pdocker-integration-tests to GitHub Action Scala 2.13 build job #30661
Conversation
@dongjoon-hyun Thanks for the work! I planed to do this work after #30660 merged and verifying all docker-integration-tests pass. |
Oh, I see. Thanks, @sarutak . |
Kubernetes integration test starting |
Kubernetes integration test status success |
Scala 2.13 build pass on GitHub Actions. Merged to |
…2.13 build job ### What changes were proposed in this pull request? This aims to add `-Pdocker-integration-tests` at GitHub Action job for Scala 2.13 compilation. ### Why are the changes needed? We fixed Scala 2.13 compilation of this module at #30660 . This PR will prevent accidental regression at that module. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Pass GitHub Action Scala 2.13 job. Closes #30661 from dongjoon-hyun/SPARK-DOCKER-IT. Authored-by: Dongjoon Hyun <dongjoon@apache.org> Signed-off-by: Kousuke Saruta <sarutak@oss.nttdata.com> (cherry picked from commit 3a6546d) Signed-off-by: Kousuke Saruta <sarutak@oss.nttdata.com>
Thanks! |
Test build #132404 has finished for PR 30661 at commit
|
Nice, LGTM! |
What changes were proposed in this pull request?
This aims to add
-Pdocker-integration-tests
at GitHub Action job for Scala 2.13 compilation.Why are the changes needed?
We fixed Scala 2.13 compilation of this module at #30660 . This PR will prevent accidental regression at that module.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GitHub Action Scala 2.13 job.