-
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
Add code origin support to kafka message listeners #8301
Conversation
Debugger benchmarksParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 6 metrics, 9 unstable metrics. See unchanged results
Request duration reports for reportsgantt
title reports - request duration [CI 0.99] : candidate=None, baseline=None
dateFormat X
axisFormat %s
section baseline
noprobe (324.976 µs) : 257, 393
. : milestone, 325,
basic (311.815 µs) : 292, 331
. : milestone, 312,
loop (10.142 ms) : 10096, 10188
. : milestone, 10142,
section candidate
noprobe (319.01 µs) : 262, 376
. : milestone, 319,
basic (312.531 µs) : 300, 325
. : milestone, 313,
loop (10.143 ms) : 10101, 10185
. : milestone, 10143,
|
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
|
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 59 metrics, 4 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.47.0-SNAPSHOT~979e40ccc9, baseline=1.47.0-SNAPSHOT~f34e5f4b33
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.047 s) : 0, 1046823
Total [baseline] (8.673 s) : 0, 8673404
Agent [candidate] (1.048 s) : 0, 1048142
Total [candidate] (8.634 s) : 0, 8634128
section iast
Agent [baseline] (1.173 s) : 0, 1173470
Total [baseline] (9.267 s) : 0, 9267376
Agent [candidate] (1.176 s) : 0, 1176449
Total [candidate] (9.215 s) : 0, 9215065
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.175 s) : 0, 1174706
Total [baseline] (9.22 s) : 0, 9219951
Agent [candidate] (1.176 s) : 0, 1176323
Total [candidate] (9.234 s) : 0, 9234268
section iast_TELEMETRY_OFF
Agent [baseline] (1.178 s) : 0, 1178064
Total [baseline] (9.232 s) : 0, 9232310
Agent [candidate] (1.179 s) : 0, 1178713
Total [candidate] (9.239 s) : 0, 9238979
gantt
title insecure-bank - break down per module: candidate=1.47.0-SNAPSHOT~979e40ccc9, baseline=1.47.0-SNAPSHOT~f34e5f4b33
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (718.374 ms) : 0, 718374
BytebuddyAgent [candidate] (720.437 ms) : 0, 720437
GlobalTracer [baseline] (244.224 ms) : 0, 244224
GlobalTracer [candidate] (243.619 ms) : 0, 243619
AppSec [baseline] (55.471 ms) : 0, 55471
AppSec [candidate] (55.175 ms) : 0, 55175
Remote Config [baseline] (715.815 µs) : 0, 716
Remote Config [candidate] (720.721 µs) : 0, 721
Telemetry [baseline] (12.868 ms) : 0, 12868
Telemetry [candidate] (12.886 ms) : 0, 12886
section iast
BytebuddyAgent [baseline] (834.891 ms) : 0, 834891
BytebuddyAgent [candidate] (836.742 ms) : 0, 836742
GlobalTracer [baseline] (234.17 ms) : 0, 234170
GlobalTracer [candidate] (234.964 ms) : 0, 234964
IAST [baseline] (22.846 ms) : 0, 22846
IAST [candidate] (22.809 ms) : 0, 22809
AppSec [baseline] (56.87 ms) : 0, 56870
AppSec [candidate] (57.371 ms) : 0, 57371
Remote Config [baseline] (626.31 µs) : 0, 626
Remote Config [candidate] (629.039 µs) : 0, 629
Telemetry [baseline] (8.829 ms) : 0, 8829
Telemetry [candidate] (8.68 ms) : 0, 8680
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (835.845 ms) : 0, 835845
BytebuddyAgent [candidate] (836.918 ms) : 0, 836918
GlobalTracer [baseline] (233.889 ms) : 0, 233889
GlobalTracer [candidate] (234.712 ms) : 0, 234712
IAST [baseline] (23.154 ms) : 0, 23154
IAST [candidate] (22.911 ms) : 0, 22911
AppSec [baseline] (57.287 ms) : 0, 57287
AppSec [candidate] (57.059 ms) : 0, 57059
Remote Config [baseline] (624.799 µs) : 0, 625
Remote Config [candidate] (620.15 µs) : 0, 620
Telemetry [baseline] (8.727 ms) : 0, 8727
Telemetry [candidate] (8.74 ms) : 0, 8740
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (838.743 ms) : 0, 838743
BytebuddyAgent [candidate] (840.04 ms) : 0, 840040
GlobalTracer [baseline] (235.266 ms) : 0, 235266
GlobalTracer [candidate] (234.498 ms) : 0, 234498
IAST [baseline] (26.422 ms) : 0, 26422
IAST [candidate] (25.513 ms) : 0, 25513
AppSec [baseline] (53.089 ms) : 0, 53089
AppSec [candidate] (54.053 ms) : 0, 54053
Remote Config [baseline] (617.025 µs) : 0, 617
Remote Config [candidate] (610.465 µs) : 0, 610
Telemetry [baseline] (8.704 ms) : 0, 8704
Telemetry [candidate] (8.653 ms) : 0, 8653
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.47.0-SNAPSHOT~979e40ccc9, baseline=1.47.0-SNAPSHOT~f34e5f4b33
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.047 s) : 0, 1047135
Total [baseline] (10.453 s) : 0, 10452812
Agent [candidate] (1.047 s) : 0, 1047295
Total [candidate] (10.418 s) : 0, 10418334
section appsec
Agent [baseline] (1.189 s) : 0, 1188761
Total [baseline] (10.751 s) : 0, 10751010
Agent [candidate] (1.198 s) : 0, 1197812
Total [candidate] (10.806 s) : 0, 10805922
section iast
Agent [baseline] (1.185 s) : 0, 1184611
Total [baseline] (10.965 s) : 0, 10964833
Agent [candidate] (1.174 s) : 0, 1173886
Total [candidate] (10.926 s) : 0, 10925779
section profiling
Agent [baseline] (1.279 s) : 0, 1279164
Total [baseline] (10.91 s) : 0, 10909930
Agent [candidate] (1.266 s) : 0, 1265526
Total [candidate] (10.888 s) : 0, 10887520
gantt
title petclinic - break down per module: candidate=1.47.0-SNAPSHOT~979e40ccc9, baseline=1.47.0-SNAPSHOT~f34e5f4b33
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (717.417 ms) : 0, 717417
BytebuddyAgent [candidate] (718.809 ms) : 0, 718809
GlobalTracer [baseline] (243.928 ms) : 0, 243928
GlobalTracer [candidate] (243.783 ms) : 0, 243783
AppSec [baseline] (55.292 ms) : 0, 55292
AppSec [candidate] (55.02 ms) : 0, 55020
Remote Config [baseline] (721.527 µs) : 0, 722
Remote Config [candidate] (711.224 µs) : 0, 711
Telemetry [baseline] (14.476 ms) : 0, 14476
Telemetry [candidate] (13.703 ms) : 0, 13703
section appsec
BytebuddyAgent [baseline] (736.573 ms) : 0, 736573
BytebuddyAgent [candidate] (741.434 ms) : 0, 741434
GlobalTracer [baseline] (241.123 ms) : 0, 241123
GlobalTracer [candidate] (243.147 ms) : 0, 243147
AppSec [baseline] (175.845 ms) : 0, 175845
AppSec [candidate] (177.69 ms) : 0, 177690
Remote Config [baseline] (660.472 µs) : 0, 660
Remote Config [candidate] (672.35 µs) : 0, 672
Telemetry [baseline] (8.315 ms) : 0, 8315
Telemetry [candidate] (8.354 ms) : 0, 8354
IAST [baseline] (21.873 ms) : 0, 21873
IAST [candidate] (22.364 ms) : 0, 22364
section iast
BytebuddyAgent [baseline] (843.821 ms) : 0, 843821
BytebuddyAgent [candidate] (835.479 ms) : 0, 835479
GlobalTracer [baseline] (235.658 ms) : 0, 235658
GlobalTracer [candidate] (233.819 ms) : 0, 233819
AppSec [baseline] (57.283 ms) : 0, 57283
AppSec [candidate] (57.061 ms) : 0, 57061
Remote Config [baseline] (636.158 µs) : 0, 636
Remote Config [candidate] (617.655 µs) : 0, 618
Telemetry [baseline] (8.795 ms) : 0, 8795
Telemetry [candidate] (8.747 ms) : 0, 8747
IAST [baseline] (23.128 ms) : 0, 23128
IAST [candidate] (22.971 ms) : 0, 22971
section profiling
ProfilingAgent [baseline] (97.444 ms) : 0, 97444
ProfilingAgent [candidate] (95.718 ms) : 0, 95718
BytebuddyAgent [baseline] (715.917 ms) : 0, 715917
BytebuddyAgent [candidate] (708.699 ms) : 0, 708699
GlobalTracer [baseline] (357.136 ms) : 0, 357136
GlobalTracer [candidate] (354.667 ms) : 0, 354667
AppSec [baseline] (56.006 ms) : 0, 56006
AppSec [candidate] (54.551 ms) : 0, 54551
Remote Config [baseline] (717.62 µs) : 0, 718
Remote Config [candidate] (716.867 µs) : 0, 717
Telemetry [baseline] (9.087 ms) : 0, 9087
Telemetry [candidate] (8.893 ms) : 0, 8893
Profiling [baseline] (97.473 ms) : 0, 97473
Profiling [candidate] (95.745 ms) : 0, 95745
LoadParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 16 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.47.0-SNAPSHOT~979e40ccc9, baseline=1.47.0-SNAPSHOT~f34e5f4b33
dateFormat X
axisFormat %s
section baseline
no_agent (1.354 ms) : 1335, 1374
. : milestone, 1354,
appsec (1.744 ms) : 1721, 1768
. : milestone, 1744,
appsec_no_iast (1.751 ms) : 1726, 1776
. : milestone, 1751,
iast (1.52 ms) : 1495, 1544
. : milestone, 1520,
profiling (1.564 ms) : 1539, 1589
. : milestone, 1564,
tracing (1.503 ms) : 1479, 1527
. : milestone, 1503,
section candidate
no_agent (1.362 ms) : 1343, 1382
. : milestone, 1362,
appsec (1.767 ms) : 1744, 1791
. : milestone, 1767,
appsec_no_iast (1.748 ms) : 1723, 1773
. : milestone, 1748,
iast (1.518 ms) : 1494, 1543
. : milestone, 1518,
profiling (1.504 ms) : 1481, 1526
. : milestone, 1504,
tracing (1.501 ms) : 1476, 1525
. : milestone, 1501,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.47.0-SNAPSHOT~979e40ccc9, baseline=1.47.0-SNAPSHOT~f34e5f4b33
dateFormat X
axisFormat %s
section baseline
no_agent (384.117 µs) : 364, 405
. : milestone, 384,
iast (505.536 µs) : 484, 527
. : milestone, 506,
iast_FULL (743.854 µs) : 722, 766
. : milestone, 744,
iast_GLOBAL (557.835 µs) : 535, 581
. : milestone, 558,
iast_HARDCODED_SECRET_DISABLED (511.548 µs) : 490, 533
. : milestone, 512,
iast_INACTIVE (461.88 µs) : 440, 483
. : milestone, 462,
iast_TELEMETRY_OFF (494.21 µs) : 472, 517
. : milestone, 494,
tracing (450.983 µs) : 430, 472
. : milestone, 451,
section candidate
no_agent (376.937 µs) : 357, 397
. : milestone, 377,
iast (516.788 µs) : 494, 539
. : milestone, 517,
iast_FULL (742.059 µs) : 720, 764
. : milestone, 742,
iast_GLOBAL (555.585 µs) : 534, 577
. : milestone, 556,
iast_HARDCODED_SECRET_DISABLED (508.282 µs) : 487, 530
. : milestone, 508,
iast_INACTIVE (465.748 µs) : 444, 487
. : milestone, 466,
iast_TELEMETRY_OFF (500.126 µs) : 477, 523
. : milestone, 500,
tracing (455.49 µs) : 435, 476
. : milestone, 455,
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 biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.47.0-SNAPSHOT~979e40ccc9, baseline=1.47.0-SNAPSHOT~f34e5f4b33
dateFormat X
axisFormat %s
section baseline
no_agent (15.651 s) : 15651000, 15651000
. : milestone, 15651000,
appsec (14.958 s) : 14958000, 14958000
. : milestone, 14958000,
iast (18.983 s) : 18983000, 18983000
. : milestone, 18983000,
iast_GLOBAL (17.89 s) : 17890000, 17890000
. : milestone, 17890000,
profiling (15.083 s) : 15083000, 15083000
. : milestone, 15083000,
tracing (14.878 s) : 14878000, 14878000
. : milestone, 14878000,
section candidate
no_agent (14.86 s) : 14860000, 14860000
. : milestone, 14860000,
appsec (14.988 s) : 14988000, 14988000
. : milestone, 14988000,
iast (18.748 s) : 18748000, 18748000
. : milestone, 18748000,
iast_GLOBAL (18.232 s) : 18232000, 18232000
. : milestone, 18232000,
profiling (14.988 s) : 14988000, 14988000
. : milestone, 14988000,
tracing (14.889 s) : 14889000, 14889000
. : milestone, 14889000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.47.0-SNAPSHOT~979e40ccc9, baseline=1.47.0-SNAPSHOT~f34e5f4b33
dateFormat X
axisFormat %s
section baseline
no_agent (1.476 ms) : 1465, 1488
. : milestone, 1476,
appsec (2.373 ms) : 2329, 2416
. : milestone, 2373,
iast (2.125 ms) : 2070, 2180
. : milestone, 2125,
iast_GLOBAL (2.163 ms) : 2107, 2218
. : milestone, 2163,
profiling (1.971 ms) : 1928, 2015
. : milestone, 1971,
tracing (1.95 ms) : 1908, 1992
. : milestone, 1950,
section candidate
no_agent (1.477 ms) : 1465, 1488
. : milestone, 1477,
appsec (2.375 ms) : 2332, 2418
. : milestone, 2375,
iast (2.126 ms) : 2070, 2181
. : milestone, 2126,
iast_GLOBAL (2.172 ms) : 2116, 2228
. : milestone, 2172,
profiling (1.991 ms) : 1946, 2036
. : milestone, 1991,
tracing (1.958 ms) : 1915, 2000
. : milestone, 1958,
|
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
|
ab26d6f
to
79df06c
Compare
28f550d
to
92505f7
Compare
# Conflicts: # dd-java-agent/agent-debugger/src/main/java/com/datadog/debugger/probe/CodeOriginProbe.java
92505f7
to
0eddcc4
Compare
update constant renames from rebase
dd-java-agent/agent-debugger/src/main/java/com/datadog/debugger/probe/CodeOriginProbe.java
Outdated
Show resolved
Hide resolved
|
||
@Override | ||
public boolean isEnabled() { | ||
return InstrumenterConfig.get().isCodeOriginEnabled() && super.isEnabled(); |
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.
@jpbempel I added this extra check after remembering the other CO advices do this check here to short circuit out in the cases where CO is not enabled.
What Does This Do
Adds code origin information to client handlers for processing kafka events.
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: DEBUG-3157