GitHub user erenavsarogullari opened a pull request:
https://github.com/apache/spark/pull/15463
[SPARK-17894] [CORE] Ensure uniqueness of TaskSetManager name.
## What changes were proposed in this pull request?
`TaskSetManager` should have unique name to avoid adding duplicate ones to
parent `Pool` via `SchedulableBuilder`. This problem has been surfaced with
following discussion: [[PR: Avoid adding of duplicate
schedulables]](https://github.com/apache/spark/pull/15326)
**Proposal** :
There is 1x1 relationship between `stageAttemptId` and `TaskSetManager` so
`taskSet.Id` covering both `stageId` and `stageAttemptId` looks to be used for
uniqueness of `TaskSetManager` name instead of just `stageId`.
**Current TaskSetManager Name** :
`var name = "TaskSet_" + taskSet.stageId.toString`
**Sample**: TaskSet_0
**Proposed TaskSetManager Name** :
`val name = "TaskSet_" + taskSet.Id ` `// taskSet.Id = (stageId + "." +
stageAttemptId)`
**Sample** : TaskSet_0.0
## How was this patch tested?
Added new Unit Test.
cc @kayousterhout @markhamstra
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/erenavsarogullari/spark SPARK-17894
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/15463.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #15463
----
commit 6f79ff857fb3db0eb9baea390c96fec198b39f62
Author: erenavsarogullari <[email protected]>
Date: 2016-10-13T09:28:40Z
Ensure uniqueness of TaskSetManager name.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
