Skip to content

Conversation

@roxchkplusony
Copy link
Contributor

v1.1 backport for #3483

@roxchkplusony roxchkplusony changed the title [SPARK-4626] Kill a task only if the executorId is (still) registered wi... [BRANCH-1.1][SPARK-4626] Kill a task only if the executorId is (still) registered with the scheduler Nov 28, 2014
@roxchkplusony
Copy link
Contributor Author

@rxin here's the v1.1 fix.

@SparkQA
Copy link

SparkQA commented Nov 28, 2014

QA tests have started for PR 3503 at commit 4f95baf.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Nov 28, 2014

QA tests have started for PR 3503 at commit 234d350.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Nov 28, 2014

QA tests have finished for PR 3503 at commit 4f95baf.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/23930/
Test PASSed.

@SparkQA
Copy link

SparkQA commented Nov 28, 2014

QA tests have finished for PR 3503 at commit 234d350.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/23931/
Test PASSed.

@rxin
Copy link
Contributor

rxin commented Nov 28, 2014

Thanks. Merging!

asfgit pushed a commit that referenced this pull request Nov 28, 2014
…) registered with the scheduler

v1.1 backport for #3483

Author: roxchkplusony <[email protected]>

Closes #3503 from roxchkplusony/bugfix/4626-1.1 and squashes the following commits:

234d350 [roxchkplusony] [SPARK-4626] Kill a task only if the executorId is (still) registered with the scheduler
@roxchkplusony roxchkplusony deleted the bugfix/4626-1.1 branch November 29, 2014 02:58
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.

4 participants