improvement: Move the ResourceReservationManager to an interface downstream clients can mock it

palantir / k8s-spark-scheduler

A Kubernetes Scheduler Extender to provide gang scheduling support for Spark on Kubernetes

Apache License 2.0

175 stars 43 forks source link

Closed k-simons closed 1 year ago

k-simons commented 1 year ago

Move the ResourceReservationManager to an interface downstream clients can mock it
Right now demand manipulation are a combination of static methods and using the raw SafeDemandCache struct which makes it impossible to test or mock
This moves this operations behind a single interface
Noop
Similar to https://github.com/palantir/k8s-spark-scheduler/pull/255

==COMMIT_MSG== ==COMMIT_MSG==

svc-autorelease commented 1 year ago

Released 0.63.0