-
Notifications
You must be signed in to change notification settings - Fork 312
Refactor akka instrumentation modules #9060
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
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,
|
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.
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 |
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
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
,fix
or any linking keywords when referencing an issue.Use
solves
instead, and assign the PR milestone to the issueJira ticket: LANGPLAT-79