-
Notifications
You must be signed in to change notification settings - Fork 318
Prefer using activeSpan() over activeScope() #8492
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
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 67 metrics, 4 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.47.0-SNAPSHOT~2fc7d24f08, baseline=1.47.0-SNAPSHOT~ebb0e55290
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.152 s) : 0, 1152096
Total [baseline] (9.379 s) : 0, 9378984
Agent [candidate] (1.134 s) : 0, 1134198
Total [candidate] (9.252 s) : 0, 9251689
section iast
Agent [baseline] (1.257 s) : 0, 1257469
Total [baseline] (9.857 s) : 0, 9856736
Agent [candidate] (1.258 s) : 0, 1258155
Total [candidate] (9.885 s) : 0, 9884598
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.256 s) : 0, 1256188
Total [baseline] (9.843 s) : 0, 9842739
Agent [candidate] (1.267 s) : 0, 1266996
Total [candidate] (9.884 s) : 0, 9883551
section iast_TELEMETRY_OFF
Agent [baseline] (1.251 s) : 0, 1250666
Total [baseline] (9.866 s) : 0, 9866104
Agent [candidate] (1.253 s) : 0, 1253249
Total [candidate] (9.909 s) : 0, 9909236
gantt
title insecure-bank - break down per module: candidate=1.47.0-SNAPSHOT~2fc7d24f08, baseline=1.47.0-SNAPSHOT~ebb0e55290
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (728.004 ms) : 0, 728004
BytebuddyAgent [candidate] (715.633 ms) : 0, 715633
GlobalTracer [baseline] (243.627 ms) : 0, 243627
GlobalTracer [candidate] (239.614 ms) : 0, 239614
AppSec [baseline] (56.391 ms) : 0, 56391
AppSec [candidate] (55.382 ms) : 0, 55382
Debugger [baseline] (95.671 ms) : 0, 95671
Debugger [candidate] (95.492 ms) : 0, 95492
Remote Config [baseline] (588.84 µs) : 0, 589
Remote Config [candidate] (586.63 µs) : 0, 587
Telemetry [baseline] (12.579 ms) : 0, 12579
Telemetry [candidate] (12.501 ms) : 0, 12501
section iast
BytebuddyAgent [baseline] (836.133 ms) : 0, 836133
BytebuddyAgent [candidate] (835.751 ms) : 0, 835751
GlobalTracer [baseline] (230.258 ms) : 0, 230258
GlobalTracer [candidate] (230.712 ms) : 0, 230712
IAST [baseline] (22.623 ms) : 0, 22623
IAST [candidate] (22.758 ms) : 0, 22758
AppSec [baseline] (57.228 ms) : 0, 57228
AppSec [candidate] (57.187 ms) : 0, 57187
Debugger [baseline] (83.458 ms) : 0, 83458
Debugger [candidate] (83.359 ms) : 0, 83359
Remote Config [baseline] (513.48 µs) : 0, 513
Remote Config [candidate] (517.115 µs) : 0, 517
Telemetry [baseline] (12.251 ms) : 0, 12251
Telemetry [candidate] (12.852 ms) : 0, 12852
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (833.431 ms) : 0, 833431
BytebuddyAgent [candidate] (840.953 ms) : 0, 840953
GlobalTracer [baseline] (230.309 ms) : 0, 230309
GlobalTracer [candidate] (231.473 ms) : 0, 231473
IAST [baseline] (22.757 ms) : 0, 22757
IAST [candidate] (23.209 ms) : 0, 23209
AppSec [baseline] (56.92 ms) : 0, 56920
AppSec [candidate] (57.402 ms) : 0, 57402
Debugger [baseline] (83.734 ms) : 0, 83734
Debugger [candidate] (84.045 ms) : 0, 84045
Remote Config [baseline] (511.825 µs) : 0, 512
Remote Config [candidate] (526.666 µs) : 0, 527
Telemetry [baseline] (13.537 ms) : 0, 13537
Telemetry [candidate] (14.235 ms) : 0, 14235
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (830.934 ms) : 0, 830934
BytebuddyAgent [candidate] (831.748 ms) : 0, 831748
GlobalTracer [baseline] (230.169 ms) : 0, 230169
GlobalTracer [candidate] (230.744 ms) : 0, 230744
IAST [baseline] (24.335 ms) : 0, 24335
IAST [candidate] (22.317 ms) : 0, 22317
AppSec [baseline] (54.452 ms) : 0, 54452
AppSec [candidate] (57.207 ms) : 0, 57207
Debugger [baseline] (83.259 ms) : 0, 83259
Debugger [candidate] (83.687 ms) : 0, 83687
Remote Config [baseline] (512.321 µs) : 0, 512
Remote Config [candidate] (514.383 µs) : 0, 514
Telemetry [baseline] (12.085 ms) : 0, 12085
Telemetry [candidate] (12.0 ms) : 0, 12000
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.47.0-SNAPSHOT~2fc7d24f08, baseline=1.47.0-SNAPSHOT~ebb0e55290
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.137 s) : 0, 1136687
Total [baseline] (11.454 s) : 0, 11454064
Agent [candidate] (1.141 s) : 0, 1140582
Total [candidate] (11.508 s) : 0, 11507549
section appsec
Agent [baseline] (1.273 s) : 0, 1272776
Total [baseline] (11.635 s) : 0, 11634958
Agent [candidate] (1.272 s) : 0, 1272466
Total [candidate] (11.7 s) : 0, 11700284
section iast
Agent [baseline] (1.268 s) : 0, 1267967
Total [baseline] (12.003 s) : 0, 12003097
Agent [candidate] (1.261 s) : 0, 1261083
Total [candidate] (11.976 s) : 0, 11976429
section profiling
Agent [baseline] (1.389 s) : 0, 1389467
Total [baseline] (11.719 s) : 0, 11719374
Agent [candidate] (1.38 s) : 0, 1379718
Total [candidate] (11.862 s) : 0, 11862020
gantt
title petclinic - break down per module: candidate=1.47.0-SNAPSHOT~2fc7d24f08, baseline=1.47.0-SNAPSHOT~ebb0e55290
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (718.471 ms) : 0, 718471
BytebuddyAgent [candidate] (720.891 ms) : 0, 720891
GlobalTracer [baseline] (239.379 ms) : 0, 239379
GlobalTracer [candidate] (240.245 ms) : 0, 240245
AppSec [baseline] (55.339 ms) : 0, 55339
AppSec [candidate] (55.873 ms) : 0, 55873
Debugger [baseline] (95.335 ms) : 0, 95335
Debugger [candidate] (95.293 ms) : 0, 95293
Remote Config [baseline] (594.036 µs) : 0, 594
Remote Config [candidate] (584.621 µs) : 0, 585
Telemetry [baseline] (12.485 ms) : 0, 12485
Telemetry [candidate] (12.634 ms) : 0, 12634
section appsec
BytebuddyAgent [baseline] (734.942 ms) : 0, 734942
BytebuddyAgent [candidate] (734.669 ms) : 0, 734669
GlobalTracer [baseline] (236.968 ms) : 0, 236968
GlobalTracer [candidate] (236.462 ms) : 0, 236462
AppSec [baseline] (177.603 ms) : 0, 177603
AppSec [candidate] (178.302 ms) : 0, 178302
Debugger [baseline] (84.576 ms) : 0, 84576
Debugger [candidate] (84.28 ms) : 0, 84280
Remote Config [baseline] (636.218 µs) : 0, 636
Remote Config [candidate] (630.619 µs) : 0, 631
Telemetry [baseline] (12.311 ms) : 0, 12311
Telemetry [candidate] (12.368 ms) : 0, 12368
IAST [baseline] (21.429 ms) : 0, 21429
IAST [candidate] (21.368 ms) : 0, 21368
section iast
BytebuddyAgent [baseline] (842.04 ms) : 0, 842040
BytebuddyAgent [candidate] (837.565 ms) : 0, 837565
GlobalTracer [baseline] (231.964 ms) : 0, 231964
GlobalTracer [candidate] (230.914 ms) : 0, 230914
AppSec [baseline] (57.454 ms) : 0, 57454
AppSec [candidate] (57.091 ms) : 0, 57091
Debugger [baseline] (84.028 ms) : 0, 84028
Debugger [candidate] (83.709 ms) : 0, 83709
Remote Config [baseline] (518.118 µs) : 0, 518
Remote Config [candidate] (510.664 µs) : 0, 511
Telemetry [baseline] (13.623 ms) : 0, 13623
Telemetry [candidate] (13.467 ms) : 0, 13467
IAST [baseline] (23.102 ms) : 0, 23102
IAST [candidate] (22.619 ms) : 0, 22619
section profiling
BytebuddyAgent [baseline] (714.648 ms) : 0, 714648
BytebuddyAgent [candidate] (708.954 ms) : 0, 708954
GlobalTracer [baseline] (353.043 ms) : 0, 353043
GlobalTracer [candidate] (351.061 ms) : 0, 351061
AppSec [baseline] (55.717 ms) : 0, 55717
AppSec [candidate] (54.082 ms) : 0, 54082
Debugger [baseline] (88.708 ms) : 0, 88708
Debugger [candidate] (88.001 ms) : 0, 88001
Remote Config [baseline] (617.972 µs) : 0, 618
Remote Config [candidate] (617.644 µs) : 0, 618
Telemetry [baseline] (12.655 ms) : 0, 12655
Telemetry [candidate] (12.622 ms) : 0, 12622
ProfilingAgent [baseline] (123.301 ms) : 0, 123301
ProfilingAgent [candidate] (123.951 ms) : 0, 123951
Profiling [baseline] (123.321 ms) : 0, 123321
Profiling [candidate] (123.972 ms) : 0, 123972
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 14 metrics, 16 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.47.0-SNAPSHOT~2fc7d24f08, baseline=1.47.0-SNAPSHOT~ebb0e55290
dateFormat X
axisFormat %s
section baseline
no_agent (1.37 ms) : 1350, 1390
. : milestone, 1370,
appsec (1.741 ms) : 1717, 1765
. : milestone, 1741,
appsec_no_iast (1.72 ms) : 1696, 1745
. : milestone, 1720,
code_origins (1.703 ms) : 1670, 1735
. : milestone, 1703,
iast (1.533 ms) : 1509, 1558
. : milestone, 1533,
profiling (1.508 ms) : 1483, 1533
. : milestone, 1508,
tracing (1.507 ms) : 1484, 1531
. : milestone, 1507,
section candidate
no_agent (1.362 ms) : 1343, 1382
. : milestone, 1362,
appsec (1.74 ms) : 1716, 1764
. : milestone, 1740,
appsec_no_iast (1.757 ms) : 1733, 1781
. : milestone, 1757,
code_origins (1.695 ms) : 1662, 1727
. : milestone, 1695,
iast (1.545 ms) : 1520, 1571
. : milestone, 1545,
profiling (1.534 ms) : 1510, 1558
. : milestone, 1534,
tracing (1.503 ms) : 1480, 1526
. : milestone, 1503,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.47.0-SNAPSHOT~2fc7d24f08, baseline=1.47.0-SNAPSHOT~ebb0e55290
dateFormat X
axisFormat %s
section baseline
no_agent (386.671 µs) : 366, 407
. : milestone, 387,
iast (518.83 µs) : 497, 541
. : milestone, 519,
iast_FULL (741.861 µs) : 720, 764
. : milestone, 742,
iast_GLOBAL (572.526 µs) : 551, 594
. : milestone, 573,
iast_HARDCODED_SECRET_DISABLED (524.131 µs) : 502, 547
. : milestone, 524,
iast_INACTIVE (469.831 µs) : 448, 492
. : milestone, 470,
iast_TELEMETRY_OFF (504.821 µs) : 483, 527
. : milestone, 505,
tracing (462.707 µs) : 442, 484
. : milestone, 463,
section candidate
no_agent (385.758 µs) : 365, 406
. : milestone, 386,
iast (521.278 µs) : 500, 543
. : milestone, 521,
iast_FULL (739.6 µs) : 717, 762
. : milestone, 740,
iast_GLOBAL (568.756 µs) : 547, 591
. : milestone, 569,
iast_HARDCODED_SECRET_DISABLED (520.34 µs) : 498, 542
. : milestone, 520,
iast_INACTIVE (473.834 µs) : 452, 495
. : milestone, 474,
iast_TELEMETRY_OFF (515.468 µs) : 493, 538
. : milestone, 515,
tracing (464.063 µs) : 443, 485
. : milestone, 464,
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 tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.47.0-SNAPSHOT~2fc7d24f08, baseline=1.47.0-SNAPSHOT~ebb0e55290
dateFormat X
axisFormat %s
section baseline
no_agent (1.474 ms) : 1462, 1486
. : milestone, 1474,
appsec (2.344 ms) : 2300, 2388
. : milestone, 2344,
iast (2.118 ms) : 2062, 2173
. : milestone, 2118,
iast_GLOBAL (2.162 ms) : 2106, 2217
. : milestone, 2162,
profiling (1.981 ms) : 1936, 2025
. : milestone, 1981,
tracing (1.957 ms) : 1915, 2000
. : milestone, 1957,
section candidate
no_agent (1.474 ms) : 1462, 1485
. : milestone, 1474,
appsec (2.342 ms) : 2298, 2386
. : milestone, 2342,
iast (2.113 ms) : 2058, 2169
. : milestone, 2113,
iast_GLOBAL (2.163 ms) : 2107, 2219
. : milestone, 2163,
profiling (1.969 ms) : 1925, 2012
. : milestone, 1969,
tracing (1.948 ms) : 1906, 1990
. : milestone, 1948,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.47.0-SNAPSHOT~2fc7d24f08, baseline=1.47.0-SNAPSHOT~ebb0e55290
dateFormat X
axisFormat %s
section baseline
no_agent (14.833 s) : 14833000, 14833000
. : milestone, 14833000,
appsec (14.981 s) : 14981000, 14981000
. : milestone, 14981000,
iast (18.929 s) : 18929000, 18929000
. : milestone, 18929000,
iast_GLOBAL (18.143 s) : 18143000, 18143000
. : milestone, 18143000,
profiling (15.087 s) : 15087000, 15087000
. : milestone, 15087000,
tracing (14.772 s) : 14772000, 14772000
. : milestone, 14772000,
section candidate
no_agent (14.941 s) : 14941000, 14941000
. : milestone, 14941000,
appsec (14.762 s) : 14762000, 14762000
. : milestone, 14762000,
iast (18.911 s) : 18911000, 18911000
. : milestone, 18911000,
iast_GLOBAL (17.764 s) : 17764000, 17764000
. : milestone, 17764000,
profiling (15.093 s) : 15093000, 15093000
. : milestone, 15093000,
tracing (15.048 s) : 15048000, 15048000
. : milestone, 15048000,
|
PerfectSlayer
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.
Looking good. Thanks for the refactoring 👍
Motivation
Continuation of #8470 - most of the time we're only interested in whether there's an active span - we don't need the actual closeable handle.
Contributor Checklist
type:and (comp:orinst:) labels in addition to any usefull labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: APMAPI-956