-
Notifications
You must be signed in to change notification settings - Fork 312
Use constant instead of magic values for IsTraceRoot #9465
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. |
dd-trace-core/src/main/java/datadog/trace/common/metrics/SerializingMetricWriter.java
Outdated
Show resolved
Hide resolved
dd-trace-core/src/main/java/datadog/trace/common/metrics/SerializingMetricWriter.java
Outdated
Show resolved
Hide resolved
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.
thanks for the refactoring
🎯 Code Coverage 🔗 Commit SHA: 3d28255 | Docs | Was this helpful? Give us feedback! |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 45 metrics, 14 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.54.0-SNAPSHOT~3d282555c6, baseline=1.54.0-SNAPSHOT~03d997e2fd
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.058 s) : 0, 1058455
Total [baseline] (10.694 s) : 0, 10693915
Agent [candidate] (1.057 s) : 0, 1057255
Total [candidate] (10.806 s) : 0, 10805584
section appsec
Agent [baseline] (1.233 s) : 0, 1233175
Total [baseline] (10.941 s) : 0, 10940575
Agent [candidate] (1.243 s) : 0, 1243399
Total [candidate] (10.921 s) : 0, 10921452
section iast
Agent [baseline] (1.179 s) : 0, 1179218
Total [baseline] (10.968 s) : 0, 10967852
Agent [candidate] (1.192 s) : 0, 1191936
Total [candidate] (10.935 s) : 0, 10935216
section profiling
Agent [baseline] (1.201 s) : 0, 1200645
Total [baseline] (10.965 s) : 0, 10964582
Agent [candidate] (1.2 s) : 0, 1200116
Total [candidate] (10.914 s) : 0, 10913578
gantt
title petclinic - break down per module: candidate=1.54.0-SNAPSHOT~3d282555c6, baseline=1.54.0-SNAPSHOT~03d997e2fd
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.475 ms) : 0, 1475
crashtracking [candidate] (1.458 ms) : 0, 1458
BytebuddyAgent [baseline] (741.526 ms) : 0, 741526
BytebuddyAgent [candidate] (738.871 ms) : 0, 738871
GlobalTracer [baseline] (245.081 ms) : 0, 245081
GlobalTracer [candidate] (245.22 ms) : 0, 245220
AppSec [baseline] (30.647 ms) : 0, 30647
AppSec [candidate] (30.538 ms) : 0, 30538
Debugger [baseline] (6.124 ms) : 0, 6124
Debugger [candidate] (6.135 ms) : 0, 6135
Remote Config [baseline] (717.086 µs) : 0, 717
Remote Config [candidate] (689.572 µs) : 0, 690
Telemetry [baseline] (11.542 ms) : 0, 11542
Telemetry [candidate] (13.073 ms) : 0, 13073
section appsec
crashtracking [baseline] (1.459 ms) : 0, 1459
crashtracking [candidate] (1.49 ms) : 0, 1490
BytebuddyAgent [baseline] (761.14 ms) : 0, 761140
BytebuddyAgent [candidate] (768.827 ms) : 0, 768827
GlobalTracer [baseline] (237.48 ms) : 0, 237480
GlobalTracer [candidate] (239.339 ms) : 0, 239339
IAST [baseline] (23.785 ms) : 0, 23785
IAST [candidate] (24.223 ms) : 0, 24223
AppSec [baseline] (169.827 ms) : 0, 169827
AppSec [candidate] (169.531 ms) : 0, 169531
Debugger [baseline] (7.401 ms) : 0, 7401
Debugger [candidate] (8.783 ms) : 0, 8783
Remote Config [baseline] (628.39 µs) : 0, 628
Remote Config [candidate] (633.832 µs) : 0, 634
Telemetry [baseline] (10.237 ms) : 0, 10237
Telemetry [candidate] (9.289 ms) : 0, 9289
section iast
crashtracking [baseline] (1.45 ms) : 0, 1450
crashtracking [candidate] (1.477 ms) : 0, 1477
BytebuddyAgent [baseline] (850.872 ms) : 0, 850872
BytebuddyAgent [candidate] (862.752 ms) : 0, 862752
GlobalTracer [baseline] (233.178 ms) : 0, 233178
GlobalTracer [candidate] (234.486 ms) : 0, 234486
IAST [baseline] (29.414 ms) : 0, 29414
IAST [candidate] (29.513 ms) : 0, 29513
AppSec [baseline] (27.684 ms) : 0, 27684
AppSec [candidate] (26.087 ms) : 0, 26087
Debugger [baseline] (6.672 ms) : 0, 6672
Debugger [candidate] (7.585 ms) : 0, 7585
Remote Config [baseline] (599.573 µs) : 0, 600
Remote Config [candidate] (597.448 µs) : 0, 597
Telemetry [baseline] (8.164 ms) : 0, 8164
Telemetry [candidate] (8.218 ms) : 0, 8218
section profiling
crashtracking [baseline] (1.437 ms) : 0, 1437
crashtracking [candidate] (1.429 ms) : 0, 1429
BytebuddyAgent [baseline] (762.723 ms) : 0, 762723
BytebuddyAgent [candidate] (762.548 ms) : 0, 762548
GlobalTracer [baseline] (223.441 ms) : 0, 223441
GlobalTracer [candidate] (223.133 ms) : 0, 223133
AppSec [baseline] (30.728 ms) : 0, 30728
AppSec [candidate] (30.56 ms) : 0, 30560
Debugger [baseline] (7.01 ms) : 0, 7010
Debugger [candidate] (7.075 ms) : 0, 7075
Remote Config [baseline] (736.509 µs) : 0, 737
Remote Config [candidate] (704.894 µs) : 0, 705
Telemetry [baseline] (15.595 ms) : 0, 15595
Telemetry [candidate] (15.636 ms) : 0, 15636
ProfilingAgent [baseline] (108.356 ms) : 0, 108356
ProfilingAgent [candidate] (108.393 ms) : 0, 108393
Profiling [baseline] (109.013 ms) : 0, 109013
Profiling [candidate] (109.087 ms) : 0, 109087
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.54.0-SNAPSHOT~3d282555c6, baseline=1.54.0-SNAPSHOT~03d997e2fd
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.057 s) : 0, 1057143
Total [baseline] (8.681 s) : 0, 8680656
Agent [candidate] (1.051 s) : 0, 1051069
Total [candidate] (8.648 s) : 0, 8647877
section iast
Agent [baseline] (1.186 s) : 0, 1185741
Total [baseline] (9.351 s) : 0, 9350858
Agent [candidate] (1.179 s) : 0, 1178987
Total [candidate] (9.319 s) : 0, 9319455
gantt
title insecure-bank - break down per module: candidate=1.54.0-SNAPSHOT~3d282555c6, baseline=1.54.0-SNAPSHOT~03d997e2fd
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.466 ms) : 0, 1466
crashtracking [candidate] (1.47 ms) : 0, 1470
BytebuddyAgent [baseline] (738.547 ms) : 0, 738547
BytebuddyAgent [candidate] (735.064 ms) : 0, 735064
GlobalTracer [baseline] (246.411 ms) : 0, 246411
GlobalTracer [candidate] (243.154 ms) : 0, 243154
AppSec [baseline] (30.872 ms) : 0, 30872
AppSec [candidate] (30.359 ms) : 0, 30359
Debugger [baseline] (6.174 ms) : 0, 6174
Debugger [candidate] (6.123 ms) : 0, 6123
Remote Config [baseline] (710.325 µs) : 0, 710
Remote Config [candidate] (687.515 µs) : 0, 688
Telemetry [baseline] (11.785 ms) : 0, 11785
Telemetry [candidate] (13.045 ms) : 0, 13045
section iast
crashtracking [baseline] (1.475 ms) : 0, 1475
crashtracking [candidate] (1.452 ms) : 0, 1452
BytebuddyAgent [baseline] (856.759 ms) : 0, 856759
BytebuddyAgent [candidate] (851.069 ms) : 0, 851069
GlobalTracer [baseline] (233.544 ms) : 0, 233544
GlobalTracer [candidate] (234.025 ms) : 0, 234025
IAST [baseline] (31.444 ms) : 0, 31444
IAST [candidate] (30.008 ms) : 0, 30008
AppSec [baseline] (26.796 ms) : 0, 26796
AppSec [candidate] (25.235 ms) : 0, 25235
Debugger [baseline] (5.78 ms) : 0, 5780
Debugger [candidate] (6.636 ms) : 0, 6636
Remote Config [baseline] (616.235 µs) : 0, 616
Remote Config [candidate] (583.895 µs) : 0, 584
Telemetry [baseline] (8.241 ms) : 0, 8241
Telemetry [candidate] (9.01 ms) : 0, 9010
LoadParameters
See matching parameters
SummaryFound 1 performance improvements and 1 performance regressions! Performance is the same for 10 metrics, 12 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~3d282555c6, baseline=1.54.0-SNAPSHOT~03d997e2fd
dateFormat X
axisFormat %s
section baseline
no_agent (36.058 ms) : 35777, 36338
. : milestone, 36058,
appsec (48.199 ms) : 47774, 48624
. : milestone, 48199,
code_origins (46.067 ms) : 45663, 46471
. : milestone, 46067,
iast (45.049 ms) : 44645, 45453
. : milestone, 45049,
profiling (47.85 ms) : 47393, 48307
. : milestone, 47850,
tracing (43.579 ms) : 43203, 43955
. : milestone, 43579,
section candidate
no_agent (36.817 ms) : 36512, 37122
. : milestone, 36817,
appsec (46.806 ms) : 46383, 47230
. : milestone, 46806,
code_origins (45.761 ms) : 45367, 46154
. : milestone, 45761,
iast (45.357 ms) : 44961, 45754
. : milestone, 45357,
profiling (47.803 ms) : 47350, 48257
. : milestone, 47803,
tracing (44.896 ms) : 44518, 45274
. : milestone, 44896,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~3d282555c6, baseline=1.54.0-SNAPSHOT~03d997e2fd
dateFormat X
axisFormat %s
section baseline
no_agent (4.244 ms) : 4196, 4291
. : milestone, 4244,
iast (9.559 ms) : 9401, 9716
. : milestone, 9559,
iast_FULL (13.866 ms) : 13592, 14140
. : milestone, 13866,
iast_GLOBAL (10.577 ms) : 10376, 10779
. : milestone, 10577,
profiling (9.008 ms) : 8867, 9148
. : milestone, 9008,
tracing (7.835 ms) : 7722, 7947
. : milestone, 7835,
section candidate
no_agent (4.358 ms) : 4305, 4411
. : milestone, 4358,
iast (9.241 ms) : 9089, 9394
. : milestone, 9241,
iast_FULL (14.143 ms) : 13856, 14430
. : milestone, 14143,
iast_GLOBAL (10.219 ms) : 10026, 10412
. : milestone, 10219,
profiling (8.643 ms) : 8493, 8793
. : milestone, 8643,
tracing (7.958 ms) : 7844, 8072
. : milestone, 7958,
DacapoParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 0 unstable metrics.
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~3d282555c6, baseline=1.54.0-SNAPSHOT~03d997e2fd
dateFormat X
axisFormat %s
section baseline
no_agent (15.566 s) : 15566000, 15566000
. : milestone, 15566000,
appsec (15.016 s) : 15016000, 15016000
. : milestone, 15016000,
iast (18.541 s) : 18541000, 18541000
. : milestone, 18541000,
iast_GLOBAL (17.811 s) : 17811000, 17811000
. : milestone, 17811000,
profiling (15.544 s) : 15544000, 15544000
. : milestone, 15544000,
tracing (15.098 s) : 15098000, 15098000
. : milestone, 15098000,
section candidate
no_agent (15.541 s) : 15541000, 15541000
. : milestone, 15541000,
appsec (14.527 s) : 14527000, 14527000
. : milestone, 14527000,
iast (18.455 s) : 18455000, 18455000
. : milestone, 18455000,
iast_GLOBAL (17.835 s) : 17835000, 17835000
. : milestone, 17835000,
profiling (15.291 s) : 15291000, 15291000
. : milestone, 15291000,
tracing (14.794 s) : 14794000, 14794000
. : milestone, 14794000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~3d282555c6, baseline=1.54.0-SNAPSHOT~03d997e2fd
dateFormat X
axisFormat %s
section baseline
no_agent (1.482 ms) : 1470, 1493
. : milestone, 1482,
appsec (3.686 ms) : 3469, 3903
. : milestone, 3686,
iast (2.21 ms) : 2148, 2273
. : milestone, 2210,
iast_GLOBAL (2.258 ms) : 2195, 2321
. : milestone, 2258,
profiling (2.081 ms) : 2029, 2133
. : milestone, 2081,
tracing (2.046 ms) : 1996, 2095
. : milestone, 2046,
section candidate
no_agent (1.482 ms) : 1471, 1494
. : milestone, 1482,
appsec (2.485 ms) : 2432, 2538
. : milestone, 2485,
iast (2.215 ms) : 2153, 2278
. : milestone, 2215,
iast_GLOBAL (2.261 ms) : 2198, 2324
. : milestone, 2261,
profiling (2.048 ms) : 1997, 2098
. : milestone, 2048,
tracing (2.034 ms) : 1985, 2083
. : milestone, 2034,
|
Signed-off-by: Brice Dutheil <[email protected]>
1f44294
to
59496fc
Compare
dd-trace-core/src/main/java/datadog/trace/common/metrics/SerializingMetricWriter.java
Outdated
Show resolved
Hide resolved
dd-trace-core/src/test/groovy/datadog/trace/common/metrics/SerializingMetricWriterTest.groovy
Outdated
Show resolved
Hide resolved
552a807
to
54ca4a7
Compare
dd-trace-core/src/test/groovy/datadog/trace/common/metrics/SerializingMetricWriterTest.groovy
Outdated
Show resolved
Hide resolved
54ca4a7
to
e3c92dd
Compare
Signed-off-by: Brice Dutheil <[email protected]>
e3c92dd
to
3d28255
Compare
What Does This Do
Avoid magic values for IsTraceRoot
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: [PROJ-IDENT]