-
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
Grizzly http client: finish spans for all handlers #7772
Merged
Merged
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
richardstartin
approved these changes
Oct 16, 2024
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 55 metrics, 8 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.41.0-SNAPSHOT~b166332a6e, baseline=1.41.0-SNAPSHOT~e08f541bb5
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.078 s) : 0, 1077654
Total [baseline] (8.564 s) : 0, 8563986
Agent [candidate] (1.069 s) : 0, 1068759
Total [candidate] (8.559 s) : 0, 8559088
section iast
Agent [baseline] (1.197 s) : 0, 1196820
Total [baseline] (9.131 s) : 0, 9131072
Agent [candidate] (1.206 s) : 0, 1206399
Total [candidate] (9.137 s) : 0, 9137011
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.205 s) : 0, 1204907
Total [baseline] (9.114 s) : 0, 9114274
Agent [candidate] (1.2 s) : 0, 1200412
Total [candidate] (9.083 s) : 0, 9083413
section iast_TELEMETRY_OFF
Agent [baseline] (1.195 s) : 0, 1194581
Total [baseline] (9.089 s) : 0, 9089188
Agent [candidate] (1.199 s) : 0, 1199190
Total [candidate] (9.137 s) : 0, 9136724
gantt
title insecure-bank - break down per module: candidate=1.41.0-SNAPSHOT~b166332a6e, baseline=1.41.0-SNAPSHOT~e08f541bb5
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (687.138 ms) : 0, 687138
BytebuddyAgent [candidate] (680.595 ms) : 0, 680595
GlobalTracer [baseline] (314.57 ms) : 0, 314570
GlobalTracer [candidate] (312.585 ms) : 0, 312585
AppSec [baseline] (53.96 ms) : 0, 53960
AppSec [candidate] (53.799 ms) : 0, 53799
Remote Config [baseline] (666.266 µs) : 0, 666
Remote Config [candidate] (660.317 µs) : 0, 660
Telemetry [baseline] (7.589 ms) : 0, 7589
Telemetry [candidate] (7.487 ms) : 0, 7487
section iast
BytebuddyAgent [baseline] (796.345 ms) : 0, 796345
BytebuddyAgent [candidate] (802.55 ms) : 0, 802550
GlobalTracer [baseline] (300.556 ms) : 0, 300556
GlobalTracer [candidate] (303.446 ms) : 0, 303446
AppSec [baseline] (56.281 ms) : 0, 56281
AppSec [candidate] (57.474 ms) : 0, 57474
IAST [baseline] (22.243 ms) : 0, 22243
IAST [candidate] (21.417 ms) : 0, 21417
Remote Config [baseline] (608.02 µs) : 0, 608
Remote Config [candidate] (606.825 µs) : 0, 607
Telemetry [baseline] (7.152 ms) : 0, 7152
Telemetry [candidate] (7.139 ms) : 0, 7139
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (800.911 ms) : 0, 800911
BytebuddyAgent [candidate] (797.795 ms) : 0, 797795
GlobalTracer [baseline] (303.637 ms) : 0, 303637
GlobalTracer [candidate] (302.687 ms) : 0, 302687
AppSec [baseline] (55.941 ms) : 0, 55941
AppSec [candidate] (56.296 ms) : 0, 56296
IAST [baseline] (22.918 ms) : 0, 22918
IAST [candidate] (21.323 ms) : 0, 21323
Remote Config [baseline] (603.853 µs) : 0, 604
Remote Config [candidate] (611.165 µs) : 0, 611
Telemetry [baseline] (7.115 ms) : 0, 7115
Telemetry [candidate] (7.939 ms) : 0, 7939
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (793.689 ms) : 0, 793689
BytebuddyAgent [candidate] (795.182 ms) : 0, 795182
GlobalTracer [baseline] (302.018 ms) : 0, 302018
GlobalTracer [candidate] (304.354 ms) : 0, 304354
AppSec [baseline] (54.507 ms) : 0, 54507
AppSec [candidate] (55.019 ms) : 0, 55019
IAST [baseline] (23.065 ms) : 0, 23065
IAST [candidate] (23.291 ms) : 0, 23291
Remote Config [baseline] (608.13 µs) : 0, 608
Remote Config [candidate] (607.23 µs) : 0, 607
Telemetry [baseline] (6.982 ms) : 0, 6982
Telemetry [candidate] (7.02 ms) : 0, 7020
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.41.0-SNAPSHOT~b166332a6e, baseline=1.41.0-SNAPSHOT~e08f541bb5
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.071 s) : 0, 1071431
Total [baseline] (10.45 s) : 0, 10450221
Agent [candidate] (1.079 s) : 0, 1078580
Total [candidate] (10.424 s) : 0, 10424474
section appsec
Agent [baseline] (1.203 s) : 0, 1203281
Total [baseline] (10.601 s) : 0, 10601293
Agent [candidate] (1.206 s) : 0, 1205827
Total [candidate] (10.619 s) : 0, 10619243
section iast
Agent [baseline] (1.198 s) : 0, 1197906
Total [baseline] (10.922 s) : 0, 10922154
Agent [candidate] (1.199 s) : 0, 1199199
Total [candidate] (10.882 s) : 0, 10882427
section profiling
Agent [baseline] (1.265 s) : 0, 1265484
Total [baseline] (10.716 s) : 0, 10715878
Agent [candidate] (1.277 s) : 0, 1276519
Total [candidate] (10.692 s) : 0, 10692493
gantt
title petclinic - break down per module: candidate=1.41.0-SNAPSHOT~b166332a6e, baseline=1.41.0-SNAPSHOT~e08f541bb5
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (682.38 ms) : 0, 682380
BytebuddyAgent [candidate] (687.025 ms) : 0, 687025
GlobalTracer [baseline] (313.367 ms) : 0, 313367
GlobalTracer [candidate] (315.453 ms) : 0, 315453
AppSec [baseline] (53.837 ms) : 0, 53837
AppSec [candidate] (54.199 ms) : 0, 54199
Remote Config [baseline] (663.839 µs) : 0, 664
Remote Config [candidate] (666.139 µs) : 0, 666
Telemetry [baseline] (7.551 ms) : 0, 7551
Telemetry [candidate] (7.512 ms) : 0, 7512
section appsec
BytebuddyAgent [baseline] (698.335 ms) : 0, 698335
BytebuddyAgent [candidate] (699.432 ms) : 0, 699432
GlobalTracer [baseline] (309.758 ms) : 0, 309758
GlobalTracer [candidate] (310.686 ms) : 0, 310686
AppSec [baseline] (162.062 ms) : 0, 162062
AppSec [candidate] (163.013 ms) : 0, 163013
IAST [baseline] (21.502 ms) : 0, 21502
IAST [candidate] (20.624 ms) : 0, 20624
Remote Config [baseline] (631.703 µs) : 0, 632
Remote Config [candidate] (632.009 µs) : 0, 632
Telemetry [baseline] (8.098 ms) : 0, 8098
Telemetry [candidate] (8.087 ms) : 0, 8087
section iast
BytebuddyAgent [baseline] (796.639 ms) : 0, 796639
BytebuddyAgent [candidate] (797.366 ms) : 0, 797366
GlobalTracer [baseline] (301.704 ms) : 0, 301704
GlobalTracer [candidate] (301.525 ms) : 0, 301525
AppSec [baseline] (56.979 ms) : 0, 56979
AppSec [candidate] (57.39 ms) : 0, 57390
IAST [baseline] (21.312 ms) : 0, 21312
IAST [candidate] (21.514 ms) : 0, 21514
Remote Config [baseline] (600.421 µs) : 0, 600
Remote Config [candidate] (618.468 µs) : 0, 618
Telemetry [baseline] (7.06 ms) : 0, 7060
Telemetry [candidate] (7.155 ms) : 0, 7155
section profiling
BytebuddyAgent [baseline] (673.53 ms) : 0, 673530
BytebuddyAgent [candidate] (680.21 ms) : 0, 680210
GlobalTracer [baseline] (394.93 ms) : 0, 394930
GlobalTracer [candidate] (398.431 ms) : 0, 398431
AppSec [baseline] (54.532 ms) : 0, 54532
AppSec [candidate] (54.611 ms) : 0, 54611
Remote Config [baseline] (648.281 µs) : 0, 648
Remote Config [candidate] (652.871 µs) : 0, 653
Telemetry [baseline] (7.423 ms) : 0, 7423
Telemetry [candidate] (7.45 ms) : 0, 7450
ProfilingAgent [baseline] (95.859 ms) : 0, 95859
ProfilingAgent [candidate] (96.346 ms) : 0, 96346
Profiling [baseline] (95.883 ms) : 0, 95883
Profiling [candidate] (96.371 ms) : 0, 96371
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 1 performance regressions! Performance is the same for 10 metrics, 17 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.41.0-SNAPSHOT~b166332a6e, baseline=1.41.0-SNAPSHOT~e08f541bb5
dateFormat X
axisFormat %s
section baseline
no_agent (1.342 ms) : 1323, 1361
. : milestone, 1342,
appsec (1.704 ms) : 1679, 1729
. : milestone, 1704,
appsec_no_iast (1.715 ms) : 1691, 1739
. : milestone, 1715,
iast (1.486 ms) : 1463, 1509
. : milestone, 1486,
profiling (1.471 ms) : 1447, 1495
. : milestone, 1471,
tracing (1.472 ms) : 1448, 1497
. : milestone, 1472,
section candidate
no_agent (1.316 ms) : 1296, 1336
. : milestone, 1316,
appsec (1.707 ms) : 1682, 1731
. : milestone, 1707,
appsec_no_iast (1.732 ms) : 1708, 1756
. : milestone, 1732,
iast (1.472 ms) : 1449, 1496
. : milestone, 1472,
profiling (1.538 ms) : 1513, 1563
. : milestone, 1538,
tracing (1.464 ms) : 1440, 1489
. : milestone, 1464,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.41.0-SNAPSHOT~b166332a6e, baseline=1.41.0-SNAPSHOT~e08f541bb5
dateFormat X
axisFormat %s
section baseline
no_agent (371.147 µs) : 352, 391
. : milestone, 371,
iast (478.798 µs) : 458, 500
. : milestone, 479,
iast_FULL (550.465 µs) : 529, 572
. : milestone, 550,
iast_GLOBAL (507.493 µs) : 487, 528
. : milestone, 507,
iast_HARDCODED_SECRET_DISABLED (481.833 µs) : 461, 503
. : milestone, 482,
iast_INACTIVE (439.754 µs) : 419, 460
. : milestone, 440,
iast_TELEMETRY_OFF (466.74 µs) : 446, 488
. : milestone, 467,
tracing (443.788 µs) : 423, 465
. : milestone, 444,
section candidate
no_agent (367.175 µs) : 348, 387
. : milestone, 367,
iast (478.781 µs) : 458, 500
. : milestone, 479,
iast_FULL (548.752 µs) : 528, 570
. : milestone, 549,
iast_GLOBAL (506.999 µs) : 485, 529
. : milestone, 507,
iast_HARDCODED_SECRET_DISABLED (482.494 µs) : 461, 504
. : milestone, 482,
iast_INACTIVE (443.403 µs) : 422, 464
. : milestone, 443,
iast_TELEMETRY_OFF (468.867 µs) : 448, 490
. : milestone, 469,
tracing (439.382 µs) : 419, 460
. : milestone, 439,
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.41.0-SNAPSHOT~b166332a6e, baseline=1.41.0-SNAPSHOT~e08f541bb5
dateFormat X
axisFormat %s
section baseline
no_agent (15.074 s) : 15074000, 15074000
. : milestone, 15074000,
appsec (15.345 s) : 15345000, 15345000
. : milestone, 15345000,
iast (18.95 s) : 18950000, 18950000
. : milestone, 18950000,
iast_GLOBAL (18.1 s) : 18100000, 18100000
. : milestone, 18100000,
profiling (15.412 s) : 15412000, 15412000
. : milestone, 15412000,
tracing (15.414 s) : 15414000, 15414000
. : milestone, 15414000,
section candidate
no_agent (15.061 s) : 15061000, 15061000
. : milestone, 15061000,
appsec (15.126 s) : 15126000, 15126000
. : milestone, 15126000,
iast (18.698 s) : 18698000, 18698000
. : milestone, 18698000,
iast_GLOBAL (18.011 s) : 18011000, 18011000
. : milestone, 18011000,
profiling (15.433 s) : 15433000, 15433000
. : milestone, 15433000,
tracing (15.09 s) : 15090000, 15090000
. : milestone, 15090000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.41.0-SNAPSHOT~b166332a6e, baseline=1.41.0-SNAPSHOT~e08f541bb5
dateFormat X
axisFormat %s
section baseline
no_agent (1.469 ms) : 1457, 1480
. : milestone, 1469,
appsec (2.329 ms) : 2288, 2370
. : milestone, 2329,
iast (2.09 ms) : 2038, 2143
. : milestone, 2090,
iast_GLOBAL (2.117 ms) : 2065, 2169
. : milestone, 2117,
profiling (1.945 ms) : 1904, 1986
. : milestone, 1945,
tracing (1.927 ms) : 1887, 1967
. : milestone, 1927,
section candidate
no_agent (1.472 ms) : 1461, 1484
. : milestone, 1472,
appsec (2.326 ms) : 2286, 2367
. : milestone, 2326,
iast (2.076 ms) : 2024, 2128
. : milestone, 2076,
iast_GLOBAL (2.125 ms) : 2073, 2177
. : milestone, 2125,
profiling (1.936 ms) : 1895, 1978
. : milestone, 1936,
tracing (1.916 ms) : 1877, 1955
. : milestone, 1916,
|
amarziali
force-pushed
the
andrea.marziali/grizzly-client
branch
from
October 16, 2024 12:53
b166332
to
b8a0eae
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
In grizzly http client we start a span that we finish when the
AsyncCompletionHandler
was called. However the general signature of the clientexecuteRequest
method is taking the super interfaceAsyncHandler
.If someone is not using explicitly a subclass of
AsyncCompletionHandler
as it's the case for mulesoft (it uses other things under the hood), the span will never finish.This PR fixes this corner case
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]