-
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
Fix default value for long-running spans with DJM #7795
Merged
paul-laffon-dd
merged 1 commit into
master
from
paul.laffon/default-long-running-span-djm
Oct 22, 2024
Merged
Fix default value for long-running spans with DJM #7795
paul-laffon-dd
merged 1 commit into
master
from
paul.laffon/default-long-running-span-djm
Oct 22, 2024
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
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 petclinicgantt
title petclinic - global startup overhead: candidate=1.42.0-SNAPSHOT~35b4f26f84, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.085 s) : 0, 1084612
Total [baseline] (10.402 s) : 0, 10402060
Agent [candidate] (1.079 s) : 0, 1078608
Total [candidate] (10.473 s) : 0, 10472602
section appsec
Agent [baseline] (1.215 s) : 0, 1215252
Total [baseline] (10.64 s) : 0, 10640271
Agent [candidate] (1.212 s) : 0, 1211811
Total [candidate] (10.598 s) : 0, 10598201
section iast
Agent [baseline] (1.223 s) : 0, 1223294
Total [baseline] (10.974 s) : 0, 10973705
Agent [candidate] (1.203 s) : 0, 1202883
Total [candidate] (10.924 s) : 0, 10923786
section profiling
Agent [baseline] (1.281 s) : 0, 1281434
Total [baseline] (10.765 s) : 0, 10765381
Agent [candidate] (1.285 s) : 0, 1284711
Total [candidate] (10.701 s) : 0, 10700853
gantt
title petclinic - break down per module: candidate=1.42.0-SNAPSHOT~35b4f26f84, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (691.312 ms) : 0, 691312
BytebuddyAgent [candidate] (688.049 ms) : 0, 688049
GlobalTracer [baseline] (314.938 ms) : 0, 314938
GlobalTracer [candidate] (313.07 ms) : 0, 313070
AppSec [baseline] (54.265 ms) : 0, 54265
AppSec [candidate] (54.101 ms) : 0, 54101
Remote Config [baseline] (674.106 µs) : 0, 674
Remote Config [candidate] (664.381 µs) : 0, 664
Telemetry [baseline] (9.599 ms) : 0, 9599
Telemetry [candidate] (8.963 ms) : 0, 8963
section appsec
BytebuddyAgent [baseline] (705.556 ms) : 0, 705556
BytebuddyAgent [candidate] (703.563 ms) : 0, 703563
GlobalTracer [baseline] (311.102 ms) : 0, 311102
GlobalTracer [candidate] (310.302 ms) : 0, 310302
AppSec [baseline] (166.123 ms) : 0, 166123
AppSec [candidate] (166.761 ms) : 0, 166761
Remote Config [baseline] (640.09 µs) : 0, 640
Remote Config [candidate] (636.79 µs) : 0, 637
Telemetry [baseline] (7.859 ms) : 0, 7859
Telemetry [candidate] (7.124 ms) : 0, 7124
IAST [baseline] (20.393 ms) : 0, 20393
IAST [candidate] (19.341 ms) : 0, 19341
section iast
BytebuddyAgent [baseline] (814.104 ms) : 0, 814104
BytebuddyAgent [candidate] (801.923 ms) : 0, 801923
GlobalTracer [baseline] (307.152 ms) : 0, 307152
GlobalTracer [candidate] (301.68 ms) : 0, 301680
AppSec [baseline] (56.625 ms) : 0, 56625
AppSec [candidate] (57.329 ms) : 0, 57329
Remote Config [baseline] (633.802 µs) : 0, 634
Remote Config [candidate] (606.109 µs) : 0, 606
Telemetry [baseline] (7.307 ms) : 0, 7307
Telemetry [candidate] (7.037 ms) : 0, 7037
IAST [baseline] (23.519 ms) : 0, 23519
IAST [candidate] (20.556 ms) : 0, 20556
section profiling
ProfilingAgent [baseline] (91.169 ms) : 0, 91169
ProfilingAgent [candidate] (91.274 ms) : 0, 91274
BytebuddyAgent [baseline] (684.825 ms) : 0, 684825
BytebuddyAgent [candidate] (685.822 ms) : 0, 685822
GlobalTracer [baseline] (398.27 ms) : 0, 398270
GlobalTracer [candidate] (398.629 ms) : 0, 398629
AppSec [baseline] (55.07 ms) : 0, 55070
AppSec [candidate] (55.22 ms) : 0, 55220
Remote Config [baseline] (658.497 µs) : 0, 658
Remote Config [candidate] (665.058 µs) : 0, 665
Telemetry [baseline] (12.228 ms) : 0, 12228
Telemetry [candidate] (13.921 ms) : 0, 13921
Profiling [baseline] (91.193 ms) : 0, 91193
Profiling [candidate] (91.298 ms) : 0, 91298
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.42.0-SNAPSHOT~35b4f26f84, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.078 s) : 0, 1077594
Total [baseline] (8.571 s) : 0, 8571394
Agent [candidate] (1.086 s) : 0, 1085783
Total [candidate] (8.572 s) : 0, 8572377
section iast
Agent [baseline] (1.213 s) : 0, 1213419
Total [baseline] (9.123 s) : 0, 9122826
Agent [candidate] (1.206 s) : 0, 1205731
Total [candidate] (9.224 s) : 0, 9224239
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.214 s) : 0, 1213585
Total [baseline] (9.144 s) : 0, 9143939
Agent [candidate] (1.205 s) : 0, 1205030
Total [candidate] (9.089 s) : 0, 9088805
section iast_TELEMETRY_OFF
Agent [baseline] (1.2 s) : 0, 1199746
Total [baseline] (9.116 s) : 0, 9115747
Agent [candidate] (1.203 s) : 0, 1203164
Total [candidate] (9.098 s) : 0, 9097697
gantt
title insecure-bank - break down per module: candidate=1.42.0-SNAPSHOT~35b4f26f84, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (687.301 ms) : 0, 687301
BytebuddyAgent [candidate] (691.687 ms) : 0, 691687
GlobalTracer [baseline] (313.566 ms) : 0, 313566
GlobalTracer [candidate] (314.864 ms) : 0, 314864
AppSec [baseline] (54.148 ms) : 0, 54148
AppSec [candidate] (54.3 ms) : 0, 54300
Remote Config [baseline] (671.637 µs) : 0, 672
Remote Config [candidate] (679.983 µs) : 0, 680
Telemetry [baseline] (8.194 ms) : 0, 8194
Telemetry [candidate] (10.411 ms) : 0, 10411
section iast
BytebuddyAgent [baseline] (808.66 ms) : 0, 808660
BytebuddyAgent [candidate] (802.604 ms) : 0, 802604
GlobalTracer [baseline] (303.738 ms) : 0, 303738
GlobalTracer [candidate] (302.475 ms) : 0, 302475
AppSec [baseline] (57.035 ms) : 0, 57035
AppSec [candidate] (58.252 ms) : 0, 58252
IAST [baseline] (22.239 ms) : 0, 22239
IAST [candidate] (20.679 ms) : 0, 20679
Remote Config [baseline] (619.41 µs) : 0, 619
Remote Config [candidate] (625.272 µs) : 0, 625
Telemetry [baseline] (7.259 ms) : 0, 7259
Telemetry [candidate] (7.22 ms) : 0, 7220
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (808.261 ms) : 0, 808261
BytebuddyAgent [candidate] (802.358 ms) : 0, 802358
GlobalTracer [baseline] (303.965 ms) : 0, 303965
GlobalTracer [candidate] (302.376 ms) : 0, 302376
AppSec [baseline] (57.885 ms) : 0, 57885
AppSec [candidate] (58.068 ms) : 0, 58068
IAST [baseline] (21.707 ms) : 0, 21707
IAST [candidate] (20.666 ms) : 0, 20666
Remote Config [baseline] (622.677 µs) : 0, 623
Remote Config [candidate] (614.218 µs) : 0, 614
Telemetry [baseline] (7.262 ms) : 0, 7262
Telemetry [candidate] (7.154 ms) : 0, 7154
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (798.075 ms) : 0, 798075
BytebuddyAgent [candidate] (800.183 ms) : 0, 800183
GlobalTracer [baseline] (302.452 ms) : 0, 302452
GlobalTracer [candidate] (303.437 ms) : 0, 303437
AppSec [baseline] (57.584 ms) : 0, 57584
AppSec [candidate] (57.248 ms) : 0, 57248
IAST [baseline] (20.191 ms) : 0, 20191
IAST [candidate] (20.861 ms) : 0, 20861
Remote Config [baseline] (627.657 µs) : 0, 628
Remote Config [candidate] (596.182 µs) : 0, 596
Telemetry [baseline] (7.083 ms) : 0, 7083
Telemetry [candidate] (7.035 ms) : 0, 7035
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 16 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.42.0-SNAPSHOT~35b4f26f84, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section baseline
no_agent (1.363 ms) : 1343, 1383
. : milestone, 1363,
appsec (1.727 ms) : 1704, 1750
. : milestone, 1727,
appsec_no_iast (1.752 ms) : 1729, 1776
. : milestone, 1752,
iast (1.495 ms) : 1472, 1518
. : milestone, 1495,
profiling (1.496 ms) : 1473, 1520
. : milestone, 1496,
tracing (1.472 ms) : 1449, 1496
. : milestone, 1472,
section candidate
no_agent (1.351 ms) : 1332, 1370
. : milestone, 1351,
appsec (1.736 ms) : 1713, 1759
. : milestone, 1736,
appsec_no_iast (1.731 ms) : 1707, 1756
. : milestone, 1731,
iast (1.518 ms) : 1496, 1541
. : milestone, 1518,
profiling (1.485 ms) : 1463, 1508
. : milestone, 1485,
tracing (1.464 ms) : 1440, 1489
. : milestone, 1464,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.42.0-SNAPSHOT~35b4f26f84, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section baseline
no_agent (378.917 µs) : 358, 399
. : milestone, 379,
iast (496.083 µs) : 474, 518
. : milestone, 496,
iast_FULL (562.34 µs) : 541, 584
. : milestone, 562,
iast_GLOBAL (522.19 µs) : 500, 545
. : milestone, 522,
iast_HARDCODED_SECRET_DISABLED (491.013 µs) : 470, 512
. : milestone, 491,
iast_INACTIVE (454.838 µs) : 434, 476
. : milestone, 455,
iast_TELEMETRY_OFF (480.088 µs) : 459, 502
. : milestone, 480,
tracing (458.048 µs) : 437, 479
. : milestone, 458,
section candidate
no_agent (379.005 µs) : 357, 401
. : milestone, 379,
iast (490.545 µs) : 469, 512
. : milestone, 491,
iast_FULL (563.423 µs) : 542, 585
. : milestone, 563,
iast_GLOBAL (508.448 µs) : 487, 530
. : milestone, 508,
iast_HARDCODED_SECRET_DISABLED (499.095 µs) : 478, 521
. : milestone, 499,
iast_INACTIVE (457.74 µs) : 437, 479
. : milestone, 458,
iast_TELEMETRY_OFF (480.805 µs) : 460, 502
. : milestone, 481,
tracing (459.672 µs) : 438, 481
. : milestone, 460,
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.42.0-SNAPSHOT~35b4f26f84, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section baseline
no_agent (1.474 ms) : 1462, 1485
. : milestone, 1474,
appsec (2.323 ms) : 2282, 2363
. : milestone, 2323,
iast (2.092 ms) : 2039, 2145
. : milestone, 2092,
iast_GLOBAL (2.128 ms) : 2074, 2181
. : milestone, 2128,
profiling (1.923 ms) : 1882, 1964
. : milestone, 1923,
tracing (1.918 ms) : 1879, 1958
. : milestone, 1918,
section candidate
no_agent (1.467 ms) : 1456, 1479
. : milestone, 1467,
appsec (2.34 ms) : 2299, 2381
. : milestone, 2340,
iast (2.076 ms) : 2024, 2128
. : milestone, 2076,
iast_GLOBAL (2.125 ms) : 2072, 2179
. : milestone, 2125,
profiling (1.938 ms) : 1897, 1979
. : milestone, 1938,
tracing (1.922 ms) : 1882, 1961
. : milestone, 1922,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.42.0-SNAPSHOT~35b4f26f84, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section baseline
no_agent (15.166 s) : 15166000, 15166000
. : milestone, 15166000,
appsec (15.198 s) : 15198000, 15198000
. : milestone, 15198000,
iast (18.611 s) : 18611000, 18611000
. : milestone, 18611000,
iast_GLOBAL (18.1 s) : 18100000, 18100000
. : milestone, 18100000,
profiling (15.192 s) : 15192000, 15192000
. : milestone, 15192000,
tracing (14.859 s) : 14859000, 14859000
. : milestone, 14859000,
section candidate
no_agent (15.278 s) : 15278000, 15278000
. : milestone, 15278000,
appsec (15.316 s) : 15316000, 15316000
. : milestone, 15316000,
iast (18.847 s) : 18847000, 18847000
. : milestone, 18847000,
iast_GLOBAL (18.124 s) : 18124000, 18124000
. : milestone, 18124000,
profiling (15.179 s) : 15179000, 15179000
. : milestone, 15179000,
tracing (15.313 s) : 15313000, 15313000
. : milestone, 15313000,
|
aboitreaud
approved these changes
Oct 18, 2024
5 tasks
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
Change the default config values for DJM before the first call to
Config.get()
that resolve the Config values.Motivation
Long running spans not being enabled when using
-Ddd.data.jobs.enabled=true
since the config values were already resolved before the default values are changedContributor 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]