Skip to content

feat(spark): Support Spark 3.5 - #1223

Merged
roryqi merged 2 commits into
apache:masterfrom
summaryzb:support_spark3.5
Oct 4, 2023
Merged

feat(spark): Support Spark 3.5#1223
roryqi merged 2 commits into
apache:masterfrom
summaryzb:support_spark3.5

Conversation

@summaryzb

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

  1. Add maven profile spark3.5
  2. Enable tests for Spark 3.5 in CI

Why are the changes needed?

#1083

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass Test

@codecov-commenter

codecov-commenter commented Sep 28, 2023

Copy link
Copy Markdown

Codecov Report

Merging #1223 (d170a76) into master (5ac76e8) will increase coverage by 1.20%.
Report is 30 commits behind head on master.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1223      +/-   ##
============================================
+ Coverage     53.69%   54.90%   +1.20%     
- Complexity     2575     2606      +31     
============================================
  Files           391      372      -19     
  Lines         22320    20147    -2173     
  Branches       1867     1889      +22     
============================================
- Hits          11984    11061     -923     
+ Misses         9627     8448    -1179     
+ Partials        709      638      -71     

see 63 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@roryqi roryqi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @summaryzb

@roryqi
roryqi merged commit caf7104 into apache:master Oct 4, 2023
@summaryzb
summaryzb deleted the support_spark3.5 branch March 14, 2025 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants