Skip to content

[#1085] feat(spark): Add dynamic allocation patch for Spark 3.4 - #1225

Merged
roryqi merged 1 commit into
apache:masterfrom
summaryzb:dynamic_spark3.4
Oct 8, 2023
Merged

[#1085] feat(spark): Add dynamic allocation patch for Spark 3.4#1225
roryqi merged 1 commit into
apache:masterfrom
summaryzb:dynamic_spark3.4

Conversation

@summaryzb

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Add the dynamic allocation patch for Spark 3.4

Why are the changes needed?

#1085

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Manual test

@codecov-commenter

codecov-commenter commented Oct 8, 2023

Copy link
Copy Markdown

Codecov Report

Merging #1225 (78c9bd6) into master (05aba70) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1225   +/-   ##
=========================================
  Coverage     54.60%   54.60%           
  Complexity     2606     2606           
=========================================
  Files           372      372           
  Lines         20186    20186           
  Branches       1925     1925           
=========================================
  Hits          11023    11023           
  Misses         8502     8502           
  Partials        661      661           

📣 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 fbaa741 into apache:master Oct 8, 2023
@roryqi roryqi changed the title feat: Add dynamic allocation patch for Spark 3.4 [#1085] feat(spark): Add dynamic allocation patch for Spark 3.4 Oct 8, 2023
@summaryzb
summaryzb deleted the dynamic_spark3.4 branch October 13, 2023 01:00
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