Skip to content

Conversation

@williamhyun
Copy link
Member

What changes were proposed in this pull request?

This PR aims to add a new executor roll policy, FAILED_TASKS.

Why are the changes needed?

This is helpful to clean up a faulty executor first.

Does this PR introduce any user-facing change?

Yes, but this is a new feature.

How was this patch tested?

Pass the CIs.

@SparkQA
Copy link

SparkQA commented Dec 18, 2021

Test build #146342 has finished for PR 34938 at commit 1646bf2.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM. Thank you, @williamhyun .

@SparkQA
Copy link

SparkQA commented Dec 18, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50816/

@SparkQA
Copy link

SparkQA commented Dec 18, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50816/

kazuyukitanimura pushed a commit to kazuyukitanimura/spark that referenced this pull request Aug 10, 2022
…he#1315)

### What changes were proposed in this pull request?

This PR aims to add a new executor roll policy, FAILED_TASKS.

### Why are the changes needed?

This is helpful to clean up a faulty executor first.

### Does this PR introduce _any_ user-facing change?
Yes, but this is a new feature.

### How was this patch tested?
Pass the CIs.

Closes apache#34938 from williamhyun/SPARK-37679.

Authored-by: William Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
(cherry picked from commit 897f056)
Signed-off-by: Dongjoon Hyun <[email protected]>

Co-authored-by: William Hyun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants