-
Notifications
You must be signed in to change notification settings - Fork 919
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
[KYUUBI #298]Add Support for Apache Spark 3.1.1 #366
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Scan Summary
Tool | Critical | High | Medium | Low | Status |
---|---|---|---|---|---|
Security Audit for Infrastructure | 0 | 0 | 0 | 0 | ✅ |
Java Source Analyzer | 0 | 0 | 0 | 1 | ✅ |
Class File Analyzer | 0 | 6 | 0 | 0 | ❌ |
Python Security Analysis | 3 | 0 | 0 | 0 | ❌ |
Python Source Analyzer | 0 | 0 | 0 | 0 | ✅ |
Scala Security Audit | 0 | 6 | 0 | 0 | ❌ |
Shell Script Analysis | 0 | 0 | 0 | 0 | ✅ |
Recommendation
Please review the findings from Code scanning alerts before approving this pull request. You can also configure the build rules or add suppressions to customize this bot 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Scan Summary
Tool | Critical | High | Medium | Low | Status |
---|---|---|---|---|---|
Dependency Scan (java) | 0 | 2 | 5 | 0 | ✅ |
Security Audit for Infrastructure | 0 | 0 | 0 | 0 | ✅ |
Java Source Analyzer | 0 | 0 | 0 | 1 | ✅ |
Class File Analyzer | 0 | 6 | 0 | 0 | ❌ |
Python Security Analysis | 3 | 0 | 0 | 0 | ❌ |
Python Source Analyzer | 0 | 0 | 0 | 0 | ✅ |
Scala Security Audit | 0 | 6 | 0 | 0 | ❌ |
Shell Script Analysis | 0 | 0 | 0 | 0 | ✅ |
Recommendation
Please review the findings from Code scanning alerts before approving this pull request. You can also configure the build rules or add suppressions to customize this bot 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Scan Summary
Tool | Critical | High | Medium | Low | Status |
---|---|---|---|---|---|
Dependency Scan (java) | 0 | 2 | 4 | 0 | ✅ |
Security Audit for Infrastructure | 0 | 0 | 0 | 0 | ✅ |
Java Source Analyzer | 0 | 0 | 0 | 1 | ✅ |
Class File Analyzer | 0 | 6 | 0 | 0 | ❌ |
Python Security Analysis | 3 | 0 | 0 | 0 | ❌ |
Python Source Analyzer | 0 | 0 | 0 | 0 | ✅ |
Scala Security Audit | 0 | 6 | 0 | 0 | ❌ |
Shell Script Analysis | 0 | 0 | 0 | 0 | ✅ |
Recommendation
Please review the findings from Code scanning alerts before approving this pull request. You can also configure the build rules or add suppressions to customize this bot 👍
Codecov Report
@@ Coverage Diff @@
## master #366 +/- ##
==========================================
+ Coverage 80.49% 80.69% +0.19%
==========================================
Files 101 99 -2
Lines 3640 3620 -20
Branches 428 426 -2
==========================================
- Hits 2930 2921 -9
+ Misses 492 479 -13
- Partials 218 220 +2
|
cc @wangyum @ulysses-you @pan3793 @Honglun @zwangsheng @xjzrc thanks for reviewing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LTGM
thanks all, merged to master for v1.1.0 |
❨?❩
Why are the changes needed?
Spark 3.1.1 released, it looks more stable than the first minor release of Spark 3.x, and a lot of great features were involved.
In this PR, I add a profile for
spark-3.1.1
support, and temporary we only supporthadoop2.7
TODOS:
fix #298
How was this patch tested?
Add some test cases that check the changes thoroughly including negative and positive cases if possible
Add screenshots for manual tests if appropriate
Run test locally before make a pull request