-
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
[🍒 7795] Fix default value for long-running spans with DJM #7810
Merged
nikita-tkachenko-datadog
merged 1 commit into
release/v1.41.x
from
paul.laffon/backport-pr-7795
Oct 18, 2024
Merged
[🍒 7795] Fix default value for long-running spans with DJM #7810
nikita-tkachenko-datadog
merged 1 commit into
release/v1.41.x
from
paul.laffon/backport-pr-7795
Oct 18, 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
nikita-tkachenko-datadog
added
type: bug
inst: apache spark
Apache Spark instrumentation
labels
Oct 18, 2024
BenchmarksStartupParameters
See matching parameters
SummaryFound 1 performance improvements and 2 performance regressions! Performance is the same for 50 metrics, 10 unstable metrics.
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.42.0-SNAPSHOT~8f37b2bdd9, baseline=1.42.0-SNAPSHOT~00856e02d3
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.087 s) : 0, 1087404
Total [baseline] (10.476 s) : 0, 10476202
Agent [candidate] (1.072 s) : 0, 1072200
Total [candidate] (10.443 s) : 0, 10442583
section appsec
Agent [baseline] (1.209 s) : 0, 1208733
Total [baseline] (10.628 s) : 0, 10627626
Agent [candidate] (1.209 s) : 0, 1208957
Total [candidate] (10.66 s) : 0, 10660292
section iast
Agent [baseline] (1.2 s) : 0, 1200437
Total [baseline] (10.883 s) : 0, 10883416
Agent [candidate] (1.203 s) : 0, 1202601
Total [candidate] (10.903 s) : 0, 10902738
section profiling
Agent [baseline] (1.28 s) : 0, 1280237
Total [baseline] (10.672 s) : 0, 10672002
Agent [candidate] (1.278 s) : 0, 1278243
Total [candidate] (10.682 s) : 0, 10681600
gantt
title petclinic - break down per module: candidate=1.42.0-SNAPSHOT~8f37b2bdd9, baseline=1.42.0-SNAPSHOT~00856e02d3
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (693.252 ms) : 0, 693252
BytebuddyAgent [candidate] (684.086 ms) : 0, 684086
GlobalTracer [baseline] (316.969 ms) : 0, 316969
GlobalTracer [candidate] (312.477 ms) : 0, 312477
AppSec [baseline] (54.435 ms) : 0, 54435
AppSec [candidate] (53.785 ms) : 0, 53785
Remote Config [baseline] (673.74 µs) : 0, 674
Remote Config [candidate] (658.776 µs) : 0, 659
Telemetry [baseline] (8.291 ms) : 0, 8291
Telemetry [candidate] (7.489 ms) : 0, 7489
section appsec
BytebuddyAgent [baseline] (701.94 ms) : 0, 701940
BytebuddyAgent [candidate] (702.394 ms) : 0, 702394
GlobalTracer [baseline] (309.496 ms) : 0, 309496
GlobalTracer [candidate] (309.966 ms) : 0, 309966
AppSec [baseline] (165.997 ms) : 0, 165997
AppSec [candidate] (163.877 ms) : 0, 163877
IAST [baseline] (19.293 ms) : 0, 19293
IAST [candidate] (19.778 ms) : 0, 19778
Remote Config [baseline] (629.808 µs) : 0, 630
Remote Config [candidate] (633.385 µs) : 0, 633
Telemetry [baseline] (7.403 ms) : 0, 7403
Telemetry [candidate] (8.487 ms) : 0, 8487
section iast
BytebuddyAgent [baseline] (799.908 ms) : 0, 799908
BytebuddyAgent [candidate] (800.432 ms) : 0, 800432
GlobalTracer [baseline] (300.856 ms) : 0, 300856
GlobalTracer [candidate] (302.0 ms) : 0, 302000
AppSec [baseline] (57.848 ms) : 0, 57848
AppSec [candidate] (56.399 ms) : 0, 56399
IAST [baseline] (20.499 ms) : 0, 20499
IAST [candidate] (22.318 ms) : 0, 22318
Remote Config [baseline] (602.739 µs) : 0, 603
Remote Config [candidate] (604.85 µs) : 0, 605
Telemetry [baseline] (7.084 ms) : 0, 7084
Telemetry [candidate] (7.105 ms) : 0, 7105
section profiling
BytebuddyAgent [baseline] (683.88 ms) : 0, 683880
BytebuddyAgent [candidate] (681.431 ms) : 0, 681431
GlobalTracer [baseline] (397.538 ms) : 0, 397538
GlobalTracer [candidate] (397.506 ms) : 0, 397506
AppSec [baseline] (54.864 ms) : 0, 54864
AppSec [candidate] (54.603 ms) : 0, 54603
Remote Config [baseline] (663.209 µs) : 0, 663
Remote Config [candidate] (668.787 µs) : 0, 669
Telemetry [baseline] (14.458 ms) : 0, 14458
Telemetry [candidate] (7.506 ms) : 0, 7506
ProfilingAgent [baseline] (89.879 ms) : 0, 89879
ProfilingAgent [candidate] (97.665 ms) : 0, 97665
Profiling [baseline] (89.902 ms) : 0, 89902
Profiling [candidate] (97.689 ms) : 0, 97689
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.42.0-SNAPSHOT~8f37b2bdd9, baseline=1.42.0-SNAPSHOT~00856e02d3
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.086 s) : 0, 1085563
Total [baseline] (8.618 s) : 0, 8618207
Agent [candidate] (1.085 s) : 0, 1084830
Total [candidate] (8.67 s) : 0, 8670184
section iast
Agent [baseline] (1.2 s) : 0, 1200096
Total [baseline] (9.116 s) : 0, 9116222
Agent [candidate] (1.201 s) : 0, 1201169
Total [candidate] (9.128 s) : 0, 9127502
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.2 s) : 0, 1200033
Total [baseline] (9.079 s) : 0, 9079491
Agent [candidate] (1.201 s) : 0, 1200845
Total [candidate] (9.067 s) : 0, 9066735
section iast_TELEMETRY_OFF
Agent [baseline] (1.213 s) : 0, 1212990
Total [baseline] (9.141 s) : 0, 9140603
Agent [candidate] (1.205 s) : 0, 1205468
Total [candidate] (9.141 s) : 0, 9140870
gantt
title insecure-bank - break down per module: candidate=1.42.0-SNAPSHOT~8f37b2bdd9, baseline=1.42.0-SNAPSHOT~00856e02d3
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (694.059 ms) : 0, 694059
BytebuddyAgent [candidate] (692.493 ms) : 0, 692493
GlobalTracer [baseline] (314.169 ms) : 0, 314169
GlobalTracer [candidate] (315.578 ms) : 0, 315578
AppSec [baseline] (53.932 ms) : 0, 53932
AppSec [candidate] (54.571 ms) : 0, 54571
Remote Config [baseline] (666.593 µs) : 0, 667
Remote Config [candidate] (674.116 µs) : 0, 674
Telemetry [baseline] (8.906 ms) : 0, 8906
Telemetry [candidate] (7.657 ms) : 0, 7657
section iast
BytebuddyAgent [baseline] (800.405 ms) : 0, 800405
BytebuddyAgent [candidate] (799.719 ms) : 0, 799719
GlobalTracer [baseline] (301.02 ms) : 0, 301020
GlobalTracer [candidate] (301.628 ms) : 0, 301628
AppSec [baseline] (57.253 ms) : 0, 57253
AppSec [candidate] (55.639 ms) : 0, 55639
IAST [baseline] (20.114 ms) : 0, 20114
IAST [candidate] (22.822 ms) : 0, 22822
Remote Config [baseline] (604.291 µs) : 0, 604
Remote Config [candidate] (595.742 µs) : 0, 596
Telemetry [baseline] (7.057 ms) : 0, 7057
Telemetry [candidate] (7.028 ms) : 0, 7028
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (799.394 ms) : 0, 799394
BytebuddyAgent [candidate] (799.095 ms) : 0, 799095
GlobalTracer [baseline] (301.458 ms) : 0, 301458
GlobalTracer [candidate] (301.96 ms) : 0, 301960
AppSec [baseline] (54.449 ms) : 0, 54449
AppSec [candidate] (57.304 ms) : 0, 57304
IAST [baseline] (23.525 ms) : 0, 23525
IAST [candidate] (21.12 ms) : 0, 21120
Remote Config [baseline] (596.756 µs) : 0, 597
Remote Config [candidate] (596.463 µs) : 0, 596
Telemetry [baseline] (6.974 ms) : 0, 6974
Telemetry [candidate] (7.068 ms) : 0, 7068
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (808.073 ms) : 0, 808073
BytebuddyAgent [candidate] (801.94 ms) : 0, 801940
GlobalTracer [baseline] (305.115 ms) : 0, 305115
GlobalTracer [candidate] (303.688 ms) : 0, 303688
AppSec [baseline] (56.927 ms) : 0, 56927
AppSec [candidate] (55.696 ms) : 0, 55696
IAST [baseline] (21.323 ms) : 0, 21323
IAST [candidate] (22.723 ms) : 0, 22723
Remote Config [baseline] (626.058 µs) : 0, 626
Remote Config [candidate] (608.534 µs) : 0, 609
Telemetry [baseline] (7.124 ms) : 0, 7124
Telemetry [candidate] (7.0 ms) : 0, 7000
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 17 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.42.0-SNAPSHOT~8f37b2bdd9, baseline=1.42.0-SNAPSHOT~00856e02d3
dateFormat X
axisFormat %s
section baseline
no_agent (376.338 µs) : 357, 396
. : milestone, 376,
iast (503.251 µs) : 481, 525
. : milestone, 503,
iast_FULL (551.927 µs) : 531, 573
. : milestone, 552,
iast_GLOBAL (515.074 µs) : 492, 538
. : milestone, 515,
iast_HARDCODED_SECRET_DISABLED (492.585 µs) : 471, 514
. : milestone, 493,
iast_INACTIVE (450.747 µs) : 430, 472
. : milestone, 451,
iast_TELEMETRY_OFF (479.139 µs) : 458, 500
. : milestone, 479,
tracing (450.524 µs) : 430, 471
. : milestone, 451,
section candidate
no_agent (375.307 µs) : 356, 395
. : milestone, 375,
iast (491.539 µs) : 470, 513
. : milestone, 492,
iast_FULL (566.32 µs) : 545, 588
. : milestone, 566,
iast_GLOBAL (505.909 µs) : 485, 527
. : milestone, 506,
iast_HARDCODED_SECRET_DISABLED (489.3 µs) : 468, 511
. : milestone, 489,
iast_INACTIVE (455.66 µs) : 435, 477
. : milestone, 456,
iast_TELEMETRY_OFF (484.029 µs) : 463, 505
. : milestone, 484,
tracing (455.225 µs) : 434, 477
. : milestone, 455,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.42.0-SNAPSHOT~8f37b2bdd9, baseline=1.42.0-SNAPSHOT~00856e02d3
dateFormat X
axisFormat %s
section baseline
no_agent (1.353 ms) : 1333, 1372
. : milestone, 1353,
appsec (1.721 ms) : 1696, 1746
. : milestone, 1721,
appsec_no_iast (1.732 ms) : 1708, 1756
. : milestone, 1732,
iast (1.473 ms) : 1450, 1496
. : milestone, 1473,
profiling (1.49 ms) : 1467, 1513
. : milestone, 1490,
tracing (1.465 ms) : 1440, 1490
. : milestone, 1465,
section candidate
no_agent (1.363 ms) : 1343, 1382
. : milestone, 1363,
appsec (1.726 ms) : 1702, 1750
. : milestone, 1726,
appsec_no_iast (1.738 ms) : 1713, 1762
. : milestone, 1738,
iast (1.502 ms) : 1479, 1525
. : milestone, 1502,
profiling (1.478 ms) : 1456, 1501
. : milestone, 1478,
tracing (1.465 ms) : 1441, 1490
. : milestone, 1465,
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.42.0-SNAPSHOT~8f37b2bdd9, baseline=1.42.0-SNAPSHOT~00856e02d3
dateFormat X
axisFormat %s
section baseline
no_agent (1.465 ms) : 1454, 1476
. : milestone, 1465,
appsec (2.334 ms) : 2293, 2375
. : milestone, 2334,
iast (2.079 ms) : 2027, 2131
. : milestone, 2079,
iast_GLOBAL (2.108 ms) : 2056, 2160
. : milestone, 2108,
profiling (2.426 ms) : 2238, 2614
. : milestone, 2426,
tracing (1.903 ms) : 1864, 1942
. : milestone, 1903,
section candidate
no_agent (1.468 ms) : 1456, 1479
. : milestone, 1468,
appsec (2.307 ms) : 2266, 2347
. : milestone, 2307,
iast (2.067 ms) : 2015, 2119
. : milestone, 2067,
iast_GLOBAL (2.109 ms) : 2057, 2161
. : milestone, 2109,
profiling (1.946 ms) : 1903, 1988
. : milestone, 1946,
tracing (1.913 ms) : 1873, 1953
. : milestone, 1913,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.42.0-SNAPSHOT~8f37b2bdd9, baseline=1.42.0-SNAPSHOT~00856e02d3
dateFormat X
axisFormat %s
section baseline
no_agent (15.045 s) : 15045000, 15045000
. : milestone, 15045000,
appsec (15.474 s) : 15474000, 15474000
. : milestone, 15474000,
iast (18.524 s) : 18524000, 18524000
. : milestone, 18524000,
iast_GLOBAL (18.15 s) : 18150000, 18150000
. : milestone, 18150000,
profiling (15.373 s) : 15373000, 15373000
. : milestone, 15373000,
tracing (14.971 s) : 14971000, 14971000
. : milestone, 14971000,
section candidate
no_agent (15.096 s) : 15096000, 15096000
. : milestone, 15096000,
appsec (15.372 s) : 15372000, 15372000
. : milestone, 15372000,
iast (19.286 s) : 19286000, 19286000
. : milestone, 19286000,
iast_GLOBAL (18.074 s) : 18074000, 18074000
. : milestone, 18074000,
profiling (15.119 s) : 15119000, 15119000
. : milestone, 15119000,
tracing (15.089 s) : 15089000, 15089000
. : milestone, 15089000,
|
nikita-tkachenko-datadog
approved these changes
Oct 18, 2024
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.
Cherry pick #7795
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]