-
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
Introduce marker mechanism for eagerly initializing helpers #8028
base: master
Are you sure you want to change the base?
Conversation
…it to register AsyncResultExtensions
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 57 metrics, 6 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.44.0-SNAPSHOT~bb3eabdb03, baseline=1.44.0-SNAPSHOT~207f770623
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.093 s) : 0, 1093040
Total [baseline] (10.475 s) : 0, 10475402
Agent [candidate] (1.09 s) : 0, 1090277
Total [candidate] (10.416 s) : 0, 10416065
section appsec
Agent [baseline] (1.227 s) : 0, 1226507
Total [baseline] (10.742 s) : 0, 10742496
Agent [candidate] (1.23 s) : 0, 1230120
Total [candidate] (10.749 s) : 0, 10748696
section iast
Agent [baseline] (1.217 s) : 0, 1217277
Total [baseline] (10.96 s) : 0, 10960468
Agent [candidate] (1.221 s) : 0, 1220828
Total [candidate] (10.955 s) : 0, 10955277
section profiling
Agent [baseline] (1.314 s) : 0, 1313979
Total [baseline] (10.795 s) : 0, 10795259
Agent [candidate] (1.321 s) : 0, 1321323
Total [candidate] (10.844 s) : 0, 10843844
gantt
title petclinic - break down per module: candidate=1.44.0-SNAPSHOT~bb3eabdb03, baseline=1.44.0-SNAPSHOT~207f770623
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (694.589 ms) : 0, 694589
BytebuddyAgent [candidate] (694.391 ms) : 0, 694391
GlobalTracer [baseline] (317.87 ms) : 0, 317870
GlobalTracer [candidate] (318.149 ms) : 0, 318149
AppSec [baseline] (54.759 ms) : 0, 54759
AppSec [candidate] (54.87 ms) : 0, 54870
Remote Config [baseline] (676.532 µs) : 0, 677
Remote Config [candidate] (677.248 µs) : 0, 677
Telemetry [baseline] (11.422 ms) : 0, 11422
Telemetry [candidate] (8.468 ms) : 0, 8468
section appsec
BytebuddyAgent [baseline] (712.627 ms) : 0, 712627
BytebuddyAgent [candidate] (714.019 ms) : 0, 714019
GlobalTracer [baseline] (314.65 ms) : 0, 314650
GlobalTracer [candidate] (316.292 ms) : 0, 316292
AppSec [baseline] (167.545 ms) : 0, 167545
AppSec [candidate] (166.18 ms) : 0, 166180
Remote Config [baseline] (647.653 µs) : 0, 648
Remote Config [candidate] (645.527 µs) : 0, 646
Telemetry [baseline] (7.812 ms) : 0, 7812
Telemetry [candidate] (9.675 ms) : 0, 9675
IAST [baseline] (18.927 ms) : 0, 18927
IAST [candidate] (19.079 ms) : 0, 19079
section iast
BytebuddyAgent [baseline] (810.949 ms) : 0, 810949
BytebuddyAgent [candidate] (813.308 ms) : 0, 813308
GlobalTracer [baseline] (305.832 ms) : 0, 305832
GlobalTracer [candidate] (306.649 ms) : 0, 306649
AppSec [baseline] (57.853 ms) : 0, 57853
AppSec [candidate] (58.145 ms) : 0, 58145
Remote Config [baseline] (630.079 µs) : 0, 630
Remote Config [candidate] (643.032 µs) : 0, 643
Telemetry [baseline] (7.478 ms) : 0, 7478
Telemetry [candidate] (7.556 ms) : 0, 7556
IAST [baseline] (20.783 ms) : 0, 20783
IAST [candidate] (20.768 ms) : 0, 20768
section profiling
BytebuddyAgent [baseline] (687.274 ms) : 0, 687274
BytebuddyAgent [candidate] (691.774 ms) : 0, 691774
GlobalTracer [baseline] (433.043 ms) : 0, 433043
GlobalTracer [candidate] (434.001 ms) : 0, 434001
AppSec [baseline] (53.448 ms) : 0, 53448
AppSec [candidate] (53.814 ms) : 0, 53814
Remote Config [baseline] (647.637 µs) : 0, 648
Remote Config [candidate] (664.403 µs) : 0, 664
Telemetry [baseline] (7.682 ms) : 0, 7682
Telemetry [candidate] (7.739 ms) : 0, 7739
ProfilingAgent [baseline] (92.85 ms) : 0, 92850
ProfilingAgent [candidate] (93.983 ms) : 0, 93983
Profiling [baseline] (92.874 ms) : 0, 92874
Profiling [candidate] (94.007 ms) : 0, 94007
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.44.0-SNAPSHOT~bb3eabdb03, baseline=1.44.0-SNAPSHOT~207f770623
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.092 s) : 0, 1092201
Total [baseline] (8.666 s) : 0, 8665822
Agent [candidate] (1.09 s) : 0, 1089888
Total [candidate] (8.693 s) : 0, 8692798
section iast
Agent [baseline] (1.225 s) : 0, 1225469
Total [baseline] (9.273 s) : 0, 9273019
Agent [candidate] (1.227 s) : 0, 1226625
Total [candidate] (9.23 s) : 0, 9230394
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.222 s) : 0, 1222087
Total [baseline] (9.166 s) : 0, 9166180
Agent [candidate] (1.227 s) : 0, 1226546
Total [candidate] (9.216 s) : 0, 9216081
section iast_TELEMETRY_OFF
Agent [baseline] (1.229 s) : 0, 1228680
Total [baseline] (9.209 s) : 0, 9208554
Agent [candidate] (1.221 s) : 0, 1221154
Total [candidate] (9.207 s) : 0, 9206671
gantt
title insecure-bank - break down per module: candidate=1.44.0-SNAPSHOT~bb3eabdb03, baseline=1.44.0-SNAPSHOT~207f770623
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (694.317 ms) : 0, 694317
BytebuddyAgent [candidate] (693.681 ms) : 0, 693681
GlobalTracer [baseline] (317.815 ms) : 0, 317815
GlobalTracer [candidate] (317.588 ms) : 0, 317588
AppSec [baseline] (54.872 ms) : 0, 54872
AppSec [candidate] (54.852 ms) : 0, 54852
Remote Config [baseline] (678.645 µs) : 0, 679
Remote Config [candidate] (679.989 µs) : 0, 680
Telemetry [baseline] (10.719 ms) : 0, 10719
Telemetry [candidate] (9.329 ms) : 0, 9329
section iast
BytebuddyAgent [baseline] (815.8 ms) : 0, 815800
BytebuddyAgent [candidate] (817.627 ms) : 0, 817627
GlobalTracer [baseline] (307.656 ms) : 0, 307656
GlobalTracer [candidate] (307.698 ms) : 0, 307698
AppSec [baseline] (57.798 ms) : 0, 57798
AppSec [candidate] (56.704 ms) : 0, 56704
IAST [baseline] (22.089 ms) : 0, 22089
IAST [candidate] (22.591 ms) : 0, 22591
Remote Config [baseline] (649.05 µs) : 0, 649
Remote Config [candidate] (632.287 µs) : 0, 632
Telemetry [baseline] (7.652 ms) : 0, 7652
Telemetry [candidate] (7.51 ms) : 0, 7510
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (813.025 ms) : 0, 813025
BytebuddyAgent [candidate] (816.929 ms) : 0, 816929
GlobalTracer [baseline] (307.916 ms) : 0, 307916
GlobalTracer [candidate] (308.126 ms) : 0, 308126
AppSec [baseline] (58.258 ms) : 0, 58258
AppSec [candidate] (58.205 ms) : 0, 58205
IAST [baseline] (20.915 ms) : 0, 20915
IAST [candidate] (21.087 ms) : 0, 21087
Remote Config [baseline] (636.331 µs) : 0, 636
Remote Config [candidate] (640.025 µs) : 0, 640
Telemetry [baseline] (7.588 ms) : 0, 7588
Telemetry [candidate] (7.55 ms) : 0, 7550
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (816.26 ms) : 0, 816260
BytebuddyAgent [candidate] (811.57 ms) : 0, 811570
GlobalTracer [baseline] (310.553 ms) : 0, 310553
GlobalTracer [candidate] (308.732 ms) : 0, 308732
AppSec [baseline] (56.558 ms) : 0, 56558
AppSec [candidate] (58.323 ms) : 0, 58323
IAST [baseline] (23.365 ms) : 0, 23365
IAST [candidate] (20.602 ms) : 0, 20602
Remote Config [baseline] (625.709 µs) : 0, 626
Remote Config [candidate] (636.634 µs) : 0, 637
Telemetry [baseline] (7.491 ms) : 0, 7491
Telemetry [candidate] (7.512 ms) : 0, 7512
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 16 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.44.0-SNAPSHOT~bb3eabdb03, baseline=1.44.0-SNAPSHOT~207f770623
dateFormat X
axisFormat %s
section baseline
no_agent (374.409 µs) : 354, 395
. : milestone, 374,
iast (503.693 µs) : 482, 526
. : milestone, 504,
iast_FULL (659.945 µs) : 638, 681
. : milestone, 660,
iast_GLOBAL (529.286 µs) : 507, 552
. : milestone, 529,
iast_HARDCODED_SECRET_DISABLED (494.182 µs) : 473, 515
. : milestone, 494,
iast_INACTIVE (462.771 µs) : 441, 484
. : milestone, 463,
iast_TELEMETRY_OFF (487.857 µs) : 466, 509
. : milestone, 488,
tracing (457.294 µs) : 436, 478
. : milestone, 457,
section candidate
no_agent (376.36 µs) : 357, 396
. : milestone, 376,
iast (505.46 µs) : 484, 527
. : milestone, 505,
iast_FULL (658.899 µs) : 637, 681
. : milestone, 659,
iast_GLOBAL (532.068 µs) : 510, 554
. : milestone, 532,
iast_HARDCODED_SECRET_DISABLED (494.235 µs) : 473, 516
. : milestone, 494,
iast_INACTIVE (467.843 µs) : 446, 489
. : milestone, 468,
iast_TELEMETRY_OFF (491.389 µs) : 470, 513
. : milestone, 491,
tracing (455.332 µs) : 435, 476
. : milestone, 455,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.44.0-SNAPSHOT~bb3eabdb03, baseline=1.44.0-SNAPSHOT~207f770623
dateFormat X
axisFormat %s
section baseline
no_agent (1.372 ms) : 1353, 1392
. : milestone, 1372,
appsec (1.771 ms) : 1746, 1795
. : milestone, 1771,
appsec_no_iast (1.77 ms) : 1746, 1794
. : milestone, 1770,
iast (1.506 ms) : 1483, 1529
. : milestone, 1506,
profiling (1.56 ms) : 1536, 1584
. : milestone, 1560,
tracing (1.5 ms) : 1475, 1524
. : milestone, 1500,
section candidate
no_agent (1.362 ms) : 1342, 1382
. : milestone, 1362,
appsec (1.762 ms) : 1737, 1787
. : milestone, 1762,
appsec_no_iast (1.767 ms) : 1742, 1791
. : milestone, 1767,
iast (1.514 ms) : 1490, 1537
. : milestone, 1514,
profiling (1.538 ms) : 1515, 1562
. : milestone, 1538,
tracing (1.489 ms) : 1464, 1514
. : milestone, 1489,
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.44.0-SNAPSHOT~bb3eabdb03, baseline=1.44.0-SNAPSHOT~207f770623
dateFormat X
axisFormat %s
section baseline
no_agent (14.698 s) : 14698000, 14698000
. : milestone, 14698000,
appsec (15.056 s) : 15056000, 15056000
. : milestone, 15056000,
iast (18.762 s) : 18762000, 18762000
. : milestone, 18762000,
iast_GLOBAL (18.06 s) : 18060000, 18060000
. : milestone, 18060000,
profiling (14.864 s) : 14864000, 14864000
. : milestone, 14864000,
tracing (15.096 s) : 15096000, 15096000
. : milestone, 15096000,
section candidate
no_agent (15.419 s) : 15419000, 15419000
. : milestone, 15419000,
appsec (14.961 s) : 14961000, 14961000
. : milestone, 14961000,
iast (18.994 s) : 18994000, 18994000
. : milestone, 18994000,
iast_GLOBAL (17.691 s) : 17691000, 17691000
. : milestone, 17691000,
profiling (15.056 s) : 15056000, 15056000
. : milestone, 15056000,
tracing (14.899 s) : 14899000, 14899000
. : milestone, 14899000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.44.0-SNAPSHOT~bb3eabdb03, baseline=1.44.0-SNAPSHOT~207f770623
dateFormat X
axisFormat %s
section baseline
no_agent (1.467 ms) : 1455, 1478
. : milestone, 1467,
appsec (2.33 ms) : 2289, 2372
. : milestone, 2330,
iast (2.078 ms) : 2026, 2130
. : milestone, 2078,
iast_GLOBAL (2.116 ms) : 2064, 2169
. : milestone, 2116,
profiling (1.933 ms) : 1892, 1975
. : milestone, 1933,
tracing (1.919 ms) : 1879, 1959
. : milestone, 1919,
section candidate
no_agent (1.465 ms) : 1454, 1477
. : milestone, 1465,
appsec (2.334 ms) : 2292, 2375
. : milestone, 2334,
iast (2.071 ms) : 2019, 2124
. : milestone, 2071,
iast_GLOBAL (2.118 ms) : 2065, 2170
. : milestone, 2118,
profiling (1.949 ms) : 1905, 1992
. : milestone, 1949,
tracing (1.92 ms) : 1880, 1960
. : milestone, 1920,
|
.getClass() | ||
.getName() | ||
.endsWith("ThrowawayClassLoader")) { | ||
return; // spring-native expects this to be thrown away, not persisted |
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.
what's the story behind that?
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.
thank you Stuart
What Does This Do
Builds on existing helper mechanism, eager helpers must declare a public static "init" method that takes no arguments.
Motivation
Use eager helper mechanism to register AsyncResultExtensions as we inject them, so we don't need to instrument lots of different constructors / methods.
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 issueJira ticket: AIDM-447