Yikun opened a new pull request #35819:
URL: https://github.com/apache/spark/pull/35819
### What changes were proposed in this pull request?
Use capability to limit disable queue
### Why are the changes needed?
In Volcano, weight should be [a positive
integer](https://github.com/volcano-sh/volcano/blob/c5bed286211af4f9d94503968c1c6b06baa882d1/pkg/webhooks/admission/queues/validate/validate_queue.go#L188),
so weight 0 is a wrong usage. As
[description](https://volcano.sh/en/docs/queue/) for queue
- weight is a soft constraint.
- capability is a hard constraint.
We better to use capability to limit disable queue.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
```
[info] VolcanoSuite:
[info] - Run SparkPi with volcano scheduler (11 seconds, 468 milliseconds)
[info] - SPARK-38187: Run SparkPi Jobs with minCPU (30 seconds, 636
milliseconds)
[info] - SPARK-38187: Run SparkPi Jobs with minMemory (31 seconds, 544
milliseconds)
[info] - SPARK-38188: Run SparkPi jobs with 2 queues (only 1 enabled) (13
seconds, 333 milliseconds)
[info] - SPARK-38188: Run SparkPi jobs with 2 queues (all enabled) (25
seconds, 486 milliseconds)
[info] - SPARK-38423: Run driver job to validate priority order (16 seconds,
488 milliseconds)
[info] Run completed in 2 minutes, 14 seconds.
[info] Total number of tests run: 6
[info] Suites: completed 2, aborted 0
[info] Tests: succeeded 6, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.
[success] Total time: 163 s (02:43), completed 2022-3-11 21:40:53
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
