----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66190/ -----------------------------------------------------------
(Updated March 21, 2018, 8:23 p.m.) Review request for Aurora, David McLaughlin, Santhosh Kumar Shanmugham, and Stephan Erb. Changes ------- Change cluster allocation values to previous values. Repository: aurora Description ------- This benchmark was using the deprecated `production` flag when building the tasks for the cluster. `PendingTaskProcessor` depends on `tier` instead, so this benchmark ended up not testing the correct codepath. Removed references to `production` and added `tier` instead. Additionally, removed some unused options. Diffs (updated) ----- src/jmh/java/org/apache/aurora/benchmark/BenchmarkSettings.java ddab2eccb22a93ecb67481f399707d2d82df5db2 src/jmh/java/org/apache/aurora/benchmark/SchedulingBenchmarks.java 1f9a5764b502f939f0345ff99fb0fc2830b4c2f0 src/jmh/java/org/apache/aurora/benchmark/Tasks.java 60c62bbf3061650a5dd8654045dc8189293d0190 src/main/java/org/apache/aurora/scheduler/http/State.java 5b3b7c52fb3623c45518d27b242527ce0e775e06 src/main/java/org/apache/aurora/scheduler/preemptor/PendingTaskProcessor.java ef06471d007b1d36300eea30cdea059c1ba231b0 src/main/java/org/apache/aurora/scheduler/preemptor/PreemptionVictim.java 780689e1b8967a9f558a464271b7b60035dce1a7 src/main/java/org/apache/aurora/scheduler/preemptor/PreemptionVictimFilter.java 569cfe6b04e6b7bf0dca7625b00698e9d8e47daf src/main/java/org/apache/aurora/scheduler/preemptor/Preemptor.java 293d106eee383dd5352a629780b897d58c9dd439 src/main/java/org/apache/aurora/scheduler/state/ClusterState.java 527cfd6bd7d55ffb526919ea772a0947ea3e958f src/main/java/org/apache/aurora/scheduler/state/ClusterStateImpl.java d804198f5b3c0f835997af6693360b8376ccd865 src/test/java/org/apache/aurora/scheduler/http/StateTest.java 852c2f8c1cf23c6f448e78d476608a829b786751 src/test/java/org/apache/aurora/scheduler/preemptor/ClusterStateImplTest.java 84987212be184edfe801278213a8b2bb0c49db9e src/test/java/org/apache/aurora/scheduler/preemptor/PendingTaskProcessorTest.java ba775f4688dc57504e2def0dc4b5dcd00da448e1 Diff: https://reviews.apache.org/r/66190/diff/3/ Changes: https://reviews.apache.org/r/66190/diff/2-3/ Testing ------- Old: ``` # Run complete. Total time: 00:08:32 Benchmark (numPendingTasks) Mode Cnt Score Error Units SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark 1 thrpt 10 57.670 ± 20.451 ops/s SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate 1 thrpt 10 595.374 ± 211.805 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate.norm 1 thrpt 10 10830342.916 ± 380.919 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space 1 thrpt 10 593.530 ± 222.002 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space.norm 1 thrpt 10 10717947.102 ± 1280229.296 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space 1 thrpt 10 0.305 ± 1.264 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space.norm 1 thrpt 10 13552.434 ± 61403.918 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.count 1 thrpt 10 60.000 counts SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.time 1 thrpt 10 202.000 ms SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·stack 1 thrpt NaN --- SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark 10 thrpt 10 52.161 ± 8.526 ops/s SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate 10 thrpt 10 550.771 ± 89.939 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate.norm 10 thrpt 10 11074211.352 ± 318.376 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space 10 thrpt 10 550.125 ± 107.470 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space.norm 10 thrpt 10 11073792.311 ± 1621636.993 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space 10 thrpt 10 0.038 ± 0.049 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space.norm 10 thrpt 10 737.753 ± 919.460 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.count 10 thrpt 10 55.000 counts SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.time 10 thrpt 10 155.000 ms SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·stack 10 thrpt NaN --- SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark 100 thrpt 10 55.574 ± 7.414 ops/s SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate 100 thrpt 10 588.906 ± 78.568 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate.norm 100 thrpt 10 11113945.746 ± 360.266 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space 100 thrpt 10 581.912 ± 91.360 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space.norm 100 thrpt 10 10980432.961 ± 933636.090 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space 100 thrpt 10 0.037 ± 0.025 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space.norm 100 thrpt 10 710.276 ± 502.196 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.count 100 thrpt 10 59.000 counts SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.time 100 thrpt 10 155.000 ms SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·stack 100 thrpt NaN --- SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark 1000 thrpt 10 53.541 ± 8.977 ops/s SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate 1000 thrpt 10 587.639 ± 98.529 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate.norm 1000 thrpt 10 11509965.960 ± 256.384 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space 1000 thrpt 10 590.747 ± 102.626 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space.norm 1000 thrpt 10 11581787.397 ± 919456.716 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space 1000 thrpt 10 0.060 ± 0.072 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space.norm 1000 thrpt 10 1206.676 ± 1392.806 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.count 1000 thrpt 10 59.000 counts SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.time 1000 thrpt 10 161.000 ms SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·stack 1000 thrpt NaN --- ``` New: ``` # Run complete. Total time: 00:08:32 Benchmark (numPendingTasks) Mode Cnt Score Error Units SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark 1 thrpt 10 9.742 ± 1.220 ops/s SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate 1 thrpt 10 1134.480 ± 142.355 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate.norm 1 thrpt 10 122145964.287 ± 162.619 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space 1 thrpt 10 1137.705 ± 130.036 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space.norm 1 thrpt 10 122666774.722 ± 7826221.674 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space 1 thrpt 10 0.038 ± 0.058 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space.norm 1 thrpt 10 4437.495 ± 7332.908 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.count 1 thrpt 10 113.000 counts SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.time 1 thrpt 10 278.000 ms SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·stack 1 thrpt NaN --- SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark 10 thrpt 10 9.004 ± 1.477 ops/s SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate 10 thrpt 10 1048.718 ± 172.117 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate.norm 10 thrpt 10 122149834.627 ± 2.709 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space 10 thrpt 10 1057.001 ± 189.010 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space.norm 10 thrpt 10 123140186.200 ± 9676263.385 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space 10 thrpt 10 0.016 ± 0.009 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space.norm 10 thrpt 10 1798.961 ± 960.508 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.count 10 thrpt 10 105.000 counts SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.time 10 thrpt 10 298.000 ms SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·stack 10 thrpt NaN --- SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark 100 thrpt 10 9.674 ± 1.712 ops/s SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate 100 thrpt 10 1127.256 ± 199.510 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate.norm 100 thrpt 10 122190185.861 ± 694.357 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space 100 thrpt 10 1130.353 ± 202.566 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space.norm 100 thrpt 10 122611030.168 ± 7904770.698 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space 100 thrpt 10 0.285 ± 1.196 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space.norm 100 thrpt 10 32082.849 ± 135251.004 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.count 100 thrpt 10 114.000 counts SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.time 100 thrpt 10 471.000 ms SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·stack 100 thrpt NaN --- SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark 1000 thrpt 10 9.391 ± 1.679 ops/s SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate 1000 thrpt 10 1097.729 ± 196.175 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.alloc.rate.norm 1000 thrpt 10 122587540.024 ± 6076.493 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space 1000 thrpt 10 1098.710 ± 200.125 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Eden_Space.norm 1000 thrpt 10 122749450.285 ± 8291123.010 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space 1000 thrpt 10 0.276 ± 1.249 MB/sec SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.churn.PS_Survivor_Space.norm 1000 thrpt 10 43334.762 ± 199709.049 B/op SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.count 1000 thrpt 10 111.000 counts SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·gc.time 1000 thrpt 10 519.000 ms SchedulingBenchmarks.PreemptorSlotSearchBenchmark.runBenchmark:·stack 1000 thrpt NaN --- ``` Benchmark is slower as it is doing more (correct) work. Thanks, Jordan Ly
