Skip to content

[#1088] feat(spark): Add dynamic allocation patch for Spark 3.0 - #1241

Merged
roryqi merged 1 commit into
apache:masterfrom
summaryzb:dynamic_301
Oct 16, 2023
Merged

[#1088] feat(spark): Add dynamic allocation patch for Spark 3.0#1241
roryqi merged 1 commit into
apache:masterfrom
summaryzb:dynamic_301

Conversation

@summaryzb

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Add the dynamic allocation patch for Spark 3.0

Why are the changes needed?

#1088

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Manual test spark

@codecov-commenter

codecov-commenter commented Oct 16, 2023

Copy link
Copy Markdown

Codecov Report

Merging #1241 (7ebd0de) into master (03358b4) will increase coverage by 0.93%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1241      +/-   ##
============================================
+ Coverage     53.85%   54.79%   +0.93%     
+ Complexity     2628     2614      -14     
============================================
  Files           394      374      -20     
  Lines         22699    20339    -2360     
  Branches       1937     1937              
============================================
- Hits          12225    11145    -1080     
+ Misses         9729     8521    -1208     
+ Partials        745      673      -72     

see 23 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 a83583e into apache:master Oct 16, 2023
@summaryzb
summaryzb deleted the dynamic_301 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