-
Notifications
You must be signed in to change notification settings - Fork 320
Keep generic signature aligned when adding TaskWrapper interface
#9794
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
|
🎯 Code Coverage 🔗 Commit SHA: 6939eba | Docs | Was this helpful? Give us feedback! |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 59 metrics, 6 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.55.0-SNAPSHOT~6939eba0aa, baseline=1.55.0-SNAPSHOT~c60b3c3cbf
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.016 s) : 0, 1016017
Total [baseline] (8.665 s) : 0, 8665272
Agent [candidate] (1.02 s) : 0, 1019611
Total [candidate] (8.643 s) : 0, 8643236
section iast
Agent [baseline] (1.152 s) : 0, 1151508
Total [baseline] (9.27 s) : 0, 9269874
Agent [candidate] (1.16 s) : 0, 1159990
Total [candidate] (9.281 s) : 0, 9281294
gantt
title insecure-bank - break down per module: candidate=1.55.0-SNAPSHOT~6939eba0aa, baseline=1.55.0-SNAPSHOT~c60b3c3cbf
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.46 ms) : 0, 1460
crashtracking [candidate] (1.472 ms) : 0, 1472
BytebuddyAgent [baseline] (691.693 ms) : 0, 691693
BytebuddyAgent [candidate] (694.29 ms) : 0, 694290
GlobalTracer [baseline] (242.366 ms) : 0, 242366
GlobalTracer [candidate] (243.063 ms) : 0, 243063
AppSec [baseline] (32.285 ms) : 0, 32285
AppSec [candidate] (32.413 ms) : 0, 32413
Debugger [baseline] (6.342 ms) : 0, 6342
Debugger [candidate] (6.327 ms) : 0, 6327
Remote Config [baseline] (679.456 µs) : 0, 679
Remote Config [candidate] (671.809 µs) : 0, 672
Telemetry [baseline] (9.3 ms) : 0, 9300
Telemetry [candidate] (9.287 ms) : 0, 9287
Flare Poller [baseline] (10.897 ms) : 0, 10897
Flare Poller [candidate] (10.986 ms) : 0, 10986
section iast
crashtracking [baseline] (1.473 ms) : 0, 1473
crashtracking [candidate] (1.489 ms) : 0, 1489
BytebuddyAgent [baseline] (815.427 ms) : 0, 815427
BytebuddyAgent [candidate] (821.832 ms) : 0, 821832
GlobalTracer [baseline] (231.512 ms) : 0, 231512
GlobalTracer [candidate] (232.947 ms) : 0, 232947
IAST [baseline] (26.713 ms) : 0, 26713
IAST [candidate] (27.031 ms) : 0, 27031
AppSec [baseline] (35.273 ms) : 0, 35273
AppSec [candidate] (35.239 ms) : 0, 35239
Debugger [baseline] (6.159 ms) : 0, 6159
Debugger [candidate] (6.234 ms) : 0, 6234
Remote Config [baseline] (607.581 µs) : 0, 608
Remote Config [candidate] (602.329 µs) : 0, 602
Telemetry [baseline] (8.692 ms) : 0, 8692
Telemetry [candidate] (8.759 ms) : 0, 8759
Flare Poller [baseline] (4.269 ms) : 0, 4269
Flare Poller [candidate] (4.285 ms) : 0, 4285
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.55.0-SNAPSHOT~6939eba0aa, baseline=1.55.0-SNAPSHOT~c60b3c3cbf
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.019 s) : 0, 1019002
Total [baseline] (10.641 s) : 0, 10640906
Agent [candidate] (1.025 s) : 0, 1024962
Total [candidate] (10.807 s) : 0, 10806879
section appsec
Agent [baseline] (1.198 s) : 0, 1198364
Total [baseline] (10.835 s) : 0, 10834814
Agent [candidate] (1.193 s) : 0, 1193085
Total [candidate] (10.831 s) : 0, 10831067
section iast
Agent [baseline] (1.152 s) : 0, 1152444
Total [baseline] (11.136 s) : 0, 11135779
Agent [candidate] (1.151 s) : 0, 1151387
Total [candidate] (11.122 s) : 0, 11121883
section profiling
Agent [baseline] (1.168 s) : 0, 1168303
Total [baseline] (10.818 s) : 0, 10817829
Agent [candidate] (1.166 s) : 0, 1165521
Total [candidate] (10.856 s) : 0, 10856176
gantt
title petclinic - break down per module: candidate=1.55.0-SNAPSHOT~6939eba0aa, baseline=1.55.0-SNAPSHOT~c60b3c3cbf
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.46 ms) : 0, 1460
crashtracking [candidate] (1.483 ms) : 0, 1483
BytebuddyAgent [baseline] (694.364 ms) : 0, 694364
BytebuddyAgent [candidate] (698.2 ms) : 0, 698200
GlobalTracer [baseline] (243.323 ms) : 0, 243323
GlobalTracer [candidate] (243.82 ms) : 0, 243820
AppSec [baseline] (32.278 ms) : 0, 32278
AppSec [candidate] (32.582 ms) : 0, 32582
Debugger [baseline] (6.286 ms) : 0, 6286
Debugger [candidate] (6.366 ms) : 0, 6366
Remote Config [baseline] (672.932 µs) : 0, 673
Remote Config [candidate] (680.681 µs) : 0, 681
Telemetry [baseline] (9.292 ms) : 0, 9292
Telemetry [candidate] (9.444 ms) : 0, 9444
Flare Poller [baseline] (10.212 ms) : 0, 10212
Flare Poller [candidate] (11.224 ms) : 0, 11224
section appsec
crashtracking [baseline] (1.469 ms) : 0, 1469
crashtracking [candidate] (1.462 ms) : 0, 1462
BytebuddyAgent [baseline] (720.422 ms) : 0, 720422
BytebuddyAgent [candidate] (717.571 ms) : 0, 717571
GlobalTracer [baseline] (235.756 ms) : 0, 235756
GlobalTracer [candidate] (234.94 ms) : 0, 234940
AppSec [baseline] (175.023 ms) : 0, 175023
AppSec [candidate] (174.293 ms) : 0, 174293
Debugger [baseline] (6.163 ms) : 0, 6163
Debugger [candidate] (6.035 ms) : 0, 6035
Remote Config [baseline] (634.191 µs) : 0, 634
Remote Config [candidate] (625.244 µs) : 0, 625
Telemetry [baseline] (8.689 ms) : 0, 8689
Telemetry [candidate] (8.489 ms) : 0, 8489
Flare Poller [baseline] (3.998 ms) : 0, 3998
Flare Poller [candidate] (3.881 ms) : 0, 3881
IAST [baseline] (25.079 ms) : 0, 25079
IAST [candidate] (24.72 ms) : 0, 24720
section iast
crashtracking [baseline] (1.462 ms) : 0, 1462
crashtracking [candidate] (1.456 ms) : 0, 1456
BytebuddyAgent [baseline] (815.62 ms) : 0, 815620
BytebuddyAgent [candidate] (815.276 ms) : 0, 815276
GlobalTracer [baseline] (232.159 ms) : 0, 232159
GlobalTracer [candidate] (231.802 ms) : 0, 231802
AppSec [baseline] (35.456 ms) : 0, 35456
AppSec [candidate] (35.028 ms) : 0, 35028
Debugger [baseline] (6.155 ms) : 0, 6155
Debugger [candidate] (6.179 ms) : 0, 6179
Remote Config [baseline] (598.718 µs) : 0, 599
Remote Config [candidate] (609.774 µs) : 0, 610
Telemetry [baseline] (8.707 ms) : 0, 8707
Telemetry [candidate] (8.794 ms) : 0, 8794
Flare Poller [baseline] (4.246 ms) : 0, 4246
Flare Poller [candidate] (4.256 ms) : 0, 4256
IAST [baseline] (26.656 ms) : 0, 26656
IAST [candidate] (26.636 ms) : 0, 26636
section profiling
ProfilingAgent [baseline] (110.059 ms) : 0, 110059
ProfilingAgent [candidate] (109.641 ms) : 0, 109641
crashtracking [baseline] (1.453 ms) : 0, 1453
crashtracking [candidate] (1.463 ms) : 0, 1463
BytebuddyAgent [baseline] (723.055 ms) : 0, 723055
BytebuddyAgent [candidate] (721.897 ms) : 0, 721897
GlobalTracer [baseline] (219.764 ms) : 0, 219764
GlobalTracer [candidate] (218.922 ms) : 0, 218922
AppSec [baseline] (32.538 ms) : 0, 32538
AppSec [candidate] (32.326 ms) : 0, 32326
Debugger [baseline] (6.778 ms) : 0, 6778
Debugger [candidate] (7.479 ms) : 0, 7479
Remote Config [baseline] (710.762 µs) : 0, 711
Remote Config [candidate] (696.536 µs) : 0, 697
Telemetry [baseline] (15.232 ms) : 0, 15232
Telemetry [candidate] (15.115 ms) : 0, 15115
Flare Poller [baseline] (4.963 ms) : 0, 4963
Flare Poller [candidate] (4.127 ms) : 0, 4127
Profiling [baseline] (110.67 ms) : 0, 110670
Profiling [candidate] (110.268 ms) : 0, 110268
LoadParameters
See matching parameters
SummaryFound 5 performance improvements and 3 performance regressions! Performance is the same for 4 metrics, 12 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.55.0-SNAPSHOT~6939eba0aa, baseline=1.55.0-SNAPSHOT~c60b3c3cbf
dateFormat X
axisFormat %s
section baseline
no_agent (4.49 ms) : 4439, 4541
. : milestone, 4490,
iast (10.267 ms) : 10093, 10442
. : milestone, 10267,
iast_FULL (14.941 ms) : 14647, 15235
. : milestone, 14941,
iast_GLOBAL (11.663 ms) : 11453, 11874
. : milestone, 11663,
profiling (8.615 ms) : 8480, 8749
. : milestone, 8615,
tracing (7.725 ms) : 7617, 7834
. : milestone, 7725,
section candidate
no_agent (4.299 ms) : 4245, 4353
. : milestone, 4299,
iast (10.02 ms) : 9850, 10191
. : milestone, 10020,
iast_FULL (14.064 ms) : 13784, 14345
. : milestone, 14064,
iast_GLOBAL (10.709 ms) : 10522, 10897
. : milestone, 10709,
profiling (8.965 ms) : 8826, 9104
. : milestone, 8965,
tracing (8.036 ms) : 7919, 8154
. : milestone, 8036,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.55.0-SNAPSHOT~6939eba0aa, baseline=1.55.0-SNAPSHOT~c60b3c3cbf
dateFormat X
axisFormat %s
section baseline
no_agent (37.383 ms) : 37075, 37690
. : milestone, 37383,
appsec (45.785 ms) : 45381, 46189
. : milestone, 45785,
code_origins (45.229 ms) : 44838, 45619
. : milestone, 45229,
iast (44.384 ms) : 43993, 44775
. : milestone, 44384,
profiling (47.151 ms) : 46734, 47568
. : milestone, 47151,
tracing (45.234 ms) : 44852, 45615
. : milestone, 45234,
section candidate
no_agent (35.373 ms) : 35090, 35657
. : milestone, 35373,
appsec (46.695 ms) : 46281, 47109
. : milestone, 46695,
code_origins (41.937 ms) : 41587, 42286
. : milestone, 41937,
iast (43.982 ms) : 43599, 44366
. : milestone, 43982,
profiling (49.203 ms) : 48743, 49664
. : milestone, 49203,
tracing (44.464 ms) : 44081, 44847
. : milestone, 44464,
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.55.0-SNAPSHOT~6939eba0aa, baseline=1.55.0-SNAPSHOT~c60b3c3cbf
dateFormat X
axisFormat %s
section baseline
no_agent (1.485 ms) : 1473, 1497
. : milestone, 1485,
appsec (3.736 ms) : 3517, 3954
. : milestone, 3736,
iast (2.218 ms) : 2154, 2282
. : milestone, 2218,
iast_GLOBAL (2.267 ms) : 2203, 2331
. : milestone, 2267,
profiling (2.06 ms) : 2009, 2111
. : milestone, 2060,
tracing (2.039 ms) : 1989, 2089
. : milestone, 2039,
section candidate
no_agent (1.482 ms) : 1471, 1494
. : milestone, 1482,
appsec (3.822 ms) : 3598, 4046
. : milestone, 3822,
iast (2.216 ms) : 2153, 2279
. : milestone, 2216,
iast_GLOBAL (2.257 ms) : 2193, 2321
. : milestone, 2257,
profiling (2.066 ms) : 2015, 2118
. : milestone, 2066,
tracing (2.04 ms) : 1990, 2089
. : milestone, 2040,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.55.0-SNAPSHOT~6939eba0aa, baseline=1.55.0-SNAPSHOT~c60b3c3cbf
dateFormat X
axisFormat %s
section baseline
no_agent (14.988 s) : 14988000, 14988000
. : milestone, 14988000,
appsec (14.87 s) : 14870000, 14870000
. : milestone, 14870000,
iast (18.754 s) : 18754000, 18754000
. : milestone, 18754000,
iast_GLOBAL (17.924 s) : 17924000, 17924000
. : milestone, 17924000,
profiling (14.865 s) : 14865000, 14865000
. : milestone, 14865000,
tracing (15.107 s) : 15107000, 15107000
. : milestone, 15107000,
section candidate
no_agent (15.283 s) : 15283000, 15283000
. : milestone, 15283000,
appsec (14.759 s) : 14759000, 14759000
. : milestone, 14759000,
iast (18.746 s) : 18746000, 18746000
. : milestone, 18746000,
iast_GLOBAL (18.116 s) : 18116000, 18116000
. : milestone, 18116000,
profiling (15.308 s) : 15308000, 15308000
. : milestone, 15308000,
tracing (15.11 s) : 15110000, 15110000
. : milestone, 15110000,
|
…o add it to the generic signature
33b5404 to
65510c3
Compare
TaskWrapper interface
65510c3 to
7ff726e
Compare
… generic interfaces in sync
7ff726e to
6939eba
Compare
What Does This Do
When adding
TaskWrapperto the declared interfaces, make sure we also add it to the generic signatureMotivation
Avoid causing issues with introspection tools/agents that expect the interface list and generic signature to be aligned.
We did a similar fix for field-injection in #2203
Contributor Checklist
type:and (comp:orinst:) labels in addition to any useful labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: PROF-12675