-
Notifications
You must be signed in to change notification settings - Fork 319
Fix Kafka consumer instrumentation #10100
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. |
Kafka / producer-benchmarkParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 3 metrics, 0 unstable metrics. See unchanged results
|
robcarlan-datadog
left a comment
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 for DSM
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 59 metrics, 6 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.57.0-SNAPSHOT~8c64d26100, baseline=1.57.0-SNAPSHOT~5c5592a6f6
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.079 s) : 0, 1078896
Total [baseline] (8.744 s) : 0, 8743947
Agent [candidate] (1.081 s) : 0, 1081003
Total [candidate] (8.736 s) : 0, 8736460
section iast
Agent [baseline] (1.221 s) : 0, 1220576
Total [baseline] (9.439 s) : 0, 9439115
Agent [candidate] (1.227 s) : 0, 1226687
Total [candidate] (9.456 s) : 0, 9456363
gantt
title insecure-bank - break down per module: candidate=1.57.0-SNAPSHOT~8c64d26100, baseline=1.57.0-SNAPSHOT~5c5592a6f6
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.194 ms) : 0, 1194
crashtracking [candidate] (1.214 ms) : 0, 1214
BytebuddyAgent [baseline] (647.899 ms) : 0, 647899
BytebuddyAgent [candidate] (649.272 ms) : 0, 649272
GlobalTracer [baseline] (281.137 ms) : 0, 281137
GlobalTracer [candidate] (281.887 ms) : 0, 281887
AppSec [baseline] (32.323 ms) : 0, 32323
AppSec [candidate] (32.31 ms) : 0, 32310
Debugger [baseline] (67.425 ms) : 0, 67425
Debugger [candidate] (67.247 ms) : 0, 67247
Remote Config [baseline] (669.224 µs) : 0, 669
Remote Config [candidate] (654.67 µs) : 0, 655
Telemetry [baseline] (9.01 ms) : 0, 9010
Telemetry [candidate] (8.994 ms) : 0, 8994
Flare Poller [baseline] (3.772 ms) : 0, 3772
Flare Poller [candidate] (3.748 ms) : 0, 3748
section iast
crashtracking [baseline] (1.203 ms) : 0, 1203
crashtracking [candidate] (1.198 ms) : 0, 1198
BytebuddyAgent [baseline] (790.199 ms) : 0, 790199
BytebuddyAgent [candidate] (793.812 ms) : 0, 793812
GlobalTracer [baseline] (254.563 ms) : 0, 254563
GlobalTracer [candidate] (255.392 ms) : 0, 255392
AppSec [baseline] (35.45 ms) : 0, 35450
AppSec [candidate] (35.836 ms) : 0, 35836
Debugger [baseline] (64.539 ms) : 0, 64539
Debugger [candidate] (65.186 ms) : 0, 65186
Remote Config [baseline] (579.993 µs) : 0, 580
Remote Config [candidate] (576.627 µs) : 0, 577
Telemetry [baseline] (8.453 ms) : 0, 8453
Telemetry [candidate] (8.443 ms) : 0, 8443
Flare Poller [baseline] (3.441 ms) : 0, 3441
Flare Poller [candidate] (3.492 ms) : 0, 3492
IAST [baseline] (26.87 ms) : 0, 26870
IAST [candidate] (27.098 ms) : 0, 27098
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.57.0-SNAPSHOT~8c64d26100, baseline=1.57.0-SNAPSHOT~5c5592a6f6
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.082 s) : 0, 1082486
Total [baseline] (10.864 s) : 0, 10863892
Agent [candidate] (1.088 s) : 0, 1088012
Total [candidate] (10.86 s) : 0, 10859584
section appsec
Agent [baseline] (1.263 s) : 0, 1263283
Total [baseline] (11.098 s) : 0, 11098289
Agent [candidate] (1.262 s) : 0, 1262421
Total [candidate] (11.067 s) : 0, 11067065
section iast
Agent [baseline] (1.227 s) : 0, 1227039
Total [baseline] (11.221 s) : 0, 11220840
Agent [candidate] (1.222 s) : 0, 1222090
Total [candidate] (11.246 s) : 0, 11245993
section profiling
Agent [baseline] (1.208 s) : 0, 1207537
Total [baseline] (10.953 s) : 0, 10952865
Agent [candidate] (1.203 s) : 0, 1203169
Total [candidate] (11.017 s) : 0, 11017391
gantt
title petclinic - break down per module: candidate=1.57.0-SNAPSHOT~8c64d26100, baseline=1.57.0-SNAPSHOT~5c5592a6f6
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.193 ms) : 0, 1193
crashtracking [candidate] (1.204 ms) : 0, 1204
BytebuddyAgent [baseline] (649.734 ms) : 0, 649734
BytebuddyAgent [candidate] (653.028 ms) : 0, 653028
GlobalTracer [baseline] (282.034 ms) : 0, 282034
GlobalTracer [candidate] (283.621 ms) : 0, 283621
AppSec [baseline] (32.271 ms) : 0, 32271
AppSec [candidate] (32.662 ms) : 0, 32662
Debugger [baseline] (68.119 ms) : 0, 68119
Debugger [candidate] (68.344 ms) : 0, 68344
Remote Config [baseline] (641.755 µs) : 0, 642
Remote Config [candidate] (627.511 µs) : 0, 628
Telemetry [baseline] (9.02 ms) : 0, 9020
Telemetry [candidate] (9.076 ms) : 0, 9076
Flare Poller [baseline] (3.851 ms) : 0, 3851
Flare Poller [candidate] (3.815 ms) : 0, 3815
section appsec
crashtracking [baseline] (1.203 ms) : 0, 1203
crashtracking [candidate] (1.201 ms) : 0, 1201
BytebuddyAgent [baseline] (687.727 ms) : 0, 687727
BytebuddyAgent [candidate] (687.972 ms) : 0, 687972
GlobalTracer [baseline] (258.855 ms) : 0, 258855
GlobalTracer [candidate] (258.917 ms) : 0, 258917
AppSec [baseline] (175.05 ms) : 0, 175050
AppSec [candidate] (174.391 ms) : 0, 174391
Debugger [baseline] (66.474 ms) : 0, 66474
Debugger [candidate] (66.439 ms) : 0, 66439
Remote Config [baseline] (717.496 µs) : 0, 717
Remote Config [candidate] (711.071 µs) : 0, 711
Telemetry [baseline] (9.122 ms) : 0, 9122
Telemetry [candidate] (9.122 ms) : 0, 9122
Flare Poller [baseline] (3.956 ms) : 0, 3956
Flare Poller [candidate] (3.953 ms) : 0, 3953
IAST [baseline] (24.693 ms) : 0, 24693
IAST [candidate] (24.404 ms) : 0, 24404
section iast
crashtracking [baseline] (1.2 ms) : 0, 1200
crashtracking [candidate] (1.193 ms) : 0, 1193
BytebuddyAgent [baseline] (792.962 ms) : 0, 792962
BytebuddyAgent [candidate] (790.313 ms) : 0, 790313
GlobalTracer [baseline] (255.786 ms) : 0, 255786
GlobalTracer [candidate] (255.312 ms) : 0, 255312
AppSec [baseline] (34.903 ms) : 0, 34903
AppSec [candidate] (35.137 ms) : 0, 35137
Debugger [baseline] (67.131 ms) : 0, 67131
Debugger [candidate] (65.497 ms) : 0, 65497
Remote Config [baseline] (604.995 µs) : 0, 605
Remote Config [candidate] (559.168 µs) : 0, 559
Telemetry [baseline] (8.586 ms) : 0, 8586
Telemetry [candidate] (8.461 ms) : 0, 8461
Flare Poller [baseline] (3.521 ms) : 0, 3521
Flare Poller [candidate] (3.444 ms) : 0, 3444
IAST [baseline] (26.925 ms) : 0, 26925
IAST [candidate] (26.847 ms) : 0, 26847
section profiling
crashtracking [baseline] (1.206 ms) : 0, 1206
crashtracking [candidate] (1.185 ms) : 0, 1185
BytebuddyAgent [baseline] (703.33 ms) : 0, 703330
BytebuddyAgent [candidate] (701.612 ms) : 0, 701612
GlobalTracer [baseline] (221.413 ms) : 0, 221413
GlobalTracer [candidate] (220.441 ms) : 0, 220441
AppSec [baseline] (32.371 ms) : 0, 32371
AppSec [candidate] (32.112 ms) : 0, 32112
Debugger [baseline] (68.396 ms) : 0, 68396
Debugger [candidate] (67.804 ms) : 0, 67804
Remote Config [baseline] (667.524 µs) : 0, 668
Remote Config [candidate] (605.474 µs) : 0, 605
Telemetry [baseline] (9.041 ms) : 0, 9041
Telemetry [candidate] (8.998 ms) : 0, 8998
Flare Poller [baseline] (3.725 ms) : 0, 3725
Flare Poller [candidate] (3.782 ms) : 0, 3782
ProfilingAgent [baseline] (97.703 ms) : 0, 97703
ProfilingAgent [candidate] (97.152 ms) : 0, 97152
Profiling [baseline] (98.288 ms) : 0, 98288
Profiling [candidate] (97.735 ms) : 0, 97735
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 3 performance regressions! Performance is the same for 18 metrics, 15 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.57.0-SNAPSHOT~8c64d26100, baseline=1.57.0-SNAPSHOT~5c5592a6f6
dateFormat X
axisFormat %s
section baseline
no_agent (1.188 ms) : 1177, 1199
. : milestone, 1188,
iast (3.249 ms) : 3206, 3291
. : milestone, 3249,
iast_FULL (5.879 ms) : 5819, 5939
. : milestone, 5879,
iast_GLOBAL (3.656 ms) : 3599, 3713
. : milestone, 3656,
profiling (2.091 ms) : 2073, 2110
. : milestone, 2091,
tracing (1.846 ms) : 1831, 1861
. : milestone, 1846,
section candidate
no_agent (1.186 ms) : 1176, 1197
. : milestone, 1186,
iast (3.201 ms) : 3157, 3244
. : milestone, 3201,
iast_FULL (5.892 ms) : 5834, 5951
. : milestone, 5892,
iast_GLOBAL (3.667 ms) : 3614, 3721
. : milestone, 3667,
profiling (1.97 ms) : 1953, 1988
. : milestone, 1970,
tracing (1.787 ms) : 1773, 1801
. : milestone, 1787,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.57.0-SNAPSHOT~8c64d26100, baseline=1.57.0-SNAPSHOT~5c5592a6f6
dateFormat X
axisFormat %s
section baseline
no_agent (18.026 ms) : 17843, 18209
. : milestone, 18026,
appsec (18.211 ms) : 18029, 18394
. : milestone, 18211,
code_origins (17.661 ms) : 17485, 17837
. : milestone, 17661,
iast (17.762 ms) : 17585, 17939
. : milestone, 17762,
profiling (18.398 ms) : 18214, 18582
. : milestone, 18398,
tracing (17.917 ms) : 17740, 18094
. : milestone, 17917,
section candidate
no_agent (19.212 ms) : 19014, 19409
. : milestone, 19212,
appsec (18.35 ms) : 18163, 18536
. : milestone, 18350,
code_origins (17.919 ms) : 17739, 18100
. : milestone, 17919,
iast (17.607 ms) : 17431, 17783
. : milestone, 17607,
profiling (18.658 ms) : 18470, 18846
. : milestone, 18658,
tracing (18.754 ms) : 18568, 18939
. : milestone, 18754,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 10 metrics, 2 unstable metrics. Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.57.0-SNAPSHOT~8c64d26100, baseline=1.57.0-SNAPSHOT~5c5592a6f6
dateFormat X
axisFormat %s
section baseline
no_agent (14.84 s) : 14840000, 14840000
. : milestone, 14840000,
appsec (14.617 s) : 14617000, 14617000
. : milestone, 14617000,
iast (18.575 s) : 18575000, 18575000
. : milestone, 18575000,
iast_GLOBAL (17.957 s) : 17957000, 17957000
. : milestone, 17957000,
profiling (14.936 s) : 14936000, 14936000
. : milestone, 14936000,
tracing (14.849 s) : 14849000, 14849000
. : milestone, 14849000,
section candidate
no_agent (14.932 s) : 14932000, 14932000
. : milestone, 14932000,
appsec (14.617 s) : 14617000, 14617000
. : milestone, 14617000,
iast (18.283 s) : 18283000, 18283000
. : milestone, 18283000,
iast_GLOBAL (18.004 s) : 18004000, 18004000
. : milestone, 18004000,
profiling (14.829 s) : 14829000, 14829000
. : milestone, 14829000,
tracing (14.86 s) : 14860000, 14860000
. : milestone, 14860000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.57.0-SNAPSHOT~8c64d26100, baseline=1.57.0-SNAPSHOT~5c5592a6f6
dateFormat X
axisFormat %s
section baseline
no_agent (1.478 ms) : 1466, 1489
. : milestone, 1478,
appsec (3.715 ms) : 3495, 3934
. : milestone, 3715,
iast (2.212 ms) : 2147, 2276
. : milestone, 2212,
iast_GLOBAL (2.256 ms) : 2191, 2321
. : milestone, 2256,
profiling (2.474 ms) : 2312, 2637
. : milestone, 2474,
tracing (2.05 ms) : 1999, 2101
. : milestone, 2050,
section candidate
no_agent (1.473 ms) : 1461, 1484
. : milestone, 1473,
appsec (3.729 ms) : 3508, 3950
. : milestone, 3729,
iast (2.218 ms) : 2153, 2283
. : milestone, 2218,
iast_GLOBAL (2.259 ms) : 2194, 2325
. : milestone, 2259,
profiling (2.094 ms) : 2040, 2149
. : milestone, 2094,
tracing (2.049 ms) : 1997, 2100
. : milestone, 2049,
|
Kafka / consumer-benchmarkParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 3 metrics, 0 unstable metrics. See unchanged results
|
What Does This Do
https://github.com/DataDog/dd-trace-java/pull/9974/files introduced an issue:
This is due to a missing helper class in
KafkaConsumerInstrumentation.javaMotivation
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any useful labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]