GitHub user rdblue opened a pull request:

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

    SPARK-13779: Avoid cancelling non-local container requests.

    ## What changes were proposed in this pull request?
    
    To maximize locality, the YarnAllocator would cancel any requests with a
    stale locality preference or no locality preference. This assumed that
    the majority of tasks had locality preferences, but may not be the case
    when scanning S3. This caused container requests for S3 tasks to be
    constantly cancelled and resubmitted.
    
    This changes the allocator's logic to cancel only stale requests and
    just enough requests without locality preferences to submit requests
    with locality preferences. This avoids cancelling requests without
    locality preferences that would be resubmitted without locality
    preferences.
    
    ## How was this patch tested?
    
    We've deployed this patch on our clusters and verified that jobs that 
couldn't get executors because they kept canceling and resubmitting requests 
are fixed. Large jobs are running fine.

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

    $ git pull https://github.com/rdblue/spark 
SPARK-13779-fix-yarn-allocator-requests

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

    https://github.com/apache/spark/pull/11612.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 #11612
    
----
commit a67fb2824300da82236e7315dae8b4b507a71296
Author: Ryan Blue <b...@apache.org>
Date:   2016-02-26T23:07:54Z

    SPARK-13779: Avoid cancelling non-local container requests.
    
    To maximize locality, the YarnAllocator would cancel any requests with a
    stale locality preference or no locality preference. This assumed that
    the majority of tasks had locality preferences, but may not be the case
    when scanning S3. This caused container requests for S3 tasks to be
    constantly cancelled and resubmitted.
    
    This changes the allocator's logic to cancel only stale requests and
    just enough requests without locality preferences to submit requests
    with locality preferences. This avoids cancelling requests without
    locality preferences that would be resubmitted without locality
    preferences.

----


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to