Skip to content

[#1089] feat(spark): Add dynamic allocation patch for Spark 2.3 - #1242

Merged
roryqi merged 1 commit into
apache:masterfrom
summaryzb:dynamic_23
Oct 17, 2023
Merged

[#1089] feat(spark): Add dynamic allocation patch for Spark 2.3#1242
roryqi merged 1 commit into
apache:masterfrom
summaryzb:dynamic_23

Conversation

@summaryzb

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Add the dynamic allocation patch for Spark 2.3

Why are the changes needed?

#1089

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Manual local test

@codecov-commenter

codecov-commenter commented Oct 17, 2023

Copy link
Copy Markdown

Codecov Report

Merging #1242 (b44bb2c) into master (a83583e) will increase coverage by 1.11%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1242      +/-   ##
============================================
+ Coverage     53.68%   54.80%   +1.11%     
- Complexity     2613     2615       +2     
============================================
  Files           394      374      -20     
  Lines         22699    20339    -2360     
  Branches       1937     1937              
============================================
- Hits          12187    11146    -1041     
+ Misses         9768     8521    -1247     
+ Partials        744      672      -72     

see 22 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 30332a4 into apache:master Oct 17, 2023
@summaryzb
summaryzb deleted the dynamic_23 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