-
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
Use method enter advices for vert.x session blocking #8225
Conversation
956d7b9
to
d88adb1
Compare
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.
LGTM for IDM. It only concerns asm
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 60 metrics, 3 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.46.0-SNAPSHOT~d88adb1f94, baseline=1.46.0-SNAPSHOT~148aef92ab
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.056 s) : 0, 1055743
Total [baseline] (10.639 s) : 0, 10639450
Agent [candidate] (1.057 s) : 0, 1057216
Total [candidate] (10.533 s) : 0, 10533168
section appsec
Agent [baseline] (1.195 s) : 0, 1194885
Total [baseline] (10.705 s) : 0, 10705090
Agent [candidate] (1.2 s) : 0, 1200025
Total [candidate] (10.768 s) : 0, 10768442
section iast
Agent [baseline] (1.191 s) : 0, 1190567
Total [baseline] (10.981 s) : 0, 10980582
Agent [candidate] (1.193 s) : 0, 1192631
Total [candidate] (11.129 s) : 0, 11128577
section profiling
Agent [baseline] (1.255 s) : 0, 1254502
Total [baseline] (10.823 s) : 0, 10822545
Agent [candidate] (1.254 s) : 0, 1254162
Total [candidate] (10.865 s) : 0, 10864956
gantt
title petclinic - break down per module: candidate=1.46.0-SNAPSHOT~d88adb1f94, baseline=1.46.0-SNAPSHOT~148aef92ab
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (714.571 ms) : 0, 714571
BytebuddyAgent [candidate] (715.163 ms) : 0, 715163
GlobalTracer [baseline] (256.391 ms) : 0, 256391
GlobalTracer [candidate] (256.185 ms) : 0, 256185
AppSec [baseline] (55.286 ms) : 0, 55286
AppSec [candidate] (55.468 ms) : 0, 55468
Remote Config [baseline] (725.42 µs) : 0, 725
Remote Config [candidate] (739.846 µs) : 0, 740
Telemetry [baseline] (13.79 ms) : 0, 13790
Telemetry [candidate] (14.58 ms) : 0, 14580
section appsec
BytebuddyAgent [baseline] (735.299 ms) : 0, 735299
BytebuddyAgent [candidate] (738.31 ms) : 0, 738310
GlobalTracer [baseline] (254.204 ms) : 0, 254204
GlobalTracer [candidate] (255.381 ms) : 0, 255381
AppSec [baseline] (171.257 ms) : 0, 171257
AppSec [candidate] (172.213 ms) : 0, 172213
IAST [baseline] (19.636 ms) : 0, 19636
IAST [candidate] (19.656 ms) : 0, 19656
Remote Config [baseline] (663.895 µs) : 0, 664
Remote Config [candidate] (678.245 µs) : 0, 678
Telemetry [baseline] (8.526 ms) : 0, 8526
Telemetry [candidate] (8.37 ms) : 0, 8370
section iast
BytebuddyAgent [baseline] (837.808 ms) : 0, 837808
BytebuddyAgent [candidate] (839.674 ms) : 0, 839674
GlobalTracer [baseline] (248.414 ms) : 0, 248414
GlobalTracer [candidate] (248.884 ms) : 0, 248884
AppSec [baseline] (58.214 ms) : 0, 58214
AppSec [candidate] (58.234 ms) : 0, 58234
IAST [baseline] (21.519 ms) : 0, 21519
IAST [candidate] (21.257 ms) : 0, 21257
Remote Config [baseline] (684.983 µs) : 0, 685
Remote Config [candidate] (672.87 µs) : 0, 673
Telemetry [baseline] (8.878 ms) : 0, 8878
Telemetry [candidate] (8.715 ms) : 0, 8715
section profiling
BytebuddyAgent [baseline] (703.198 ms) : 0, 703198
BytebuddyAgent [candidate] (703.83 ms) : 0, 703830
GlobalTracer [baseline] (349.707 ms) : 0, 349707
GlobalTracer [candidate] (350.141 ms) : 0, 350141
AppSec [baseline] (54.395 ms) : 0, 54395
AppSec [candidate] (53.646 ms) : 0, 53646
Remote Config [baseline] (669.913 µs) : 0, 670
Remote Config [candidate] (672.206 µs) : 0, 672
Telemetry [baseline] (8.812 ms) : 0, 8812
Telemetry [candidate] (8.856 ms) : 0, 8856
ProfilingAgent [baseline] (95.664 ms) : 0, 95664
ProfilingAgent [candidate] (94.952 ms) : 0, 94952
Profiling [baseline] (95.687 ms) : 0, 95687
Profiling [candidate] (94.977 ms) : 0, 94977
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.46.0-SNAPSHOT~d88adb1f94, baseline=1.46.0-SNAPSHOT~148aef92ab
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.061 s) : 0, 1061086
Total [baseline] (8.633 s) : 0, 8632548
Agent [candidate] (1.076 s) : 0, 1076105
Total [candidate] (8.709 s) : 0, 8709383
section iast
Agent [baseline] (1.19 s) : 0, 1189692
Total [baseline] (9.266 s) : 0, 9266474
Agent [candidate] (1.185 s) : 0, 1185404
Total [candidate] (9.191 s) : 0, 9190858
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.183 s) : 0, 1182925
Total [baseline] (9.158 s) : 0, 9158171
Agent [candidate] (1.184 s) : 0, 1183923
Total [candidate] (9.207 s) : 0, 9207157
section iast_TELEMETRY_OFF
Agent [baseline] (1.186 s) : 0, 1185797
Total [baseline] (9.196 s) : 0, 9195776
Agent [candidate] (1.178 s) : 0, 1177594
Total [candidate] (9.189 s) : 0, 9188856
gantt
title insecure-bank - break down per module: candidate=1.46.0-SNAPSHOT~d88adb1f94, baseline=1.46.0-SNAPSHOT~148aef92ab
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (718.234 ms) : 0, 718234
BytebuddyAgent [candidate] (730.308 ms) : 0, 730308
GlobalTracer [baseline] (257.194 ms) : 0, 257194
GlobalTracer [candidate] (259.757 ms) : 0, 259757
AppSec [baseline] (55.454 ms) : 0, 55454
AppSec [candidate] (56.016 ms) : 0, 56016
Remote Config [baseline] (727.212 µs) : 0, 727
Remote Config [candidate] (729.012 µs) : 0, 729
Telemetry [baseline] (14.34 ms) : 0, 14340
Telemetry [candidate] (13.978 ms) : 0, 13978
section iast
BytebuddyAgent [baseline] (837.017 ms) : 0, 837017
BytebuddyAgent [candidate] (833.785 ms) : 0, 833785
GlobalTracer [baseline] (248.202 ms) : 0, 248202
GlobalTracer [candidate] (247.19 ms) : 0, 247190
AppSec [baseline] (58.127 ms) : 0, 58127
AppSec [candidate] (58.088 ms) : 0, 58088
IAST [baseline] (21.741 ms) : 0, 21741
IAST [candidate] (21.663 ms) : 0, 21663
Remote Config [baseline] (685.318 µs) : 0, 685
Remote Config [candidate] (684.838 µs) : 0, 685
Telemetry [baseline] (8.959 ms) : 0, 8959
Telemetry [candidate] (8.924 ms) : 0, 8924
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (832.055 ms) : 0, 832055
BytebuddyAgent [candidate] (832.406 ms) : 0, 832406
GlobalTracer [baseline] (247.062 ms) : 0, 247062
GlobalTracer [candidate] (247.246 ms) : 0, 247246
AppSec [baseline] (58.038 ms) : 0, 58038
AppSec [candidate] (58.12 ms) : 0, 58120
IAST [baseline] (21.291 ms) : 0, 21291
IAST [candidate] (21.677 ms) : 0, 21677
Remote Config [baseline] (668.904 µs) : 0, 669
Remote Config [candidate] (677.985 µs) : 0, 678
Telemetry [baseline] (8.724 ms) : 0, 8724
Telemetry [candidate] (8.745 ms) : 0, 8745
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (834.536 ms) : 0, 834536
BytebuddyAgent [candidate] (827.76 ms) : 0, 827760
GlobalTracer [baseline] (248.309 ms) : 0, 248309
GlobalTracer [candidate] (246.591 ms) : 0, 246591
AppSec [baseline] (57.699 ms) : 0, 57699
AppSec [candidate] (57.968 ms) : 0, 57968
IAST [baseline] (20.835 ms) : 0, 20835
IAST [candidate] (20.966 ms) : 0, 20966
Remote Config [baseline] (671.586 µs) : 0, 672
Remote Config [candidate] (675.041 µs) : 0, 675
Telemetry [baseline] (8.595 ms) : 0, 8595
Telemetry [candidate] (8.625 ms) : 0, 8625
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 13 metrics, 15 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.46.0-SNAPSHOT~d88adb1f94, baseline=1.46.0-SNAPSHOT~148aef92ab
dateFormat X
axisFormat %s
section baseline
no_agent (379.764 µs) : 360, 400
. : milestone, 380,
iast (515.533 µs) : 493, 538
. : milestone, 516,
iast_FULL (746.971 µs) : 725, 769
. : milestone, 747,
iast_GLOBAL (565.805 µs) : 543, 589
. : milestone, 566,
iast_HARDCODED_SECRET_DISABLED (512.43 µs) : 491, 534
. : milestone, 512,
iast_INACTIVE (460.55 µs) : 440, 482
. : milestone, 461,
iast_TELEMETRY_OFF (499.02 µs) : 477, 521
. : milestone, 499,
tracing (455.007 µs) : 434, 476
. : milestone, 455,
section candidate
no_agent (381.071 µs) : 361, 401
. : milestone, 381,
iast (511.866 µs) : 490, 534
. : milestone, 512,
iast_FULL (747.17 µs) : 725, 770
. : milestone, 747,
iast_GLOBAL (558.893 µs) : 537, 581
. : milestone, 559,
iast_HARDCODED_SECRET_DISABLED (510.656 µs) : 489, 532
. : milestone, 511,
iast_INACTIVE (463.926 µs) : 443, 485
. : milestone, 464,
iast_TELEMETRY_OFF (503.791 µs) : 481, 526
. : milestone, 504,
tracing (455.481 µs) : 435, 476
. : milestone, 455,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.46.0-SNAPSHOT~d88adb1f94, baseline=1.46.0-SNAPSHOT~148aef92ab
dateFormat X
axisFormat %s
section baseline
no_agent (1.369 ms) : 1349, 1388
. : milestone, 1369,
appsec (1.777 ms) : 1753, 1801
. : milestone, 1777,
appsec_no_iast (1.767 ms) : 1743, 1791
. : milestone, 1767,
iast (1.522 ms) : 1499, 1546
. : milestone, 1522,
profiling (1.554 ms) : 1527, 1581
. : milestone, 1554,
tracing (1.501 ms) : 1477, 1526
. : milestone, 1501,
section candidate
no_agent (1.369 ms) : 1349, 1389
. : milestone, 1369,
appsec (1.753 ms) : 1729, 1777
. : milestone, 1753,
appsec_no_iast (1.739 ms) : 1715, 1762
. : milestone, 1739,
iast (1.502 ms) : 1477, 1526
. : milestone, 1502,
profiling (1.533 ms) : 1509, 1558
. : milestone, 1533,
tracing (1.488 ms) : 1462, 1513
. : milestone, 1488,
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 tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.46.0-SNAPSHOT~d88adb1f94, baseline=1.46.0-SNAPSHOT~148aef92ab
dateFormat X
axisFormat %s
section baseline
no_agent (1.477 ms) : 1466, 1489
. : milestone, 1477,
appsec (2.358 ms) : 2315, 2400
. : milestone, 2358,
iast (2.112 ms) : 2058, 2167
. : milestone, 2112,
iast_GLOBAL (2.153 ms) : 2099, 2207
. : milestone, 2153,
profiling (1.968 ms) : 1925, 2011
. : milestone, 1968,
tracing (1.955 ms) : 1913, 1997
. : milestone, 1955,
section candidate
no_agent (1.482 ms) : 1471, 1494
. : milestone, 1482,
appsec (2.378 ms) : 2335, 2421
. : milestone, 2378,
iast (2.114 ms) : 2060, 2169
. : milestone, 2114,
iast_GLOBAL (2.164 ms) : 2109, 2219
. : milestone, 2164,
profiling (1.986 ms) : 1942, 2030
. : milestone, 1986,
tracing (1.955 ms) : 1913, 1997
. : milestone, 1955,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.46.0-SNAPSHOT~d88adb1f94, baseline=1.46.0-SNAPSHOT~148aef92ab
dateFormat X
axisFormat %s
section baseline
no_agent (15.211 s) : 15211000, 15211000
. : milestone, 15211000,
appsec (15.196 s) : 15196000, 15196000
. : milestone, 15196000,
iast (18.808 s) : 18808000, 18808000
. : milestone, 18808000,
iast_GLOBAL (17.952 s) : 17952000, 17952000
. : milestone, 17952000,
profiling (15.054 s) : 15054000, 15054000
. : milestone, 15054000,
tracing (14.908 s) : 14908000, 14908000
. : milestone, 14908000,
section candidate
no_agent (14.726 s) : 14726000, 14726000
. : milestone, 14726000,
appsec (15.305 s) : 15305000, 15305000
. : milestone, 15305000,
iast (18.816 s) : 18816000, 18816000
. : milestone, 18816000,
iast_GLOBAL (18.382 s) : 18382000, 18382000
. : milestone, 18382000,
profiling (14.97 s) : 14970000, 14970000
. : milestone, 14970000,
tracing (15.12 s) : 15120000, 15120000
. : milestone, 15120000,
|
What Does This Do
Move blocking behavior to method enter in vert.x session tracking.
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]