-
Notifications
You must be signed in to change notification settings - Fork 320
Fixing oversight of missing synchronized on long version of recalibrate #9480
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
bric3
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dooh missed this
|
🎯 Code Coverage 🔗 Commit SHA: 0522f88 | Docs | Was this helpful? Give us feedback! |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 46 metrics, 13 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.54.0-SNAPSHOT~0522f88545, baseline=1.54.0-SNAPSHOT~7528bd70cc
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.056 s) : 0, 1056420
Total [baseline] (8.67 s) : 0, 8670483
Agent [candidate] (1.05 s) : 0, 1049731
Total [candidate] (8.637 s) : 0, 8636875
section iast
Agent [baseline] (1.182 s) : 0, 1182212
Total [baseline] (9.317 s) : 0, 9317220
Agent [candidate] (1.191 s) : 0, 1190940
Total [candidate] (9.331 s) : 0, 9330907
gantt
title insecure-bank - break down per module: candidate=1.54.0-SNAPSHOT~0522f88545, baseline=1.54.0-SNAPSHOT~7528bd70cc
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.461 ms) : 0, 1461
crashtracking [candidate] (1.463 ms) : 0, 1463
BytebuddyAgent [baseline] (739.19 ms) : 0, 739190
BytebuddyAgent [candidate] (734.484 ms) : 0, 734484
GlobalTracer [baseline] (244.146 ms) : 0, 244146
GlobalTracer [candidate] (242.612 ms) : 0, 242612
AppSec [baseline] (30.324 ms) : 0, 30324
AppSec [candidate] (30.286 ms) : 0, 30286
Debugger [baseline] (6.091 ms) : 0, 6091
Debugger [candidate] (6.101 ms) : 0, 6101
Remote Config [baseline] (699.133 µs) : 0, 699
Remote Config [candidate] (694.381 µs) : 0, 694
Telemetry [baseline] (13.262 ms) : 0, 13262
Telemetry [candidate] (12.939 ms) : 0, 12939
section iast
crashtracking [baseline] (1.47 ms) : 0, 1470
crashtracking [candidate] (1.465 ms) : 0, 1465
BytebuddyAgent [baseline] (853.369 ms) : 0, 853369
BytebuddyAgent [candidate] (860.625 ms) : 0, 860625
GlobalTracer [baseline] (233.797 ms) : 0, 233797
GlobalTracer [candidate] (234.704 ms) : 0, 234704
IAST [baseline] (31.876 ms) : 0, 31876
IAST [candidate] (29.391 ms) : 0, 29391
AppSec [baseline] (25.148 ms) : 0, 25148
AppSec [candidate] (26.188 ms) : 0, 26188
Debugger [baseline] (6.756 ms) : 0, 6756
Debugger [candidate] (8.422 ms) : 0, 8422
Remote Config [baseline] (596.848 µs) : 0, 597
Remote Config [candidate] (600.714 µs) : 0, 601
Telemetry [baseline] (8.198 ms) : 0, 8198
Telemetry [candidate] (8.267 ms) : 0, 8267
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.54.0-SNAPSHOT~0522f88545, baseline=1.54.0-SNAPSHOT~7528bd70cc
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.055 s) : 0, 1054949
Total [baseline] (10.823 s) : 0, 10823434
Agent [candidate] (1.048 s) : 0, 1048204
Total [candidate] (10.642 s) : 0, 10642160
section appsec
Agent [baseline] (1.232 s) : 0, 1231597
Total [baseline] (10.837 s) : 0, 10837158
Agent [candidate] (1.226 s) : 0, 1225766
Total [candidate] (10.775 s) : 0, 10775354
section iast
Agent [baseline] (1.181 s) : 0, 1181065
Total [baseline] (10.956 s) : 0, 10955819
Agent [candidate] (1.179 s) : 0, 1179193
Total [candidate] (10.926 s) : 0, 10925677
section profiling
Agent [baseline] (1.199 s) : 0, 1199223
Total [baseline] (10.936 s) : 0, 10936229
Agent [candidate] (1.2 s) : 0, 1200076
Total [candidate] (10.854 s) : 0, 10853560
gantt
title petclinic - break down per module: candidate=1.54.0-SNAPSHOT~0522f88545, baseline=1.54.0-SNAPSHOT~7528bd70cc
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.468 ms) : 0, 1468
crashtracking [candidate] (1.455 ms) : 0, 1455
BytebuddyAgent [baseline] (737.65 ms) : 0, 737650
BytebuddyAgent [candidate] (733.306 ms) : 0, 733306
GlobalTracer [baseline] (244.148 ms) : 0, 244148
GlobalTracer [candidate] (242.567 ms) : 0, 242567
AppSec [baseline] (30.367 ms) : 0, 30367
AppSec [candidate] (30.157 ms) : 0, 30157
Debugger [baseline] (6.12 ms) : 0, 6120
Debugger [candidate] (6.057 ms) : 0, 6057
Remote Config [baseline] (690.681 µs) : 0, 691
Remote Config [candidate] (683.017 µs) : 0, 683
Telemetry [baseline] (13.124 ms) : 0, 13124
Telemetry [candidate] (12.846 ms) : 0, 12846
section appsec
crashtracking [baseline] (1.467 ms) : 0, 1467
crashtracking [candidate] (1.453 ms) : 0, 1453
BytebuddyAgent [baseline] (761.292 ms) : 0, 761292
BytebuddyAgent [candidate] (757.132 ms) : 0, 757132
GlobalTracer [baseline] (236.964 ms) : 0, 236964
GlobalTracer [candidate] (235.881 ms) : 0, 235881
AppSec [baseline] (169.569 ms) : 0, 169569
AppSec [candidate] (169.107 ms) : 0, 169107
Debugger [baseline] (8.141 ms) : 0, 8141
Debugger [candidate] (7.35 ms) : 0, 7350
Remote Config [baseline] (623.885 µs) : 0, 624
Remote Config [candidate] (626.049 µs) : 0, 626
Telemetry [baseline] (8.447 ms) : 0, 8447
Telemetry [candidate] (9.319 ms) : 0, 9319
IAST [baseline] (23.898 ms) : 0, 23898
IAST [candidate] (23.721 ms) : 0, 23721
section iast
crashtracking [baseline] (1.465 ms) : 0, 1465
crashtracking [candidate] (1.449 ms) : 0, 1449
BytebuddyAgent [baseline] (852.375 ms) : 0, 852375
BytebuddyAgent [candidate] (850.917 ms) : 0, 850917
GlobalTracer [baseline] (233.474 ms) : 0, 233474
GlobalTracer [candidate] (233.063 ms) : 0, 233063
AppSec [baseline] (27.759 ms) : 0, 27759
AppSec [candidate] (24.574 ms) : 0, 24574
Debugger [baseline] (7.554 ms) : 0, 7554
Debugger [candidate] (8.46 ms) : 0, 8460
Remote Config [baseline] (593.92 µs) : 0, 594
Remote Config [candidate] (589.531 µs) : 0, 590
Telemetry [baseline] (8.232 ms) : 0, 8232
Telemetry [candidate] (8.212 ms) : 0, 8212
IAST [baseline] (28.417 ms) : 0, 28417
IAST [candidate] (30.882 ms) : 0, 30882
section profiling
ProfilingAgent [baseline] (107.773 ms) : 0, 107773
ProfilingAgent [candidate] (108.499 ms) : 0, 108499
crashtracking [baseline] (1.429 ms) : 0, 1429
crashtracking [candidate] (1.438 ms) : 0, 1438
BytebuddyAgent [baseline] (763.052 ms) : 0, 763052
BytebuddyAgent [candidate] (762.507 ms) : 0, 762507
GlobalTracer [baseline] (222.455 ms) : 0, 222455
GlobalTracer [candidate] (222.899 ms) : 0, 222899
AppSec [baseline] (30.494 ms) : 0, 30494
AppSec [candidate] (30.685 ms) : 0, 30685
Debugger [baseline] (7.797 ms) : 0, 7797
Debugger [candidate] (6.301 ms) : 0, 6301
Remote Config [baseline] (706.519 µs) : 0, 707
Remote Config [candidate] (693.196 µs) : 0, 693
Telemetry [baseline] (14.787 ms) : 0, 14787
Telemetry [candidate] (16.464 ms) : 0, 16464
Profiling [baseline] (108.455 ms) : 0, 108455
Profiling [candidate] (109.177 ms) : 0, 109177
LoadParameters
See matching parameters
SummaryFound 3 performance improvements and 0 performance regressions! Performance is the same for 9 metrics, 12 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~0522f88545, baseline=1.54.0-SNAPSHOT~7528bd70cc
dateFormat X
axisFormat %s
section baseline
no_agent (4.441 ms) : 4390, 4493
. : milestone, 4441,
iast (9.345 ms) : 9187, 9503
. : milestone, 9345,
iast_FULL (14.17 ms) : 13890, 14450
. : milestone, 14170,
iast_GLOBAL (10.123 ms) : 9939, 10307
. : milestone, 10123,
profiling (9.297 ms) : 9125, 9470
. : milestone, 9297,
tracing (7.808 ms) : 7696, 7920
. : milestone, 7808,
section candidate
no_agent (4.279 ms) : 4218, 4340
. : milestone, 4279,
iast (9.286 ms) : 9133, 9438
. : milestone, 9286,
iast_FULL (13.833 ms) : 13559, 14107
. : milestone, 13833,
iast_GLOBAL (10.275 ms) : 10091, 10459
. : milestone, 10275,
profiling (9.111 ms) : 8965, 9258
. : milestone, 9111,
tracing (7.754 ms) : 7631, 7878
. : milestone, 7754,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~0522f88545, baseline=1.54.0-SNAPSHOT~7528bd70cc
dateFormat X
axisFormat %s
section baseline
no_agent (36.266 ms) : 35975, 36557
. : milestone, 36266,
appsec (47.923 ms) : 47498, 48349
. : milestone, 47923,
code_origins (47.276 ms) : 46848, 47705
. : milestone, 47276,
iast (46.461 ms) : 46044, 46878
. : milestone, 46461,
profiling (48.745 ms) : 48285, 49204
. : milestone, 48745,
tracing (44.679 ms) : 44298, 45061
. : milestone, 44679,
section candidate
no_agent (37.195 ms) : 36898, 37491
. : milestone, 37195,
appsec (48.385 ms) : 47966, 48803
. : milestone, 48385,
code_origins (45.487 ms) : 45100, 45874
. : milestone, 45487,
iast (44.394 ms) : 44016, 44771
. : milestone, 44394,
profiling (49.288 ms) : 48807, 49768
. : milestone, 49288,
tracing (43.828 ms) : 43459, 44197
. : milestone, 43828,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 1 unstable metrics. Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~0522f88545, baseline=1.54.0-SNAPSHOT~7528bd70cc
dateFormat X
axisFormat %s
section baseline
no_agent (1.473 ms) : 1462, 1485
. : milestone, 1473,
appsec (3.658 ms) : 3442, 3874
. : milestone, 3658,
iast (2.192 ms) : 2129, 2254
. : milestone, 2192,
iast_GLOBAL (2.25 ms) : 2187, 2313
. : milestone, 2250,
profiling (2.046 ms) : 1996, 2097
. : milestone, 2046,
tracing (2.022 ms) : 1974, 2071
. : milestone, 2022,
section candidate
no_agent (1.479 ms) : 1467, 1491
. : milestone, 1479,
appsec (3.586 ms) : 3377, 3796
. : milestone, 3586,
iast (2.199 ms) : 2137, 2262
. : milestone, 2199,
iast_GLOBAL (2.244 ms) : 2181, 2308
. : milestone, 2244,
profiling (2.056 ms) : 2004, 2108
. : milestone, 2056,
tracing (2.012 ms) : 1963, 2061
. : milestone, 2012,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~0522f88545, baseline=1.54.0-SNAPSHOT~7528bd70cc
dateFormat X
axisFormat %s
section baseline
no_agent (15.432 s) : 15432000, 15432000
. : milestone, 15432000,
appsec (14.943 s) : 14943000, 14943000
. : milestone, 14943000,
iast (18.295 s) : 18295000, 18295000
. : milestone, 18295000,
iast_GLOBAL (17.776 s) : 17776000, 17776000
. : milestone, 17776000,
profiling (15.985 s) : 15985000, 15985000
. : milestone, 15985000,
tracing (14.976 s) : 14976000, 14976000
. : milestone, 14976000,
section candidate
no_agent (15.517 s) : 15517000, 15517000
. : milestone, 15517000,
appsec (14.901 s) : 14901000, 14901000
. : milestone, 14901000,
iast (18.411 s) : 18411000, 18411000
. : milestone, 18411000,
iast_GLOBAL (18.227 s) : 18227000, 18227000
. : milestone, 18227000,
profiling (15.352 s) : 15352000, 15352000
. : milestone, 15352000,
tracing (14.936 s) : 14936000, 14936000
. : milestone, 14936000,
|
The racy updates of cache stat fields is deliberate -- and that's the cause of the inconsistent sync-ing
What Does This Do
Fixing oversight in #9434
Made recalibrate() synchronized but not recalibrate(long)
Motivation
Consistency - in this particular case, synchronized isn't strictly needed for correctness,
but does limit raciness with an operation that is doing a bulk update to the cache