-
Notifications
You must be signed in to change notification settings - Fork 277
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
unwrap akka TaskInvocation
and scala CallbackRunnable
#6975
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
changed the title
unwrap akka TaskInvocations
unwrap akka Apr 30, 2024
TaskInvocation
and scala CallbackRunnable
richardstartin
added
comp: profiling
Profiling
inst: scala
Scala instrumentation
inst: akka
Akka instrumentation
labels
Apr 30, 2024
jbachorik
approved these changes
Apr 30, 2024
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 50 metrics, 13 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.34.0-SNAPSHOT~13553aefad, baseline=1.34.0-SNAPSHOT~fcfaf54d06
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.074 s) : 0, 1074197
Total [baseline] (8.549 s) : 0, 8548679
Agent [candidate] (1.077 s) : 0, 1077016
Total [candidate] (8.578 s) : 0, 8577574
section iast
Agent [baseline] (1.212 s) : 0, 1212331
Total [baseline] (9.069 s) : 0, 9069057
Agent [candidate] (1.202 s) : 0, 1202388
Total [candidate] (9.015 s) : 0, 9015264
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.203 s) : 0, 1202981
Total [baseline] (8.985 s) : 0, 8984956
Agent [candidate] (1.199 s) : 0, 1198670
Total [candidate] (9.0 s) : 0, 8999906
section iast_TELEMETRY_OFF
Agent [baseline] (1.206 s) : 0, 1206351
Total [baseline] (9.04 s) : 0, 9039931
Agent [candidate] (1.205 s) : 0, 1204974
Total [candidate] (9.011 s) : 0, 9010939
gantt
title insecure-bank - break down per module: candidate=1.34.0-SNAPSHOT~13553aefad, baseline=1.34.0-SNAPSHOT~fcfaf54d06
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (672.705 ms) : 0, 672705
BytebuddyAgent [candidate] (674.377 ms) : 0, 674377
GlobalTracer [baseline] (309.343 ms) : 0, 309343
GlobalTracer [candidate] (310.244 ms) : 0, 310244
AppSec [baseline] (49.512 ms) : 0, 49512
AppSec [candidate] (49.672 ms) : 0, 49672
Remote Config [baseline] (663.588 µs) : 0, 664
Remote Config [candidate] (670.631 µs) : 0, 671
Telemetry [baseline] (7.645 ms) : 0, 7645
Telemetry [candidate] (7.672 ms) : 0, 7672
section iast
BytebuddyAgent [baseline] (803.105 ms) : 0, 803105
BytebuddyAgent [candidate] (795.883 ms) : 0, 795883
GlobalTracer [baseline] (291.246 ms) : 0, 291246
GlobalTracer [candidate] (289.35 ms) : 0, 289350
AppSec [baseline] (50.314 ms) : 0, 50314
AppSec [candidate] (50.456 ms) : 0, 50456
IAST [baseline] (25.745 ms) : 0, 25745
IAST [candidate] (24.986 ms) : 0, 24986
Remote Config [baseline] (593.973 µs) : 0, 594
Remote Config [candidate] (737.774 µs) : 0, 738
Telemetry [baseline] (6.643 ms) : 0, 6643
Telemetry [candidate] (6.652 ms) : 0, 6652
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (796.515 ms) : 0, 796515
BytebuddyAgent [candidate] (793.45 ms) : 0, 793450
GlobalTracer [baseline] (289.194 ms) : 0, 289194
GlobalTracer [candidate] (288.812 ms) : 0, 288812
AppSec [baseline] (50.345 ms) : 0, 50345
AppSec [candidate] (49.082 ms) : 0, 49082
IAST [baseline] (24.415 ms) : 0, 24415
IAST [candidate] (24.886 ms) : 0, 24886
Remote Config [baseline] (1.347 ms) : 0, 1347
Remote Config [candidate] (1.304 ms) : 0, 1304
Telemetry [baseline] (6.752 ms) : 0, 6752
Telemetry [candidate] (6.709 ms) : 0, 6709
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (798.524 ms) : 0, 798524
BytebuddyAgent [candidate] (797.599 ms) : 0, 797599
GlobalTracer [baseline] (290.126 ms) : 0, 290126
GlobalTracer [candidate] (290.461 ms) : 0, 290461
AppSec [baseline] (51.224 ms) : 0, 51224
AppSec [candidate] (52.246 ms) : 0, 52246
IAST [baseline] (24.668 ms) : 0, 24668
IAST [candidate] (22.835 ms) : 0, 22835
Remote Config [baseline] (588.575 µs) : 0, 589
Remote Config [candidate] (583.082 µs) : 0, 583
Telemetry [baseline] (6.691 ms) : 0, 6691
Telemetry [candidate] (6.631 ms) : 0, 6631
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.34.0-SNAPSHOT~13553aefad, baseline=1.34.0-SNAPSHOT~fcfaf54d06
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.076 s) : 0, 1076250
Total [baseline] (10.47 s) : 0, 10470405
Agent [candidate] (1.077 s) : 0, 1077211
Total [candidate] (10.377 s) : 0, 10377011
section appsec
Agent [baseline] (1.198 s) : 0, 1198325
Total [baseline] (10.518 s) : 0, 10517927
Agent [candidate] (1.194 s) : 0, 1194008
Total [candidate] (10.534 s) : 0, 10533530
section iast
Agent [baseline] (1.202 s) : 0, 1202023
Total [baseline] (10.823 s) : 0, 10822927
Agent [candidate] (1.209 s) : 0, 1209031
Total [candidate] (10.794 s) : 0, 10793508
section profiling
Agent [baseline] (1.268 s) : 0, 1268484
Total [baseline] (10.541 s) : 0, 10541483
Agent [candidate] (1.268 s) : 0, 1268135
Total [candidate] (10.614 s) : 0, 10613910
gantt
title petclinic - break down per module: candidate=1.34.0-SNAPSHOT~13553aefad, baseline=1.34.0-SNAPSHOT~fcfaf54d06
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (674.219 ms) : 0, 674219
BytebuddyAgent [candidate] (674.863 ms) : 0, 674863
GlobalTracer [baseline] (310.088 ms) : 0, 310088
GlobalTracer [candidate] (310.265 ms) : 0, 310265
AppSec [baseline] (49.304 ms) : 0, 49304
AppSec [candidate] (49.409 ms) : 0, 49409
Remote Config [baseline] (660.003 µs) : 0, 660
Remote Config [candidate] (652.805 µs) : 0, 653
Telemetry [baseline] (7.618 ms) : 0, 7618
Telemetry [candidate] (7.556 ms) : 0, 7556
section appsec
BytebuddyAgent [baseline] (702.274 ms) : 0, 702274
BytebuddyAgent [candidate] (698.1 ms) : 0, 698100
GlobalTracer [baseline] (293.1 ms) : 0, 293100
GlobalTracer [candidate] (292.908 ms) : 0, 292908
AppSec [baseline] (149.665 ms) : 0, 149665
AppSec [candidate] (149.372 ms) : 0, 149372
Remote Config [baseline] (628.771 µs) : 0, 629
Remote Config [candidate] (628.162 µs) : 0, 628
Telemetry [baseline] (8.249 ms) : 0, 8249
Telemetry [candidate] (8.82 ms) : 0, 8820
IAST [baseline] (19.322 ms) : 0, 19322
IAST [candidate] (19.242 ms) : 0, 19242
section iast
BytebuddyAgent [baseline] (796.001 ms) : 0, 796001
BytebuddyAgent [candidate] (800.8 ms) : 0, 800800
GlobalTracer [baseline] (289.021 ms) : 0, 289021
GlobalTracer [candidate] (290.416 ms) : 0, 290416
AppSec [baseline] (48.948 ms) : 0, 48948
AppSec [candidate] (50.814 ms) : 0, 50814
Remote Config [baseline] (1.28 ms) : 0, 1280
Remote Config [candidate] (2.102 ms) : 0, 2102
Telemetry [baseline] (7.352 ms) : 0, 7352
Telemetry [candidate] (7.517 ms) : 0, 7517
IAST [baseline] (25.041 ms) : 0, 25041
IAST [candidate] (22.744 ms) : 0, 22744
section profiling
ProfilingAgent [baseline] (95.506 ms) : 0, 95506
ProfilingAgent [candidate] (95.6 ms) : 0, 95600
BytebuddyAgent [baseline] (677.914 ms) : 0, 677914
BytebuddyAgent [candidate] (677.465 ms) : 0, 677465
GlobalTracer [baseline] (380.25 ms) : 0, 380250
GlobalTracer [candidate] (380.592 ms) : 0, 380592
AppSec [baseline] (50.23 ms) : 0, 50230
AppSec [candidate] (49.968 ms) : 0, 49968
Remote Config [baseline] (710.773 µs) : 0, 711
Remote Config [candidate] (697.135 µs) : 0, 697
Telemetry [baseline] (7.456 ms) : 0, 7456
Telemetry [candidate] (7.442 ms) : 0, 7442
Profiling [baseline] (95.53 ms) : 0, 95530
Profiling [candidate] (95.624 ms) : 0, 95624
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.34.0-SNAPSHOT~13553aefad, baseline=1.34.0-SNAPSHOT~fcfaf54d06
dateFormat X
axisFormat %s
section baseline
no_agent (371.515 µs) : 351, 392
. : milestone, 372,
iast (477.267 µs) : 457, 498
. : milestone, 477,
iast_FULL (547.013 µs) : 526, 568
. : milestone, 547,
iast_GLOBAL (505.968 µs) : 485, 527
. : milestone, 506,
iast_HARDCODED_SECRET_DISABLED (478.837 µs) : 457, 500
. : milestone, 479,
iast_INACTIVE (453.565 µs) : 433, 474
. : milestone, 454,
iast_TELEMETRY_OFF (476.035 µs) : 455, 497
. : milestone, 476,
tracing (455.295 µs) : 435, 476
. : milestone, 455,
section candidate
no_agent (373.975 µs) : 355, 393
. : milestone, 374,
iast (481.879 µs) : 461, 503
. : milestone, 482,
iast_FULL (546.203 µs) : 526, 567
. : milestone, 546,
iast_GLOBAL (498.614 µs) : 478, 519
. : milestone, 499,
iast_HARDCODED_SECRET_DISABLED (478.114 µs) : 456, 500
. : milestone, 478,
iast_INACTIVE (459.484 µs) : 438, 481
. : milestone, 459,
iast_TELEMETRY_OFF (477.603 µs) : 456, 499
. : milestone, 478,
tracing (451.952 µs) : 431, 473
. : milestone, 452,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.34.0-SNAPSHOT~13553aefad, baseline=1.34.0-SNAPSHOT~fcfaf54d06
dateFormat X
axisFormat %s
section baseline
no_agent (1.338 ms) : 1319, 1358
. : milestone, 1338,
appsec (1.726 ms) : 1702, 1751
. : milestone, 1726,
appsec_no_iast (1.71 ms) : 1685, 1734
. : milestone, 1710,
iast (1.496 ms) : 1473, 1519
. : milestone, 1496,
profiling (1.487 ms) : 1463, 1512
. : milestone, 1487,
tracing (1.498 ms) : 1474, 1523
. : milestone, 1498,
section candidate
no_agent (1.345 ms) : 1326, 1365
. : milestone, 1345,
appsec (1.706 ms) : 1681, 1730
. : milestone, 1706,
appsec_no_iast (1.711 ms) : 1686, 1737
. : milestone, 1711,
iast (1.49 ms) : 1467, 1513
. : milestone, 1490,
profiling (1.499 ms) : 1475, 1524
. : milestone, 1499,
tracing (1.501 ms) : 1477, 1525
. : milestone, 1501,
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.34.0-SNAPSHOT~13553aefad, baseline=1.34.0-SNAPSHOT~fcfaf54d06
dateFormat X
axisFormat %s
section baseline
no_agent (1.472 ms) : 1460, 1483
. : milestone, 1472,
appsec (2.208 ms) : 2175, 2242
. : milestone, 2208,
iast (1.885 ms) : 1850, 1920
. : milestone, 1885,
iast_GLOBAL (1.93 ms) : 1895, 1965
. : milestone, 1930,
profiling (1.861 ms) : 1828, 1895
. : milestone, 1861,
tracing (1.841 ms) : 1809, 1873
. : milestone, 1841,
section candidate
no_agent (1.467 ms) : 1456, 1479
. : milestone, 1467,
appsec (2.208 ms) : 2175, 2242
. : milestone, 2208,
iast (1.89 ms) : 1855, 1925
. : milestone, 1890,
iast_GLOBAL (1.925 ms) : 1891, 1960
. : milestone, 1925,
profiling (1.863 ms) : 1829, 1897
. : milestone, 1863,
tracing (1.838 ms) : 1806, 1869
. : milestone, 1838,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.34.0-SNAPSHOT~13553aefad, baseline=1.34.0-SNAPSHOT~fcfaf54d06
dateFormat X
axisFormat %s
section baseline
no_agent (15.652 s) : 15652000, 15652000
. : milestone, 15652000,
appsec (15.137 s) : 15137000, 15137000
. : milestone, 15137000,
iast (19.06 s) : 19060000, 19060000
. : milestone, 19060000,
iast_GLOBAL (18.009 s) : 18009000, 18009000
. : milestone, 18009000,
profiling (15.813 s) : 15813000, 15813000
. : milestone, 15813000,
tracing (15.114 s) : 15114000, 15114000
. : milestone, 15114000,
section candidate
no_agent (15.263 s) : 15263000, 15263000
. : milestone, 15263000,
appsec (14.936 s) : 14936000, 14936000
. : milestone, 14936000,
iast (18.618 s) : 18618000, 18618000
. : milestone, 18618000,
iast_GLOBAL (17.984 s) : 17984000, 17984000
. : milestone, 17984000,
profiling (15.352 s) : 15352000, 15352000
. : milestone, 15352000,
tracing (15.019 s) : 15019000, 15019000
. : milestone, 15019000,
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
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
Makes akka
TaskInvocation
instances unwrappable so we can access the name of theRunnable
they carry.Also allows access to Scala's
CalbackRunnable
'sonComplete
field.Motivation
This will allow the time spent queueing by
TaskInvocation
s to be disaggregated.Additional Notes
Jira ticket: PROF-9696