Fixing oversight of missing synchronized on long version of recalibrate#9480
Merged
Fixing oversight of missing synchronized on long version of recalibrate#9480
Conversation
Contributor
|
🎯 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
bric3
approved these changes
Sep 8, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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