-
Notifications
You must be signed in to change notification settings - Fork 312
Return noop continuation when not async propagating #8365
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
Merged
Merged
Conversation
This file contains hidden or 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
79ed9a2
to
145b63f
Compare
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 insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.47.0-SNAPSHOT~2c02593c3d, baseline=1.47.0-SNAPSHOT~652b46574b
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.055 s) : 0, 1055052
Total [baseline] (8.681 s) : 0, 8680808
Agent [candidate] (1.047 s) : 0, 1047142
Total [candidate] (8.681 s) : 0, 8681089
section iast
Agent [baseline] (1.172 s) : 0, 1172298
Total [baseline] (9.206 s) : 0, 9206086
Agent [candidate] (1.18 s) : 0, 1179550
Total [candidate] (9.193 s) : 0, 9192856
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.189 s) : 0, 1189185
Total [baseline] (9.223 s) : 0, 9223336
Agent [candidate] (1.173 s) : 0, 1172535
Total [candidate] (9.185 s) : 0, 9185113
section iast_TELEMETRY_OFF
Agent [baseline] (1.176 s) : 0, 1176094
Total [baseline] (9.233 s) : 0, 9232549
Agent [candidate] (1.169 s) : 0, 1168518
Total [candidate] (9.191 s) : 0, 9191098
gantt
title insecure-bank - break down per module: candidate=1.47.0-SNAPSHOT~2c02593c3d, baseline=1.47.0-SNAPSHOT~652b46574b
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (724.967 ms) : 0, 724967
BytebuddyAgent [candidate] (720.467 ms) : 0, 720467
GlobalTracer [baseline] (246.254 ms) : 0, 246254
GlobalTracer [candidate] (244.553 ms) : 0, 244553
AppSec [baseline] (55.396 ms) : 0, 55396
AppSec [candidate] (55.302 ms) : 0, 55302
Remote Config [baseline] (720.664 µs) : 0, 721
Remote Config [candidate] (719.287 µs) : 0, 719
Telemetry [baseline] (12.37 ms) : 0, 12370
Telemetry [candidate] (10.869 ms) : 0, 10869
section iast
BytebuddyAgent [baseline] (834.874 ms) : 0, 834874
BytebuddyAgent [candidate] (839.305 ms) : 0, 839305
GlobalTracer [baseline] (233.141 ms) : 0, 233141
GlobalTracer [candidate] (234.68 ms) : 0, 234680
IAST [baseline] (22.833 ms) : 0, 22833
IAST [candidate] (23.958 ms) : 0, 23958
AppSec [baseline] (56.876 ms) : 0, 56876
AppSec [candidate] (56.911 ms) : 0, 56911
Remote Config [baseline] (612.478 µs) : 0, 612
Remote Config [candidate] (625.246 µs) : 0, 625
Telemetry [baseline] (8.754 ms) : 0, 8754
Telemetry [candidate] (8.793 ms) : 0, 8793
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (846.964 ms) : 0, 846964
BytebuddyAgent [candidate] (833.953 ms) : 0, 833953
GlobalTracer [baseline] (236.531 ms) : 0, 236531
GlobalTracer [candidate] (233.975 ms) : 0, 233975
IAST [baseline] (23.367 ms) : 0, 23367
IAST [candidate] (22.985 ms) : 0, 22985
AppSec [baseline] (57.492 ms) : 0, 57492
AppSec [candidate] (57.116 ms) : 0, 57116
Remote Config [baseline] (626.899 µs) : 0, 627
Remote Config [candidate] (612.859 µs) : 0, 613
Telemetry [baseline] (8.811 ms) : 0, 8811
Telemetry [candidate] (8.645 ms) : 0, 8645
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (836.764 ms) : 0, 836764
BytebuddyAgent [candidate] (831.712 ms) : 0, 831712
GlobalTracer [baseline] (234.723 ms) : 0, 234723
GlobalTracer [candidate] (233.481 ms) : 0, 233481
IAST [baseline] (24.97 ms) : 0, 24970
IAST [candidate] (26.226 ms) : 0, 26226
AppSec [baseline] (54.971 ms) : 0, 54971
AppSec [candidate] (52.731 ms) : 0, 52731
Remote Config [baseline] (617.46 µs) : 0, 617
Remote Config [candidate] (604.639 µs) : 0, 605
Telemetry [baseline] (8.72 ms) : 0, 8720
Telemetry [candidate] (8.561 ms) : 0, 8561
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.47.0-SNAPSHOT~2c02593c3d, baseline=1.47.0-SNAPSHOT~652b46574b
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.042 s) : 0, 1041510
Total [baseline] (10.393 s) : 0, 10393460
Agent [candidate] (1.041 s) : 0, 1041359
Total [candidate] (10.498 s) : 0, 10497631
section appsec
Agent [baseline] (1.186 s) : 0, 1185940
Total [baseline] (10.768 s) : 0, 10768058
Agent [candidate] (1.194 s) : 0, 1194138
Total [candidate] (10.759 s) : 0, 10758941
section iast
Agent [baseline] (1.173 s) : 0, 1173430
Total [baseline] (10.951 s) : 0, 10951113
Agent [candidate] (1.181 s) : 0, 1181259
Total [candidate] (10.948 s) : 0, 10947527
section profiling
Agent [baseline] (1.261 s) : 0, 1260649
Total [baseline] (10.813 s) : 0, 10813443
Agent [candidate] (1.265 s) : 0, 1265414
Total [candidate] (10.929 s) : 0, 10929058
gantt
title petclinic - break down per module: candidate=1.47.0-SNAPSHOT~2c02593c3d, baseline=1.47.0-SNAPSHOT~652b46574b
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (716.317 ms) : 0, 716317
BytebuddyAgent [candidate] (715.629 ms) : 0, 715629
GlobalTracer [baseline] (242.964 ms) : 0, 242964
GlobalTracer [candidate] (243.075 ms) : 0, 243075
AppSec [baseline] (55.051 ms) : 0, 55051
AppSec [candidate] (55.164 ms) : 0, 55164
Remote Config [baseline] (702.861 µs) : 0, 703
Remote Config [candidate] (709.79 µs) : 0, 710
Telemetry [baseline] (11.385 ms) : 0, 11385
Telemetry [candidate] (11.55 ms) : 0, 11550
section appsec
BytebuddyAgent [baseline] (733.473 ms) : 0, 733473
BytebuddyAgent [candidate] (739.037 ms) : 0, 739037
GlobalTracer [baseline] (240.85 ms) : 0, 240850
GlobalTracer [candidate] (242.581 ms) : 0, 242581
IAST [baseline] (21.594 ms) : 0, 21594
IAST [candidate] (21.998 ms) : 0, 21998
AppSec [baseline] (176.77 ms) : 0, 176770
AppSec [candidate] (177.259 ms) : 0, 177259
Remote Config [baseline] (664.49 µs) : 0, 664
Remote Config [candidate] (660.508 µs) : 0, 661
Telemetry [baseline] (8.245 ms) : 0, 8245
Telemetry [candidate] (8.306 ms) : 0, 8306
section iast
BytebuddyAgent [baseline] (834.542 ms) : 0, 834542
BytebuddyAgent [candidate] (841.345 ms) : 0, 841345
GlobalTracer [baseline] (233.867 ms) : 0, 233867
GlobalTracer [candidate] (235.18 ms) : 0, 235180
IAST [baseline] (22.944 ms) : 0, 22944
IAST [candidate] (23.209 ms) : 0, 23209
AppSec [baseline] (57.546 ms) : 0, 57546
AppSec [candidate] (56.689 ms) : 0, 56689
Remote Config [baseline] (622.717 µs) : 0, 623
Remote Config [candidate] (615.22 µs) : 0, 615
Telemetry [baseline] (8.665 ms) : 0, 8665
Telemetry [candidate] (8.727 ms) : 0, 8727
section profiling
BytebuddyAgent [baseline] (705.935 ms) : 0, 705935
BytebuddyAgent [candidate] (706.806 ms) : 0, 706806
GlobalTracer [baseline] (352.834 ms) : 0, 352834
GlobalTracer [candidate] (355.927 ms) : 0, 355927
AppSec [baseline] (55.101 ms) : 0, 55101
AppSec [candidate] (54.776 ms) : 0, 54776
Remote Config [baseline] (691.399 µs) : 0, 691
Remote Config [candidate] (716.225 µs) : 0, 716
Telemetry [baseline] (8.86 ms) : 0, 8860
Telemetry [candidate] (8.98 ms) : 0, 8980
ProfilingAgent [baseline] (95.007 ms) : 0, 95007
ProfilingAgent [candidate] (95.868 ms) : 0, 95868
Profiling [baseline] (95.031 ms) : 0, 95031
Profiling [candidate] (95.893 ms) : 0, 95893
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.47.0-SNAPSHOT~2c02593c3d, baseline=1.47.0-SNAPSHOT~652b46574b
dateFormat X
axisFormat %s
section baseline
no_agent (390.092 µs) : 370, 410
. : milestone, 390,
iast (516.291 µs) : 495, 538
. : milestone, 516,
iast_FULL (748.657 µs) : 727, 771
. : milestone, 749,
iast_GLOBAL (563.68 µs) : 542, 586
. : milestone, 564,
iast_HARDCODED_SECRET_DISABLED (515.008 µs) : 493, 537
. : milestone, 515,
iast_INACTIVE (467.0 µs) : 445, 489
. : milestone, 467,
iast_TELEMETRY_OFF (502.339 µs) : 481, 524
. : milestone, 502,
tracing (456.698 µs) : 434, 479
. : milestone, 457,
section candidate
no_agent (381.258 µs) : 361, 401
. : milestone, 381,
iast (520.573 µs) : 499, 542
. : milestone, 521,
iast_FULL (753.097 µs) : 731, 775
. : milestone, 753,
iast_GLOBAL (561.219 µs) : 540, 583
. : milestone, 561,
iast_HARDCODED_SECRET_DISABLED (517.195 µs) : 495, 539
. : milestone, 517,
iast_INACTIVE (467.189 µs) : 446, 489
. : milestone, 467,
iast_TELEMETRY_OFF (505.398 µs) : 482, 529
. : milestone, 505,
tracing (458.495 µs) : 437, 480
. : milestone, 458,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.47.0-SNAPSHOT~2c02593c3d, baseline=1.47.0-SNAPSHOT~652b46574b
dateFormat X
axisFormat %s
section baseline
no_agent (1.385 ms) : 1366, 1405
. : milestone, 1385,
appsec (1.754 ms) : 1730, 1777
. : milestone, 1754,
appsec_no_iast (1.759 ms) : 1734, 1784
. : milestone, 1759,
iast (1.53 ms) : 1506, 1555
. : milestone, 1530,
profiling (1.493 ms) : 1469, 1516
. : milestone, 1493,
tracing (1.468 ms) : 1442, 1493
. : milestone, 1468,
section candidate
no_agent (1.36 ms) : 1340, 1380
. : milestone, 1360,
appsec (1.75 ms) : 1727, 1774
. : milestone, 1750,
appsec_no_iast (1.771 ms) : 1747, 1795
. : milestone, 1771,
iast (1.503 ms) : 1478, 1528
. : milestone, 1503,
profiling (1.517 ms) : 1493, 1540
. : milestone, 1517,
tracing (1.51 ms) : 1485, 1534
. : milestone, 1510,
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.47.0-SNAPSHOT~2c02593c3d, baseline=1.47.0-SNAPSHOT~652b46574b
dateFormat X
axisFormat %s
section baseline
no_agent (15.031 s) : 15031000, 15031000
. : milestone, 15031000,
appsec (15.263 s) : 15263000, 15263000
. : milestone, 15263000,
iast (18.79 s) : 18790000, 18790000
. : milestone, 18790000,
iast_GLOBAL (17.823 s) : 17823000, 17823000
. : milestone, 17823000,
profiling (15.165 s) : 15165000, 15165000
. : milestone, 15165000,
tracing (14.82 s) : 14820000, 14820000
. : milestone, 14820000,
section candidate
no_agent (15.418 s) : 15418000, 15418000
. : milestone, 15418000,
appsec (15.093 s) : 15093000, 15093000
. : milestone, 15093000,
iast (18.724 s) : 18724000, 18724000
. : milestone, 18724000,
iast_GLOBAL (17.995 s) : 17995000, 17995000
. : milestone, 17995000,
profiling (14.978 s) : 14978000, 14978000
. : milestone, 14978000,
tracing (15.152 s) : 15152000, 15152000
. : milestone, 15152000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.47.0-SNAPSHOT~2c02593c3d, baseline=1.47.0-SNAPSHOT~652b46574b
dateFormat X
axisFormat %s
section baseline
no_agent (1.474 ms) : 1462, 1485
. : milestone, 1474,
appsec (2.376 ms) : 2332, 2419
. : milestone, 2376,
iast (2.118 ms) : 2063, 2174
. : milestone, 2118,
iast_GLOBAL (2.162 ms) : 2106, 2217
. : milestone, 2162,
profiling (1.992 ms) : 1948, 2037
. : milestone, 1992,
tracing (1.949 ms) : 1906, 1991
. : milestone, 1949,
section candidate
no_agent (1.471 ms) : 1460, 1483
. : milestone, 1471,
appsec (2.364 ms) : 2321, 2407
. : milestone, 2364,
iast (2.118 ms) : 2062, 2173
. : milestone, 2118,
iast_GLOBAL (2.159 ms) : 2103, 2214
. : milestone, 2159,
profiling (1.973 ms) : 1929, 2016
. : milestone, 1973,
tracing (1.94 ms) : 1898, 1983
. : milestone, 1940,
|
2c02593
to
c7a7002
Compare
PerfectSlayer
approved these changes
Feb 11, 2025
c7a7002
to
2fa7d15
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
comp: core
Tracer core
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.
Motivation
Improves API robustness
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: [/APMAPI-955]