GitHub user skonto opened a pull request:
https://github.com/apache/spark/pull/19390
[SPARK-18935][MESOS] Fix dynamic reservations
## What changes were proposed in this pull request?
- Solves the issue described in the ticket by preserving reservation and
allocation info.
- Adds extra debug level logging to make debugging easier in the future eg.
> 17/09/29 14:53:07 DEBUG MesosCoarseGrainedSchedulerBackend: Accepting
offer: f20de49b-dee3-45dd-a3c1-73418b7de891-O32 with attributes: Map()
allocation info: role: "spark-prive"
> reservation info: name: "ports"
> type: RANGES
> ranges {
> range {
> begin: 31000
> end: 32000
> }
> }
> role: "spark-prive"
> reservation {
> principal: "test"
> }
> allocation_info {
> role: "spark-prive"
> }
- Some style cleanup.
## How was this patch tested?
Manually by running the example in the ticket with and without a principal.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/skonto/spark fix_dynamic_reservation
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/19390.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 #19390
----
commit 28717d9be1f6bcca541eeb8da0d4083db8465610
Author: Stavros Kontopoulos <[email protected]>
Date: 2017-09-29T11:49:23Z
fix dynamic reservations
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
