Skip to content

Conversation

@pgillet
Copy link
Contributor

@pgillet pgillet commented Feb 23, 2018

As for Yarn, Mesos users should be able to specify priority queues to define a workload management policy for queued drivers in the Mesos Cluster Dispatcher.

What changes were proposed in this pull request?

This commit introduces the necessary changes to manage such priority queues.

How was this patch tested?

unit tests

@felixcheung
Copy link
Member

@susanxhuynh

@skonto
Copy link
Contributor

skonto commented Feb 28, 2018

@pgillet since you are modifying the ui could you add a screenshot in the description?

@pgillet
Copy link
Contributor Author

pgillet commented Feb 28, 2018

@skonto I attached a screenshot in the JIRA.

@skonto
Copy link
Contributor

skonto commented Mar 1, 2018

@pgillet why not here it makes more sense.

@pgillet
Copy link
Contributor Author

pgillet commented Mar 1, 2018

@skonto Below a screenshot of the MesosClusterDispatcher UI showing Spark jobs along with the queue to which they are submitted:
screenshot from 2018-02-28 17-22-47

@felixcheung
Copy link
Member

any thought on this?

@felixcheung
Copy link
Member

@tnachen

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@github-actions
Copy link

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!

@github-actions github-actions bot added the Stale label Apr 12, 2020
@github-actions github-actions bot closed this Apr 13, 2020
dongjoon-hyun pushed a commit that referenced this pull request Nov 17, 2020
### What changes were proposed in this pull request?

I push this PR as I could not re-open the stale one #20665 .

As for Yarn or Kubernetes, Mesos users should be able to specify priority queues to define a workload management policy for queued drivers in the Mesos Cluster Dispatcher.

This would ensure scheduling order while enqueuing Spark applications for a Mesos cluster.

### Why are the changes needed?

Currently, submitted drivers are kept in order of their submission: the first driver added to the queue will be the first one to be executed (FIFO), regardless of their priority.

See https://issues.apache.org/jira/projects/SPARK/issues/SPARK-23499 for more details.

### Does this PR introduce _any_ user-facing change?

The MesosClusterDispatcher UI shows now Spark jobs along with the queue to which they are submitted.

### How was this patch tested?

Unit tests.
Also, this feature has been in production for 3 years now as we use a modified Spark 2.4.0 since then.

Closes #30352 from pgillet/mesos-scheduler-priority-queue.

Lead-authored-by: Pascal Gillet <[email protected]>
Co-authored-by: pgillet <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
a0x8o added a commit to a0x8o/spark that referenced this pull request Nov 17, 2020
### What changes were proposed in this pull request?

I push this PR as I could not re-open the stale one apache/spark#20665 .

As for Yarn or Kubernetes, Mesos users should be able to specify priority queues to define a workload management policy for queued drivers in the Mesos Cluster Dispatcher.

This would ensure scheduling order while enqueuing Spark applications for a Mesos cluster.

### Why are the changes needed?

Currently, submitted drivers are kept in order of their submission: the first driver added to the queue will be the first one to be executed (FIFO), regardless of their priority.

See https://issues.apache.org/jira/projects/SPARK/issues/SPARK-23499 for more details.

### Does this PR introduce _any_ user-facing change?

The MesosClusterDispatcher UI shows now Spark jobs along with the queue to which they are submitted.

### How was this patch tested?

Unit tests.
Also, this feature has been in production for 3 years now as we use a modified Spark 2.4.0 since then.

Closes #30352 from pgillet/mesos-scheduler-priority-queue.

Lead-authored-by: Pascal Gillet <[email protected]>
Co-authored-by: pgillet <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants