Skip to content

Add constrained scheduler benchmarks#1028

Merged
aaronlehmann merged 1 commit into
moby:masterfrom
aaronlehmann:add-constrained-scheduling-benchmarks
Jun 29, 2016
Merged

Add constrained scheduler benchmarks#1028
aaronlehmann merged 1 commit into
moby:masterfrom
aaronlehmann:add-constrained-scheduling-benchmarks

Conversation

@aaronlehmann
Copy link
Copy Markdown
Collaborator

This benchmarks scheduling with constraints, where only 1/3 of the nodes
are eligible to run the tasks.

Under the current implementation, this converges towards the worst-case
linear behavior, because the nodes that meet the constraints end up at
the end of the heap, and findMin has to scan much of the heap to find
them.

@codecov-io
Copy link
Copy Markdown

Current coverage is 53.91%

Merging #1028 into master will increase coverage by 0.22%

@@             master      #1028   diff @@
==========================================
  Files            75         75          
  Lines         11671      11642    -29   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           6266       6277    +11   
+ Misses         4501       4474    -27   
+ Partials        904        891    -13   

Sunburst

Powered by Codecov. Last updated by 5007feb...2482a83

Comment thread manager/scheduler/scheduler_test.go Outdated
}

func benchScheduler(b *testing.B, nodes, tasks int, worstCase bool) {
func benchScheduler(b *testing.B, nodes, tasks int, constraints, worstCase bool) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be better change constraints to networkConstraints. It's different from --constraint in service.

This benchmarks scheduling with constraints, where only 1/3 of the nodes
are eligible to run the tasks.

Under the current implementation, this converges towards the worst-case
linear behavior, because the nodes that meet the constraints end up at
the end of the heap, and findMin has to scan much of the heap to find
them.

Signed-off-by: Aaron Lehmann <aaron.lehmann@docker.com>
@aaronlehmann aaronlehmann force-pushed the add-constrained-scheduling-benchmarks branch from 2482a83 to 822cdfe Compare June 29, 2016 17:42
@aaronlehmann
Copy link
Copy Markdown
Collaborator Author

Updated, PTAL.

@dongluochen
Copy link
Copy Markdown
Contributor

LGTM

@aaronlehmann aaronlehmann merged commit 206e9ee into moby:master Jun 29, 2016
@aaronlehmann aaronlehmann deleted the add-constrained-scheduling-benchmarks branch June 29, 2016 17:53
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.

4 participants