Refactor akka instrumentation modules#9060
Conversation
|
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
a148425 to
03800ba
Compare
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 44 metrics, 9 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.52.0-SNAPSHOT~f16dbb8bcf, baseline=1.52.0-SNAPSHOT~ad6d6b9786
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (993.022 ms) : 0, 993022
Total [baseline] (10.59 s) : 0, 10589743
Agent [candidate] (995.22 ms) : 0, 995220
Total [candidate] (10.642 s) : 0, 10641769
section appsec
Agent [baseline] (1.171 s) : 0, 1171447
Total [baseline] (10.747 s) : 0, 10747348
Agent [candidate] (1.182 s) : 0, 1181976
Total [candidate] (10.773 s) : 0, 10773299
section iast
Agent [baseline] (1.135 s) : 0, 1135382
Total [baseline] (10.826 s) : 0, 10826261
Agent [candidate] (1.141 s) : 0, 1140828
Total [candidate] (10.912 s) : 0, 10911597
section profiling
Agent [baseline] (1.252 s) : 0, 1251637
Total [baseline] (10.955 s) : 0, 10955211
Agent [candidate] (1.242 s) : 0, 1242445
Total [candidate] (10.932 s) : 0, 10932299
gantt
title petclinic - break down per module: candidate=1.52.0-SNAPSHOT~f16dbb8bcf, baseline=1.52.0-SNAPSHOT~ad6d6b9786
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (685.772 ms) : 0, 685772
BytebuddyAgent [candidate] (687.043 ms) : 0, 687043
GlobalTracer [baseline] (241.511 ms) : 0, 241511
GlobalTracer [candidate] (242.113 ms) : 0, 242113
AppSec [baseline] (30.174 ms) : 0, 30174
AppSec [candidate] (30.241 ms) : 0, 30241
Debugger [baseline] (5.947 ms) : 0, 5947
Debugger [candidate] (6.013 ms) : 0, 6013
Remote Config [baseline] (670.9 µs) : 0, 671
Remote Config [candidate] (673.857 µs) : 0, 674
Telemetry [baseline] (8.16 ms) : 0, 8160
Telemetry [candidate] (8.254 ms) : 0, 8254
section appsec
BytebuddyAgent [baseline] (709.091 ms) : 0, 709091
BytebuddyAgent [candidate] (715.431 ms) : 0, 715431
GlobalTracer [baseline] (234.654 ms) : 0, 234654
GlobalTracer [candidate] (236.354 ms) : 0, 236354
IAST [baseline] (23.54 ms) : 0, 23540
IAST [candidate] (23.586 ms) : 0, 23586
AppSec [baseline] (169.024 ms) : 0, 169024
AppSec [candidate] (171.231 ms) : 0, 171231
Debugger [baseline] (5.729 ms) : 0, 5729
Debugger [candidate] (5.743 ms) : 0, 5743
Remote Config [baseline] (607.195 µs) : 0, 607
Remote Config [candidate] (599.772 µs) : 0, 600
Telemetry [baseline] (8.077 ms) : 0, 8077
Telemetry [candidate] (8.015 ms) : 0, 8015
section iast
BytebuddyAgent [baseline] (808.614 ms) : 0, 808614
BytebuddyAgent [candidate] (813.382 ms) : 0, 813382
GlobalTracer [baseline] (233.196 ms) : 0, 233196
GlobalTracer [candidate] (233.706 ms) : 0, 233706
IAST [baseline] (28.412 ms) : 0, 28412
IAST [candidate] (25.736 ms) : 0, 25736
AppSec [baseline] (29.099 ms) : 0, 29099
AppSec [candidate] (31.878 ms) : 0, 31878
Debugger [baseline] (5.816 ms) : 0, 5816
Debugger [candidate] (6.659 ms) : 0, 6659
Remote Config [baseline] (596.612 µs) : 0, 597
Remote Config [candidate] (585.036 µs) : 0, 585
Telemetry [baseline] (8.047 ms) : 0, 8047
Telemetry [candidate] (7.981 ms) : 0, 7981
section profiling
BytebuddyAgent [baseline] (682.344 ms) : 0, 682344
BytebuddyAgent [candidate] (676.787 ms) : 0, 676787
GlobalTracer [baseline] (363.06 ms) : 0, 363060
GlobalTracer [candidate] (360.436 ms) : 0, 360436
AppSec [baseline] (32.485 ms) : 0, 32485
AppSec [candidate] (30.87 ms) : 0, 30870
Debugger [baseline] (11.515 ms) : 0, 11515
Debugger [candidate] (11.19 ms) : 0, 11190
Remote Config [baseline] (667.692 µs) : 0, 668
Remote Config [candidate] (661.17 µs) : 0, 661
Telemetry [baseline] (8.765 ms) : 0, 8765
Telemetry [candidate] (9.557 ms) : 0, 9557
ProfilingAgent [baseline] (103.907 ms) : 0, 103907
ProfilingAgent [candidate] (104.409 ms) : 0, 104409
Profiling [baseline] (103.931 ms) : 0, 103931
Profiling [candidate] (104.433 ms) : 0, 104433
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.52.0-SNAPSHOT~f16dbb8bcf, baseline=1.52.0-SNAPSHOT~ad6d6b9786
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (993.866 ms) : 0, 993866
Total [baseline] (8.581 s) : 0, 8581177
Agent [candidate] (997.359 ms) : 0, 997359
Total [candidate] (8.552 s) : 0, 8551691
section iast
Agent [baseline] (1.13 s) : 0, 1129545
Total [baseline] (9.247 s) : 0, 9247265
Agent [candidate] (1.141 s) : 0, 1141051
Total [candidate] (9.312 s) : 0, 9311881
gantt
title insecure-bank - break down per module: candidate=1.52.0-SNAPSHOT~f16dbb8bcf, baseline=1.52.0-SNAPSHOT~ad6d6b9786
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (686.803 ms) : 0, 686803
BytebuddyAgent [candidate] (688.534 ms) : 0, 688534
GlobalTracer [baseline] (241.282 ms) : 0, 241282
GlobalTracer [candidate] (242.716 ms) : 0, 242716
AppSec [baseline] (30.181 ms) : 0, 30181
AppSec [candidate] (30.383 ms) : 0, 30383
Debugger [baseline] (5.966 ms) : 0, 5966
Debugger [candidate] (6.02 ms) : 0, 6020
Remote Config [baseline] (676.045 µs) : 0, 676
Remote Config [candidate] (676.936 µs) : 0, 677
Telemetry [baseline] (8.257 ms) : 0, 8257
Telemetry [candidate] (8.189 ms) : 0, 8189
section iast
BytebuddyAgent [baseline] (805.897 ms) : 0, 805897
BytebuddyAgent [candidate] (814.365 ms) : 0, 814365
GlobalTracer [baseline] (231.496 ms) : 0, 231496
GlobalTracer [candidate] (233.053 ms) : 0, 233053
IAST [baseline] (28.701 ms) : 0, 28701
IAST [candidate] (27.584 ms) : 0, 27584
AppSec [baseline] (28.624 ms) : 0, 28624
AppSec [candidate] (30.758 ms) : 0, 30758
Debugger [baseline] (5.706 ms) : 0, 5706
Debugger [candidate] (5.777 ms) : 0, 5777
Remote Config [baseline] (580.805 µs) : 0, 581
Remote Config [candidate] (601.159 µs) : 0, 601
Telemetry [baseline] (7.885 ms) : 0, 7885
Telemetry [candidate] (8.007 ms) : 0, 8007
LoadParameters
See matching parameters
SummaryFound 2 performance improvements and 1 performance regressions! Performance is the same for 9 metrics, 12 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.52.0-SNAPSHOT~f16dbb8bcf, baseline=1.52.0-SNAPSHOT~ad6d6b9786
dateFormat X
axisFormat %s
section baseline
no_agent (37.04 ms) : 36733, 37347
. : milestone, 37040,
appsec (45.297 ms) : 44902, 45691
. : milestone, 45297,
code_origins (44.445 ms) : 44064, 44826
. : milestone, 44445,
iast (44.417 ms) : 44031, 44802
. : milestone, 44417,
profiling (47.292 ms) : 46848, 47736
. : milestone, 47292,
tracing (44.394 ms) : 44029, 44759
. : milestone, 44394,
section candidate
no_agent (36.786 ms) : 36491, 37082
. : milestone, 36786,
appsec (46.079 ms) : 45675, 46483
. : milestone, 46079,
code_origins (44.467 ms) : 44101, 44833
. : milestone, 44467,
iast (43.695 ms) : 43308, 44083
. : milestone, 43695,
profiling (47.779 ms) : 47351, 48206
. : milestone, 47779,
tracing (44.654 ms) : 44278, 45029
. : milestone, 44654,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.52.0-SNAPSHOT~f16dbb8bcf, baseline=1.52.0-SNAPSHOT~ad6d6b9786
dateFormat X
axisFormat %s
section baseline
no_agent (4.485 ms) : 4432, 4539
. : milestone, 4485,
iast (9.494 ms) : 9322, 9665
. : milestone, 9494,
iast_FULL (13.754 ms) : 13482, 14026
. : milestone, 13754,
iast_GLOBAL (10.215 ms) : 10037, 10393
. : milestone, 10215,
profiling (8.956 ms) : 8819, 9094
. : milestone, 8956,
tracing (7.556 ms) : 7442, 7670
. : milestone, 7556,
section candidate
no_agent (4.235 ms) : 4187, 4284
. : milestone, 4235,
iast (9.372 ms) : 9219, 9525
. : milestone, 9372,
iast_FULL (13.92 ms) : 13648, 14193
. : milestone, 13920,
iast_GLOBAL (10.752 ms) : 10552, 10952
. : milestone, 10752,
profiling (8.595 ms) : 8455, 8735
. : milestone, 8595,
tracing (7.436 ms) : 7324, 7548
. : milestone, 7436,
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.52.0-SNAPSHOT~f16dbb8bcf, baseline=1.52.0-SNAPSHOT~ad6d6b9786
dateFormat X
axisFormat %s
section baseline
no_agent (1.478 ms) : 1466, 1489
. : milestone, 1478,
appsec (2.405 ms) : 2356, 2455
. : milestone, 2405,
iast (2.198 ms) : 2136, 2261
. : milestone, 2198,
iast_GLOBAL (2.237 ms) : 2175, 2300
. : milestone, 2237,
profiling (2.043 ms) : 1993, 2093
. : milestone, 2043,
tracing (2.006 ms) : 1958, 2054
. : milestone, 2006,
section candidate
no_agent (1.472 ms) : 1461, 1484
. : milestone, 1472,
appsec (2.411 ms) : 2362, 2461
. : milestone, 2411,
iast (2.188 ms) : 2126, 2250
. : milestone, 2188,
iast_GLOBAL (2.231 ms) : 2169, 2294
. : milestone, 2231,
profiling (2.484 ms) : 2308, 2659
. : milestone, 2484,
tracing (2.01 ms) : 1962, 2059
. : milestone, 2010,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.52.0-SNAPSHOT~f16dbb8bcf, baseline=1.52.0-SNAPSHOT~ad6d6b9786
dateFormat X
axisFormat %s
section baseline
no_agent (15.02 s) : 15020000, 15020000
. : milestone, 15020000,
appsec (15.003 s) : 15003000, 15003000
. : milestone, 15003000,
iast (18.723 s) : 18723000, 18723000
. : milestone, 18723000,
iast_GLOBAL (17.688 s) : 17688000, 17688000
. : milestone, 17688000,
profiling (14.921 s) : 14921000, 14921000
. : milestone, 14921000,
tracing (14.635 s) : 14635000, 14635000
. : milestone, 14635000,
section candidate
no_agent (14.983 s) : 14983000, 14983000
. : milestone, 14983000,
appsec (14.97 s) : 14970000, 14970000
. : milestone, 14970000,
iast (18.454 s) : 18454000, 18454000
. : milestone, 18454000,
iast_GLOBAL (18.046 s) : 18046000, 18046000
. : milestone, 18046000,
profiling (15.808 s) : 15808000, 15808000
. : milestone, 15808000,
tracing (14.821 s) : 14821000, 14821000
. : milestone, 14821000,
|
PerfectSlayer
left a comment
There was a problem hiding this comment.
Why would akka-http has its own sub-folder while play don't?
Do we want to always have both major and minor version for consistency? (eg play-ws-2 to play-ws-2.0)
Time for one-pager :)
Play will but it's not part of this pull request. Let's keep the changes as little as we can |
03800ba to
f16dbb8
Compare
What Does This Do
Improves repository instrumentation structure according to the naming conventions.
Changes made:
Motivation
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any usefull labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: LANGPLAT-79