You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@aurora.apache.org by Jordan Ly <jo...@gmail.com> on 2018/03/21 00:26:37 UTC

Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66190/
-----------------------------------------------------------

(Updated March 21, 2018, 12:26 a.m.)


Review request for Aurora, David McLaughlin, Santhosh Kumar Shanmugham, and Stephan Erb.


Changes
-------

Fix title (seemed a bit misleading oops).


Summary (updated)
-----------------

Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark


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 


Diff: https://reviews.apache.org/r/66190/diff/2/

Changes: https://reviews.apache.org/r/66190/diff/1-2/


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


Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark

Posted by Santhosh Kumar Shanmugham <sa...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66190/#review199628
-----------------------------------------------------------


Ship it!




Ship It!

- Santhosh Kumar Shanmugham


On March 20, 2018, 5:26 p.m., Jordan Ly wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/66190/
> -----------------------------------------------------------
> 
> (Updated March 20, 2018, 5:26 p.m.)
> 
> 
> Review request for Aurora, David McLaughlin, Santhosh Kumar Shanmugham, and Stephan Erb.
> 
> 
> 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
> -----
> 
>   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 
> 
> 
> Diff: https://reviews.apache.org/r/66190/diff/2/
> 
> 
> 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
> 
>


Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark

Posted by David McLaughlin <da...@dmclaughlin.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66190/#review199636
-----------------------------------------------------------


Ship it!




Ship It!

- David McLaughlin


On March 21, 2018, 12:26 a.m., Jordan Ly wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/66190/
> -----------------------------------------------------------
> 
> (Updated March 21, 2018, 12:26 a.m.)
> 
> 
> Review request for Aurora, David McLaughlin, Santhosh Kumar Shanmugham, and Stephan Erb.
> 
> 
> 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
> -----
> 
>   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 
> 
> 
> Diff: https://reviews.apache.org/r/66190/diff/2/
> 
> 
> 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
> 
>


Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark

Posted by Aurora ReviewBot <wf...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66190/#review199625
-----------------------------------------------------------


Ship it!




Master (b3fa9fe) is green with this patch.
  ./build-support/jenkins/build.sh

I will refresh this build result if you post a review containing "@ReviewBot retry"

- Aurora ReviewBot


On March 20, 2018, 5:26 p.m., Jordan Ly wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/66190/
> -----------------------------------------------------------
> 
> (Updated March 20, 2018, 5:26 p.m.)
> 
> 
> Review request for Aurora, David McLaughlin, Santhosh Kumar Shanmugham, and Stephan Erb.
> 
> 
> 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
> -----
> 
>   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 
> 
> 
> Diff: https://reviews.apache.org/r/66190/diff/2/
> 
> 
> 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
> 
>


Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark

Posted by Aurora ReviewBot <wf...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66190/#review199702
-----------------------------------------------------------


Ship it!




Master (f32086d) is green with this patch.
  ./build-support/jenkins/build.sh

I will refresh this build result if you post a review containing "@ReviewBot retry"

- Aurora ReviewBot


On March 21, 2018, 8:23 p.m., Jordan Ly wrote:
> 
> -----------------------------------------------------------
> 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.
> 
> 
> 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
> -----
> 
>   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 
> 
> 
> Diff: https://reviews.apache.org/r/66190/diff/4/
> 
> 
> 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
> 
>


Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark

Posted by Stephan Erb <se...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66190/#review199774
-----------------------------------------------------------


Ship it!




Ship It!

- Stephan Erb


On March 21, 2018, 9:23 p.m., Jordan Ly wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/66190/
> -----------------------------------------------------------
> 
> (Updated March 21, 2018, 9:23 p.m.)
> 
> 
> Review request for Aurora, David McLaughlin, Santhosh Kumar Shanmugham, and Stephan Erb.
> 
> 
> 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
> -----
> 
>   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 
> 
> 
> Diff: https://reviews.apache.org/r/66190/diff/4/
> 
> 
> 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
> 
>


Re: Review Request 66190: Fix 'PreemptorSlotSearchBenchmark', remove 'isProduction' references in benchmark

Posted by Jordan Ly <jo...@gmail.com>.
-----------------------------------------------------------
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