-
Notifications
You must be signed in to change notification settings - Fork 292
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
Remove old unused code #8343
Merged
Merged
Remove old unused code #8343
Conversation
This file contains 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
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 52 metrics, 11 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.47.0-SNAPSHOT~d3c54a213e, baseline=1.47.0-SNAPSHOT~88f59f8817
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.043 s) : 0, 1043317
Total [baseline] (10.456 s) : 0, 10455788
Agent [candidate] (1.045 s) : 0, 1045054
Total [candidate] (10.591 s) : 0, 10590725
section appsec
Agent [baseline] (1.188 s) : 0, 1188473
Total [baseline] (10.761 s) : 0, 10760653
Agent [candidate] (1.188 s) : 0, 1188091
Total [candidate] (10.723 s) : 0, 10722779
section iast
Agent [baseline] (1.175 s) : 0, 1174550
Total [baseline] (10.964 s) : 0, 10964348
Agent [candidate] (1.18 s) : 0, 1180129
Total [candidate] (11.056 s) : 0, 11055894
section profiling
Agent [baseline] (1.264 s) : 0, 1263856
Total [baseline] (10.863 s) : 0, 10862530
Agent [candidate] (1.263 s) : 0, 1263276
Total [candidate] (10.839 s) : 0, 10838525
gantt
title petclinic - break down per module: candidate=1.47.0-SNAPSHOT~d3c54a213e, baseline=1.47.0-SNAPSHOT~88f59f8817
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (716.206 ms) : 0, 716206
BytebuddyAgent [candidate] (716.298 ms) : 0, 716298
GlobalTracer [baseline] (242.907 ms) : 0, 242907
GlobalTracer [candidate] (243.929 ms) : 0, 243929
AppSec [baseline] (55.222 ms) : 0, 55222
AppSec [candidate] (55.314 ms) : 0, 55314
Remote Config [baseline] (720.953 µs) : 0, 721
Remote Config [candidate] (716.376 µs) : 0, 716
Telemetry [baseline] (13.082 ms) : 0, 13082
Telemetry [candidate] (13.611 ms) : 0, 13611
section appsec
BytebuddyAgent [baseline] (735.637 ms) : 0, 735637
BytebuddyAgent [candidate] (735.086 ms) : 0, 735086
GlobalTracer [baseline] (241.051 ms) : 0, 241051
GlobalTracer [candidate] (241.266 ms) : 0, 241266
AppSec [baseline] (176.382 ms) : 0, 176382
AppSec [candidate] (176.279 ms) : 0, 176279
Remote Config [baseline] (659.392 µs) : 0, 659
Remote Config [candidate] (666.752 µs) : 0, 667
Telemetry [baseline] (8.687 ms) : 0, 8687
Telemetry [candidate] (8.704 ms) : 0, 8704
IAST [baseline] (21.733 ms) : 0, 21733
IAST [candidate] (21.736 ms) : 0, 21736
section iast
BytebuddyAgent [baseline] (836.116 ms) : 0, 836116
BytebuddyAgent [candidate] (840.589 ms) : 0, 840589
GlobalTracer [baseline] (234.252 ms) : 0, 234252
GlobalTracer [candidate] (234.746 ms) : 0, 234746
AppSec [baseline] (56.815 ms) : 0, 56815
AppSec [candidate] (54.17 ms) : 0, 54170
Remote Config [baseline] (618.73 µs) : 0, 619
Remote Config [candidate] (609.064 µs) : 0, 609
Telemetry [baseline] (8.668 ms) : 0, 8668
Telemetry [candidate] (8.828 ms) : 0, 8828
IAST [baseline] (22.802 ms) : 0, 22802
IAST [candidate] (25.847 ms) : 0, 25847
section profiling
BytebuddyAgent [baseline] (708.27 ms) : 0, 708270
BytebuddyAgent [candidate] (707.669 ms) : 0, 707669
GlobalTracer [baseline] (352.716 ms) : 0, 352716
GlobalTracer [candidate] (353.847 ms) : 0, 353847
AppSec [baseline] (55.235 ms) : 0, 55235
AppSec [candidate] (54.159 ms) : 0, 54159
Remote Config [baseline] (704.505 µs) : 0, 705
Remote Config [candidate] (690.403 µs) : 0, 690
Telemetry [baseline] (8.862 ms) : 0, 8862
Telemetry [candidate] (8.894 ms) : 0, 8894
ProfilingAgent [baseline] (95.835 ms) : 0, 95835
ProfilingAgent [candidate] (95.655 ms) : 0, 95655
Profiling [baseline] (95.86 ms) : 0, 95860
Profiling [candidate] (95.68 ms) : 0, 95680
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.47.0-SNAPSHOT~d3c54a213e, baseline=1.47.0-SNAPSHOT~88f59f8817
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.041 s) : 0, 1041418
Total [baseline] (8.627 s) : 0, 8626866
Agent [candidate] (1.044 s) : 0, 1043934
Total [candidate] (8.638 s) : 0, 8638389
section iast
Agent [baseline] (1.185 s) : 0, 1185214
Total [baseline] (9.206 s) : 0, 9206464
Agent [candidate] (1.188 s) : 0, 1188054
Total [candidate] (9.269 s) : 0, 9269061
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.18 s) : 0, 1179539
Total [baseline] (9.201 s) : 0, 9201237
Agent [candidate] (1.17 s) : 0, 1169536
Total [candidate] (9.2 s) : 0, 9199925
section iast_TELEMETRY_OFF
Agent [baseline] (1.177 s) : 0, 1176719
Total [baseline] (9.204 s) : 0, 9203565
Agent [candidate] (1.171 s) : 0, 1170904
Total [candidate] (9.237 s) : 0, 9237443
gantt
title insecure-bank - break down per module: candidate=1.47.0-SNAPSHOT~d3c54a213e, baseline=1.47.0-SNAPSHOT~88f59f8817
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (716.082 ms) : 0, 716082
BytebuddyAgent [candidate] (715.717 ms) : 0, 715717
GlobalTracer [baseline] (242.834 ms) : 0, 242834
GlobalTracer [candidate] (243.329 ms) : 0, 243329
AppSec [baseline] (55.272 ms) : 0, 55272
AppSec [candidate] (55.22 ms) : 0, 55220
Remote Config [baseline] (713.553 µs) : 0, 714
Remote Config [candidate] (718.932 µs) : 0, 719
Telemetry [baseline] (11.452 ms) : 0, 11452
Telemetry [candidate] (13.707 ms) : 0, 13707
section iast
BytebuddyAgent [baseline] (843.599 ms) : 0, 843599
BytebuddyAgent [candidate] (847.061 ms) : 0, 847061
GlobalTracer [baseline] (236.159 ms) : 0, 236159
GlobalTracer [candidate] (236.196 ms) : 0, 236196
AppSec [baseline] (57.496 ms) : 0, 57496
AppSec [candidate] (55.506 ms) : 0, 55506
Remote Config [baseline] (621.563 µs) : 0, 622
Remote Config [candidate] (647.271 µs) : 0, 647
Telemetry [baseline] (8.763 ms) : 0, 8763
Telemetry [candidate] (8.82 ms) : 0, 8820
IAST [baseline] (23.269 ms) : 0, 23269
IAST [candidate] (24.367 ms) : 0, 24367
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (839.966 ms) : 0, 839966
BytebuddyAgent [candidate] (832.779 ms) : 0, 832779
GlobalTracer [baseline] (234.668 ms) : 0, 234668
GlobalTracer [candidate] (232.862 ms) : 0, 232862
AppSec [baseline] (57.142 ms) : 0, 57142
AppSec [candidate] (54.64 ms) : 0, 54640
Remote Config [baseline] (637.192 µs) : 0, 637
Remote Config [candidate] (608.49 µs) : 0, 608
Telemetry [baseline] (8.794 ms) : 0, 8794
Telemetry [candidate] (8.653 ms) : 0, 8653
IAST [baseline] (23.011 ms) : 0, 23011
IAST [candidate] (24.859 ms) : 0, 24859
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (837.839 ms) : 0, 837839
BytebuddyAgent [candidate] (832.227 ms) : 0, 832227
GlobalTracer [baseline] (234.84 ms) : 0, 234840
GlobalTracer [candidate] (234.178 ms) : 0, 234178
AppSec [baseline] (54.721 ms) : 0, 54721
AppSec [candidate] (55.153 ms) : 0, 55153
Remote Config [baseline] (621.165 µs) : 0, 621
Remote Config [candidate] (635.143 µs) : 0, 635
Telemetry [baseline] (8.618 ms) : 0, 8618
Telemetry [candidate] (8.76 ms) : 0, 8760
IAST [baseline] (24.743 ms) : 0, 24743
IAST [candidate] (24.635 ms) : 0, 24635
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 insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.47.0-SNAPSHOT~d3c54a213e, baseline=1.47.0-SNAPSHOT~88f59f8817
dateFormat X
axisFormat %s
section baseline
no_agent (378.596 µs) : 358, 399
. : milestone, 379,
iast (510.671 µs) : 489, 532
. : milestone, 511,
iast_FULL (749.346 µs) : 727, 771
. : milestone, 749,
iast_GLOBAL (553.675 µs) : 532, 576
. : milestone, 554,
iast_HARDCODED_SECRET_DISABLED (507.93 µs) : 486, 529
. : milestone, 508,
iast_INACTIVE (462.613 µs) : 441, 484
. : milestone, 463,
iast_TELEMETRY_OFF (492.288 µs) : 471, 514
. : milestone, 492,
tracing (453.146 µs) : 433, 474
. : milestone, 453,
section candidate
no_agent (379.736 µs) : 359, 400
. : milestone, 380,
iast (507.061 µs) : 485, 529
. : milestone, 507,
iast_FULL (738.941 µs) : 717, 761
. : milestone, 739,
iast_GLOBAL (553.887 µs) : 532, 576
. : milestone, 554,
iast_HARDCODED_SECRET_DISABLED (506.591 µs) : 485, 528
. : milestone, 507,
iast_INACTIVE (458.453 µs) : 437, 480
. : milestone, 458,
iast_TELEMETRY_OFF (493.758 µs) : 472, 516
. : milestone, 494,
tracing (452.882 µs) : 432, 474
. : milestone, 453,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.47.0-SNAPSHOT~d3c54a213e, baseline=1.47.0-SNAPSHOT~88f59f8817
dateFormat X
axisFormat %s
section baseline
no_agent (1.364 ms) : 1344, 1384
. : milestone, 1364,
appsec (1.74 ms) : 1717, 1763
. : milestone, 1740,
appsec_no_iast (1.754 ms) : 1729, 1778
. : milestone, 1754,
iast (1.505 ms) : 1480, 1529
. : milestone, 1505,
profiling (1.544 ms) : 1519, 1568
. : milestone, 1544,
tracing (1.482 ms) : 1456, 1508
. : milestone, 1482,
section candidate
no_agent (1.363 ms) : 1343, 1382
. : milestone, 1363,
appsec (1.753 ms) : 1729, 1777
. : milestone, 1753,
appsec_no_iast (1.757 ms) : 1733, 1781
. : milestone, 1757,
iast (1.503 ms) : 1479, 1528
. : milestone, 1503,
profiling (1.517 ms) : 1494, 1540
. : milestone, 1517,
tracing (1.487 ms) : 1463, 1511
. : milestone, 1487,
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.47.0-SNAPSHOT~d3c54a213e, baseline=1.47.0-SNAPSHOT~88f59f8817
dateFormat X
axisFormat %s
section baseline
no_agent (1.467 ms) : 1456, 1479
. : milestone, 1467,
appsec (2.34 ms) : 2297, 2383
. : milestone, 2340,
iast (2.104 ms) : 2049, 2159
. : milestone, 2104,
iast_GLOBAL (2.148 ms) : 2093, 2203
. : milestone, 2148,
profiling (2.459 ms) : 2268, 2649
. : milestone, 2459,
tracing (1.924 ms) : 1882, 1965
. : milestone, 1924,
section candidate
no_agent (1.468 ms) : 1456, 1479
. : milestone, 1468,
appsec (2.361 ms) : 2317, 2404
. : milestone, 2361,
iast (2.097 ms) : 2043, 2152
. : milestone, 2097,
iast_GLOBAL (2.144 ms) : 2089, 2199
. : milestone, 2144,
profiling (2.448 ms) : 2259, 2637
. : milestone, 2448,
tracing (1.939 ms) : 1897, 1982
. : milestone, 1939,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.47.0-SNAPSHOT~d3c54a213e, baseline=1.47.0-SNAPSHOT~88f59f8817
dateFormat X
axisFormat %s
section baseline
no_agent (14.875 s) : 14875000, 14875000
. : milestone, 14875000,
appsec (15.206 s) : 15206000, 15206000
. : milestone, 15206000,
iast (18.106 s) : 18106000, 18106000
. : milestone, 18106000,
iast_GLOBAL (18.096 s) : 18096000, 18096000
. : milestone, 18096000,
profiling (15.537 s) : 15537000, 15537000
. : milestone, 15537000,
tracing (15.002 s) : 15002000, 15002000
. : milestone, 15002000,
section candidate
no_agent (14.996 s) : 14996000, 14996000
. : milestone, 14996000,
appsec (14.924 s) : 14924000, 14924000
. : milestone, 14924000,
iast (18.579 s) : 18579000, 18579000
. : milestone, 18579000,
iast_GLOBAL (18.255 s) : 18255000, 18255000
. : milestone, 18255000,
profiling (15.053 s) : 15053000, 15053000
. : milestone, 15053000,
tracing (15.052 s) : 15052000, 15052000
. : milestone, 15052000,
|
PerfectSlayer
approved these changes
Feb 6, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
comp: profiling
Profiling
tag: no release notes
Changes to exclude from release notes
type: refactoring
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
This is a cleanup of old, unused code related to no longer active concepts of checkpoints, per-span-trace and auxiliary profilers.
Motivation
Code hygiene - avoid bit rotting unused code.
Additional Notes
Contributor Checklist
type:
and (comp:
orinst:
) labels in addition to any usefull labelsclose
,fix
or any linking keywords when referencing an issue.Use
solves
instead, and assign the PR milestone to the issue