-
Notifications
You must be signed in to change notification settings - Fork 300
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
Spark2 CI #435
Conversation
Codecov Report
@@ Coverage Diff @@
## master #435 +/- ##
=======================================
Coverage 84.18% 84.18%
=======================================
Files 338 338
Lines 25486 25486
Branches 2093 2093
=======================================
Hits 21455 21455
Misses 3458 3458
Partials 573 573 Continue to review full report at Codecov.
|
@wild-endeavor PTAL, thanks. |
Signed-off-by: Hongxin Liang <[email protected]>
Signed-off-by: Hongxin Liang <[email protected]>
Sorry I rebased from main branch and need a new approval. @wild-endeavor |
* CI for spark2 Signed-off-by: Hongxin Liang <[email protected]> * Simplify Makefile Signed-off-by: Hongxin Liang <[email protected]> Signed-off-by: Max Hoffman <[email protected]>
* CI for spark2 Signed-off-by: Hongxin Liang <[email protected]> * Simplify Makefile Signed-off-by: Hongxin Liang <[email protected]>
TL;DR
alll-spark3
is the same asall
for some time, so we are not testing spark2 in CI. This PR adds it.Type
Are all requirements met?
Complete description
Remove
all-spark3
and testall-spark2.4
.Tracking Issue
NA
Follow-up issue
NA