-
Notifications
You must be signed in to change notification settings - Fork 290
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
Ensure time waiting for CyclicBarrier
and CountDownLatch
is eligible for wallclock profiling
#6941
Conversation
01016ec
to
7a32cbe
Compare
7a32cbe
to
37c38e5
Compare
@Override | ||
public boolean isEnabled() { | ||
// only needed if wallclock profiling is enabled, which requires tracing | ||
return super.isEnabled() |
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.
I assume super.isEnabled()
will return true
by default? Eg. it is not necessary to enable the integration by adding a config option?
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.
Exactly, we want to bypass the tracer based enablement mechanism and use our own. Maybe we can refactor this at a later date so we don't need to do this, but this was just carried over from the instrumentation being generalised.
425e1ff
to
0192701
Compare
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 50 metrics, 13 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.34.0-SNAPSHOT~a06db6224e, baseline=1.34.0-SNAPSHOT~ae1c4c9475
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.075 s) : 0, 1075406
Total [baseline] (10.456 s) : 0, 10456231
Agent [candidate] (1.076 s) : 0, 1076079
Total [candidate] (10.414 s) : 0, 10414422
section appsec
Agent [baseline] (1.192 s) : 0, 1192258
Total [baseline] (10.502 s) : 0, 10501904
Agent [candidate] (1.188 s) : 0, 1188032
Total [candidate] (10.482 s) : 0, 10482406
section iast
Agent [baseline] (1.206 s) : 0, 1205807
Total [baseline] (10.801 s) : 0, 10800907
Agent [candidate] (1.202 s) : 0, 1201961
Total [candidate] (10.787 s) : 0, 10787170
section profiling
Agent [baseline] (1.268 s) : 0, 1268394
Total [baseline] (10.743 s) : 0, 10743100
Agent [candidate] (1.278 s) : 0, 1277738
Total [candidate] (10.634 s) : 0, 10634385
gantt
title petclinic - break down per module: candidate=1.34.0-SNAPSHOT~a06db6224e, baseline=1.34.0-SNAPSHOT~ae1c4c9475
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (673.622 ms) : 0, 673622
BytebuddyAgent [candidate] (673.584 ms) : 0, 673584
GlobalTracer [baseline] (309.667 ms) : 0, 309667
GlobalTracer [candidate] (310.198 ms) : 0, 310198
AppSec [baseline] (49.506 ms) : 0, 49506
AppSec [candidate] (49.543 ms) : 0, 49543
Remote Config [baseline] (650.923 µs) : 0, 651
Remote Config [candidate] (659.743 µs) : 0, 660
Telemetry [baseline] (7.573 ms) : 0, 7573
Telemetry [candidate] (7.69 ms) : 0, 7690
section appsec
BytebuddyAgent [baseline] (697.264 ms) : 0, 697264
BytebuddyAgent [candidate] (694.904 ms) : 0, 694904
GlobalTracer [baseline] (292.168 ms) : 0, 292168
GlobalTracer [candidate] (291.191 ms) : 0, 291191
AppSec [baseline] (149.818 ms) : 0, 149818
AppSec [candidate] (149.369 ms) : 0, 149369
IAST [baseline] (19.435 ms) : 0, 19435
IAST [candidate] (19.3 ms) : 0, 19300
Remote Config [baseline] (623.262 µs) : 0, 623
Remote Config [candidate] (615.747 µs) : 0, 616
Telemetry [baseline] (8.174 ms) : 0, 8174
Telemetry [candidate] (7.849 ms) : 0, 7849
section iast
BytebuddyAgent [baseline] (799.275 ms) : 0, 799275
BytebuddyAgent [candidate] (794.647 ms) : 0, 794647
GlobalTracer [baseline] (289.391 ms) : 0, 289391
GlobalTracer [candidate] (289.868 ms) : 0, 289868
AppSec [baseline] (50.402 ms) : 0, 50402
AppSec [candidate] (50.721 ms) : 0, 50721
IAST [baseline] (24.165 ms) : 0, 24165
IAST [candidate] (23.663 ms) : 0, 23663
Remote Config [baseline] (577.982 µs) : 0, 578
Remote Config [candidate] (573.753 µs) : 0, 574
Telemetry [baseline] (7.399 ms) : 0, 7399
Telemetry [candidate] (8.175 ms) : 0, 8175
section profiling
BytebuddyAgent [baseline] (677.431 ms) : 0, 677431
BytebuddyAgent [candidate] (683.021 ms) : 0, 683021
GlobalTracer [baseline] (380.489 ms) : 0, 380489
GlobalTracer [candidate] (382.598 ms) : 0, 382598
AppSec [baseline] (50.395 ms) : 0, 50395
AppSec [candidate] (50.643 ms) : 0, 50643
Remote Config [baseline] (719.204 µs) : 0, 719
Remote Config [candidate] (711.347 µs) : 0, 711
Telemetry [baseline] (7.498 ms) : 0, 7498
Telemetry [candidate] (7.542 ms) : 0, 7542
ProfilingAgent [baseline] (95.431 ms) : 0, 95431
ProfilingAgent [candidate] (96.356 ms) : 0, 96356
Profiling [baseline] (95.455 ms) : 0, 95455
Profiling [candidate] (96.38 ms) : 0, 96380
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.34.0-SNAPSHOT~a06db6224e, baseline=1.34.0-SNAPSHOT~ae1c4c9475
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.076 s) : 0, 1075971
Total [baseline] (8.562 s) : 0, 8562399
Agent [candidate] (1.094 s) : 0, 1093850
Total [candidate] (8.582 s) : 0, 8581660
section iast
Agent [baseline] (1.2 s) : 0, 1200363
Total [baseline] (9.017 s) : 0, 9016975
Agent [candidate] (1.212 s) : 0, 1211701
Total [candidate] (9.082 s) : 0, 9081938
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.197 s) : 0, 1196805
Total [baseline] (8.973 s) : 0, 8973479
Agent [candidate] (1.198 s) : 0, 1198440
Total [candidate] (9.021 s) : 0, 9020912
section iast_TELEMETRY_OFF
Agent [baseline] (1.207 s) : 0, 1207086
Total [baseline] (9.017 s) : 0, 9017244
Agent [candidate] (1.198 s) : 0, 1198279
Total [candidate] (9.006 s) : 0, 9006115
gantt
title insecure-bank - break down per module: candidate=1.34.0-SNAPSHOT~a06db6224e, baseline=1.34.0-SNAPSHOT~ae1c4c9475
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (673.838 ms) : 0, 673838
BytebuddyAgent [candidate] (686.928 ms) : 0, 686928
GlobalTracer [baseline] (309.687 ms) : 0, 309687
GlobalTracer [candidate] (313.687 ms) : 0, 313687
AppSec [baseline] (49.695 ms) : 0, 49695
AppSec [candidate] (49.834 ms) : 0, 49834
Remote Config [baseline] (659.848 µs) : 0, 660
Remote Config [candidate] (662.845 µs) : 0, 663
Telemetry [baseline] (7.692 ms) : 0, 7692
Telemetry [candidate] (7.667 ms) : 0, 7667
section iast
BytebuddyAgent [baseline] (795.114 ms) : 0, 795114
BytebuddyAgent [candidate] (801.241 ms) : 0, 801241
GlobalTracer [baseline] (288.37 ms) : 0, 288370
GlobalTracer [candidate] (292.094 ms) : 0, 292094
AppSec [baseline] (50.441 ms) : 0, 50441
AppSec [candidate] (50.857 ms) : 0, 50857
IAST [baseline] (24.849 ms) : 0, 24849
IAST [candidate] (24.455 ms) : 0, 24455
Remote Config [baseline] (567.415 µs) : 0, 567
Remote Config [candidate] (1.479 ms) : 0, 1479
Telemetry [baseline] (6.615 ms) : 0, 6615
Telemetry [candidate] (6.816 ms) : 0, 6816
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (793.16 ms) : 0, 793160
BytebuddyAgent [candidate] (794.069 ms) : 0, 794069
GlobalTracer [baseline] (287.61 ms) : 0, 287610
GlobalTracer [candidate] (288.087 ms) : 0, 288087
AppSec [baseline] (52.109 ms) : 0, 52109
AppSec [candidate] (50.962 ms) : 0, 50962
IAST [baseline] (21.609 ms) : 0, 21609
IAST [candidate] (22.851 ms) : 0, 22851
Remote Config [baseline] (578.124 µs) : 0, 578
Remote Config [candidate] (586.116 µs) : 0, 586
Telemetry [baseline] (7.45 ms) : 0, 7450
Telemetry [candidate] (7.5 ms) : 0, 7500
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (799.074 ms) : 0, 799074
BytebuddyAgent [candidate] (792.928 ms) : 0, 792928
GlobalTracer [baseline] (290.601 ms) : 0, 290601
GlobalTracer [candidate] (288.944 ms) : 0, 288944
AppSec [baseline] (50.547 ms) : 0, 50547
AppSec [candidate] (48.705 ms) : 0, 48705
IAST [baseline] (22.593 ms) : 0, 22593
IAST [candidate] (23.729 ms) : 0, 23729
Remote Config [baseline] (588.76 µs) : 0, 589
Remote Config [candidate] (588.006 µs) : 0, 588
Telemetry [baseline] (9.026 ms) : 0, 9026
Telemetry [candidate] (8.866 ms) : 0, 8866
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 13 metrics, 15 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.34.0-SNAPSHOT~a06db6224e, baseline=1.34.0-SNAPSHOT~ae1c4c9475
dateFormat X
axisFormat %s
section baseline
no_agent (1.339 ms) : 1319, 1359
. : milestone, 1339,
appsec (1.727 ms) : 1703, 1751
. : milestone, 1727,
appsec_no_iast (1.727 ms) : 1702, 1751
. : milestone, 1727,
iast (1.484 ms) : 1461, 1507
. : milestone, 1484,
profiling (1.5 ms) : 1475, 1525
. : milestone, 1500,
tracing (1.486 ms) : 1462, 1509
. : milestone, 1486,
section candidate
no_agent (1.358 ms) : 1339, 1378
. : milestone, 1358,
appsec (1.736 ms) : 1712, 1760
. : milestone, 1736,
appsec_no_iast (1.701 ms) : 1676, 1726
. : milestone, 1701,
iast (1.496 ms) : 1473, 1519
. : milestone, 1496,
profiling (1.532 ms) : 1507, 1557
. : milestone, 1532,
tracing (1.462 ms) : 1437, 1487
. : milestone, 1462,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.34.0-SNAPSHOT~a06db6224e, baseline=1.34.0-SNAPSHOT~ae1c4c9475
dateFormat X
axisFormat %s
section baseline
no_agent (373.868 µs) : 354, 393
. : milestone, 374,
iast (473.283 µs) : 453, 494
. : milestone, 473,
iast_FULL (541.387 µs) : 520, 563
. : milestone, 541,
iast_GLOBAL (498.682 µs) : 477, 520
. : milestone, 499,
iast_HARDCODED_SECRET_DISABLED (475.238 µs) : 455, 496
. : milestone, 475,
iast_INACTIVE (452.65 µs) : 431, 474
. : milestone, 453,
iast_TELEMETRY_OFF (468.506 µs) : 448, 489
. : milestone, 469,
tracing (442.237 µs) : 422, 463
. : milestone, 442,
section candidate
no_agent (382.027 µs) : 361, 403
. : milestone, 382,
iast (476.308 µs) : 455, 498
. : milestone, 476,
iast_FULL (541.91 µs) : 521, 563
. : milestone, 542,
iast_GLOBAL (500.119 µs) : 479, 522
. : milestone, 500,
iast_HARDCODED_SECRET_DISABLED (477.636 µs) : 457, 499
. : milestone, 478,
iast_INACTIVE (457.907 µs) : 438, 478
. : milestone, 458,
iast_TELEMETRY_OFF (469.264 µs) : 448, 490
. : milestone, 469,
tracing (445.754 µs) : 426, 466
. : milestone, 446,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 0 unstable metrics. Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.34.0-SNAPSHOT~a06db6224e, baseline=1.34.0-SNAPSHOT~ae1c4c9475
dateFormat X
axisFormat %s
section baseline
no_agent (15.545 s) : 15545000, 15545000
. : milestone, 15545000,
appsec (14.943 s) : 14943000, 14943000
. : milestone, 14943000,
iast (18.682 s) : 18682000, 18682000
. : milestone, 18682000,
iast_GLOBAL (17.901 s) : 17901000, 17901000
. : milestone, 17901000,
profiling (15.285 s) : 15285000, 15285000
. : milestone, 15285000,
tracing (15.033 s) : 15033000, 15033000
. : milestone, 15033000,
section candidate
no_agent (15.139 s) : 15139000, 15139000
. : milestone, 15139000,
appsec (15.153 s) : 15153000, 15153000
. : milestone, 15153000,
iast (19.023 s) : 19023000, 19023000
. : milestone, 19023000,
iast_GLOBAL (17.946 s) : 17946000, 17946000
. : milestone, 17946000,
profiling (15.829 s) : 15829000, 15829000
. : milestone, 15829000,
tracing (15.074 s) : 15074000, 15074000
. : milestone, 15074000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.34.0-SNAPSHOT~a06db6224e, baseline=1.34.0-SNAPSHOT~ae1c4c9475
dateFormat X
axisFormat %s
section baseline
no_agent (1.455 ms) : 1444, 1466
. : milestone, 1455,
appsec (2.189 ms) : 2156, 2222
. : milestone, 2189,
iast (1.884 ms) : 1849, 1919
. : milestone, 1884,
iast_GLOBAL (1.92 ms) : 1885, 1954
. : milestone, 1920,
profiling (1.84 ms) : 1807, 1874
. : milestone, 1840,
tracing (1.824 ms) : 1793, 1856
. : milestone, 1824,
section candidate
no_agent (1.455 ms) : 1443, 1466
. : milestone, 1455,
appsec (2.186 ms) : 2153, 2220
. : milestone, 2186,
iast (1.87 ms) : 1836, 1905
. : milestone, 1870,
iast_GLOBAL (1.909 ms) : 1874, 1944
. : milestone, 1909,
profiling (1.84 ms) : 1808, 1873
. : milestone, 1840,
tracing (1.824 ms) : 1792, 1856
. : milestone, 1824,
|
8dd8a8f
to
7db9e25
Compare
7db9e25
to
a06db62
Compare
What Does This Do
CyclicBarrier
andCountDownLatch
waiting time, regardless of whether there is an active traceMotivation
More complete wallclock profiling
Additional Notes
Jira ticket: PROF-9658