GitHub user kayousterhout opened a pull request:

    https://github.com/apache/spark/pull/301

    Renamed stageIdToActiveJob to jobIdToActiveJob.

    This data structure was misused and, as a result, later renamed to an 
incorrect name.
    
    This data structure seems to have gotten into this tangled state as a 
result of @henrydavidge using the stageID instead of the job Id to index into 
it and later @andrewor14 renaming the data structure to reflect this 
misunderstanding.
    
    This patch renames it and removes an incorrect indexing into it.  The 
incorrect indexing into it meant that the code added by @henrydavidge to warn 
when a task size is too large (added here 
https://github.com/apache/spark/commit/57579934f0454f258615c10e69ac2adafc5b9835)
 was not always executed; this commit fixes that.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/kayousterhout/spark-1 fixCancellation

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/301.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 #301
    
----
commit bd3d3a450e6c44e38f4933e6d20f16ada00f9bfb
Author: Kay Ousterhout <[email protected]>
Date:   2014-04-02T07:37:04Z

    Renamed stageIdToActiveJob to jobIdToActiveJob.
    
    This data structure was misnamed and, as a result, misused.

----


---
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.
---

Reply via email to