-
Notifications
You must be signed in to change notification settings - Fork 312
chore: Cleaning up HttpServerDecorator #9139
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 43 metrics, 10 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.52.0-SNAPSHOT~a4f6a9a2b3, baseline=1.52.0-SNAPSHOT~a90ccbcd72
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (995.659 ms) : 0, 995659
Total [baseline] (8.582 s) : 0, 8581910
Agent [candidate] (996.986 ms) : 0, 996986
Total [candidate] (8.585 s) : 0, 8584939
section iast
Agent [baseline] (1.133 s) : 0, 1133377
Total [baseline] (9.328 s) : 0, 9327760
Agent [candidate] (1.135 s) : 0, 1134876
Total [candidate] (9.302 s) : 0, 9301798
gantt
title insecure-bank - break down per module: candidate=1.52.0-SNAPSHOT~a4f6a9a2b3, baseline=1.52.0-SNAPSHOT~a90ccbcd72
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (687.954 ms) : 0, 687954
BytebuddyAgent [candidate] (688.472 ms) : 0, 688472
GlobalTracer [baseline] (242.024 ms) : 0, 242024
GlobalTracer [candidate] (242.437 ms) : 0, 242437
AppSec [baseline] (30.13 ms) : 0, 30130
AppSec [candidate] (30.353 ms) : 0, 30353
Debugger [baseline] (5.943 ms) : 0, 5943
Debugger [candidate] (6.045 ms) : 0, 6045
Remote Config [baseline] (682.31 µs) : 0, 682
Remote Config [candidate] (676.416 µs) : 0, 676
Telemetry [baseline] (8.217 ms) : 0, 8217
Telemetry [candidate] (8.301 ms) : 0, 8301
section iast
BytebuddyAgent [baseline] (808.326 ms) : 0, 808326
BytebuddyAgent [candidate] (809.164 ms) : 0, 809164
GlobalTracer [baseline] (232.169 ms) : 0, 232169
GlobalTracer [candidate] (232.56 ms) : 0, 232560
IAST [baseline] (26.511 ms) : 0, 26511
IAST [candidate] (27.512 ms) : 0, 27512
AppSec [baseline] (31.348 ms) : 0, 31348
AppSec [candidate] (29.787 ms) : 0, 29787
Debugger [baseline] (5.8 ms) : 0, 5800
Debugger [candidate] (6.583 ms) : 0, 6583
Remote Config [baseline] (592.02 µs) : 0, 592
Remote Config [candidate] (578.455 µs) : 0, 578
Telemetry [baseline] (7.94 ms) : 0, 7940
Telemetry [candidate] (7.968 ms) : 0, 7968
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.52.0-SNAPSHOT~a4f6a9a2b3, baseline=1.52.0-SNAPSHOT~a90ccbcd72
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (997.638 ms) : 0, 997638
Total [baseline] (10.695 s) : 0, 10694504
Agent [candidate] (1.002 s) : 0, 1002371
Total [candidate] (10.712 s) : 0, 10711743
section appsec
Agent [baseline] (1.185 s) : 0, 1185203
Total [baseline] (10.867 s) : 0, 10866783
Agent [candidate] (1.176 s) : 0, 1176373
Total [candidate] (10.732 s) : 0, 10731612
section iast
Agent [baseline] (1.143 s) : 0, 1143207
Total [baseline] (10.984 s) : 0, 10984168
Agent [candidate] (1.133 s) : 0, 1133139
Total [candidate] (10.903 s) : 0, 10903427
section profiling
Agent [baseline] (1.246 s) : 0, 1245748
Total [baseline] (10.946 s) : 0, 10945610
Agent [candidate] (1.247 s) : 0, 1246990
Total [candidate] (11.057 s) : 0, 11057007
gantt
title petclinic - break down per module: candidate=1.52.0-SNAPSHOT~a4f6a9a2b3, baseline=1.52.0-SNAPSHOT~a90ccbcd72
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (688.319 ms) : 0, 688319
BytebuddyAgent [candidate] (692.347 ms) : 0, 692347
GlobalTracer [baseline] (242.998 ms) : 0, 242998
GlobalTracer [candidate] (243.63 ms) : 0, 243630
AppSec [baseline] (30.49 ms) : 0, 30490
AppSec [candidate] (30.422 ms) : 0, 30422
Debugger [baseline] (6.04 ms) : 0, 6040
Debugger [candidate] (6.082 ms) : 0, 6082
Remote Config [baseline] (685.579 µs) : 0, 686
Remote Config [candidate] (678.438 µs) : 0, 678
Telemetry [baseline] (8.259 ms) : 0, 8259
Telemetry [candidate] (8.272 ms) : 0, 8272
section appsec
BytebuddyAgent [baseline] (716.709 ms) : 0, 716709
BytebuddyAgent [candidate] (711.407 ms) : 0, 711407
GlobalTracer [baseline] (236.853 ms) : 0, 236853
GlobalTracer [candidate] (235.351 ms) : 0, 235351
IAST [baseline] (23.822 ms) : 0, 23822
IAST [candidate] (23.504 ms) : 0, 23504
AppSec [baseline] (172.278 ms) : 0, 172278
AppSec [candidate] (171.032 ms) : 0, 171032
Debugger [baseline] (5.762 ms) : 0, 5762
Debugger [candidate] (5.69 ms) : 0, 5690
Remote Config [baseline] (630.324 µs) : 0, 630
Remote Config [candidate] (598.785 µs) : 0, 599
Telemetry [baseline] (8.127 ms) : 0, 8127
Telemetry [candidate] (7.948 ms) : 0, 7948
section iast
BytebuddyAgent [baseline] (815.24 ms) : 0, 815240
BytebuddyAgent [candidate] (807.86 ms) : 0, 807860
GlobalTracer [baseline] (233.772 ms) : 0, 233772
GlobalTracer [candidate] (232.304 ms) : 0, 232304
IAST [baseline] (25.862 ms) : 0, 25862
IAST [candidate] (29.01 ms) : 0, 29010
AppSec [baseline] (32.937 ms) : 0, 32937
AppSec [candidate] (28.855 ms) : 0, 28855
Debugger [baseline] (5.816 ms) : 0, 5816
Debugger [candidate] (5.776 ms) : 0, 5776
Remote Config [baseline] (617.46 µs) : 0, 617
Remote Config [candidate] (586.447 µs) : 0, 586
Telemetry [baseline] (8.023 ms) : 0, 8023
Telemetry [candidate] (7.975 ms) : 0, 7975
section profiling
BytebuddyAgent [baseline] (678.466 ms) : 0, 678466
BytebuddyAgent [candidate] (679.086 ms) : 0, 679086
GlobalTracer [baseline] (361.748 ms) : 0, 361748
GlobalTracer [candidate] (362.151 ms) : 0, 362151
AppSec [baseline] (33.413 ms) : 0, 33413
AppSec [candidate] (32.434 ms) : 0, 32434
Debugger [baseline] (9.672 ms) : 0, 9672
Debugger [candidate] (10.491 ms) : 0, 10491
Remote Config [baseline] (659.014 µs) : 0, 659
Remote Config [candidate] (659.332 µs) : 0, 659
Telemetry [baseline] (9.58 ms) : 0, 9580
Telemetry [candidate] (8.768 ms) : 0, 8768
ProfilingAgent [baseline] (103.698 ms) : 0, 103698
ProfilingAgent [candidate] (104.568 ms) : 0, 104568
Profiling [baseline] (103.722 ms) : 0, 103722
Profiling [candidate] (104.592 ms) : 0, 104592
LoadParameters
See matching parameters
SummaryFound 3 performance improvements and 2 performance regressions! Performance is the same for 7 metrics, 12 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.52.0-SNAPSHOT~a4f6a9a2b3, baseline=1.52.0-SNAPSHOT~a90ccbcd72
dateFormat X
axisFormat %s
section baseline
no_agent (4.277 ms) : 4229, 4325
. : milestone, 4277,
iast (9.464 ms) : 9303, 9626
. : milestone, 9464,
iast_FULL (13.652 ms) : 13377, 13926
. : milestone, 13652,
iast_GLOBAL (10.559 ms) : 10371, 10747
. : milestone, 10559,
profiling (8.486 ms) : 8350, 8622
. : milestone, 8486,
tracing (7.55 ms) : 7441, 7658
. : milestone, 7550,
section candidate
no_agent (4.545 ms) : 4494, 4597
. : milestone, 4545,
iast (9.318 ms) : 9164, 9473
. : milestone, 9318,
iast_FULL (13.774 ms) : 13499, 14049
. : milestone, 13774,
iast_GLOBAL (10.266 ms) : 10086, 10445
. : milestone, 10266,
profiling (8.576 ms) : 8434, 8718
. : milestone, 8576,
tracing (7.679 ms) : 7568, 7789
. : milestone, 7679,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.52.0-SNAPSHOT~a4f6a9a2b3, baseline=1.52.0-SNAPSHOT~a90ccbcd72
dateFormat X
axisFormat %s
section baseline
no_agent (37.838 ms) : 37527, 38149
. : milestone, 37838,
appsec (49.623 ms) : 49190, 50056
. : milestone, 49623,
code_origins (43.582 ms) : 43214, 43951
. : milestone, 43582,
iast (44.716 ms) : 44320, 45112
. : milestone, 44716,
profiling (48.135 ms) : 47717, 48552
. : milestone, 48135,
tracing (44.025 ms) : 43649, 44400
. : milestone, 44025,
section candidate
no_agent (36.625 ms) : 36330, 36920
. : milestone, 36625,
appsec (45.735 ms) : 45336, 46134
. : milestone, 45735,
code_origins (46.075 ms) : 45676, 46474
. : milestone, 46075,
iast (44.665 ms) : 44267, 45063
. : milestone, 44665,
profiling (48.978 ms) : 48520, 49435
. : milestone, 48978,
tracing (42.512 ms) : 42160, 42864
. : milestone, 42512,
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 biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.52.0-SNAPSHOT~a4f6a9a2b3, baseline=1.52.0-SNAPSHOT~a90ccbcd72
dateFormat X
axisFormat %s
section baseline
no_agent (15.332 s) : 15332000, 15332000
. : milestone, 15332000,
appsec (14.959 s) : 14959000, 14959000
. : milestone, 14959000,
iast (18.332 s) : 18332000, 18332000
. : milestone, 18332000,
iast_GLOBAL (18.083 s) : 18083000, 18083000
. : milestone, 18083000,
profiling (15.363 s) : 15363000, 15363000
. : milestone, 15363000,
tracing (14.552 s) : 14552000, 14552000
. : milestone, 14552000,
section candidate
no_agent (15.3 s) : 15300000, 15300000
. : milestone, 15300000,
appsec (14.638 s) : 14638000, 14638000
. : milestone, 14638000,
iast (18.552 s) : 18552000, 18552000
. : milestone, 18552000,
iast_GLOBAL (18.261 s) : 18261000, 18261000
. : milestone, 18261000,
profiling (15.356 s) : 15356000, 15356000
. : milestone, 15356000,
tracing (15.055 s) : 15055000, 15055000
. : milestone, 15055000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.52.0-SNAPSHOT~a4f6a9a2b3, baseline=1.52.0-SNAPSHOT~a90ccbcd72
dateFormat X
axisFormat %s
section baseline
no_agent (1.474 ms) : 1462, 1485
. : milestone, 1474,
appsec (2.416 ms) : 2365, 2466
. : milestone, 2416,
iast (2.203 ms) : 2140, 2265
. : milestone, 2203,
iast_GLOBAL (2.242 ms) : 2180, 2305
. : milestone, 2242,
profiling (2.494 ms) : 2319, 2669
. : milestone, 2494,
tracing (2.006 ms) : 1958, 2055
. : milestone, 2006,
section candidate
no_agent (1.474 ms) : 1463, 1486
. : milestone, 1474,
appsec (2.428 ms) : 2378, 2478
. : milestone, 2428,
iast (2.196 ms) : 2134, 2259
. : milestone, 2196,
iast_GLOBAL (2.238 ms) : 2174, 2301
. : milestone, 2238,
profiling (2.061 ms) : 2010, 2112
. : milestone, 2061,
tracing (2.014 ms) : 1966, 2063
. : milestone, 2014,
|
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
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.
Thanks for the clean up!
What Does This Do
Previously, we supported ways to extract span contexts and start spans with span contexts. During our migration to Context API, we maintained support for span contexts while migrating all instrumentations to extract and start spans using generic contexts. This PR removes support for extracting and starting spans with span contexts alone.
Motivation
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 issueJira ticket: [PROJ-IDENT]