-
Notifications
You must be signed in to change notification settings - Fork 278
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
Avro instrumentation for schema tracking #7236
Avro instrumentation for schema tracking #7236
Conversation
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 1 performance regressions! Performance is the same for 50 metrics, 12 unstable metrics.
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.36.0-SNAPSHOT~dc5f4fb76e, baseline=1.36.0-SNAPSHOT~2f63f03a0d
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.06 s) : 0, 1060415
Total [baseline] (10.313 s) : 0, 10313058
Agent [candidate] (1.061 s) : 0, 1061488
Total [candidate] (10.334 s) : 0, 10334268
section appsec
Agent [baseline] (1.186 s) : 0, 1186418
Total [baseline] (10.444 s) : 0, 10443535
Agent [candidate] (1.18 s) : 0, 1179706
Total [candidate] (10.42 s) : 0, 10420339
section iast
Agent [baseline] (1.168 s) : 0, 1167941
Total [baseline] (10.671 s) : 0, 10671203
Agent [candidate] (1.186 s) : 0, 1186494
Total [candidate] (10.833 s) : 0, 10833228
section profiling
Agent [baseline] (1.266 s) : 0, 1265938
Total [baseline] (10.595 s) : 0, 10595381
Agent [candidate] (1.27 s) : 0, 1270176
Total [candidate] (10.692 s) : 0, 10691913
gantt
title petclinic - break down per module: candidate=1.36.0-SNAPSHOT~dc5f4fb76e, baseline=1.36.0-SNAPSHOT~2f63f03a0d
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (664.573 ms) : 0, 664573
BytebuddyAgent [candidate] (665.4 ms) : 0, 665400
GlobalTracer [baseline] (303.51 ms) : 0, 303510
GlobalTracer [candidate] (303.3 ms) : 0, 303300
AppSec [baseline] (49.66 ms) : 0, 49660
AppSec [candidate] (50.148 ms) : 0, 50148
Remote Config [baseline] (691.096 µs) : 0, 691
Remote Config [candidate] (680.043 µs) : 0, 680
Telemetry [baseline] (7.603 ms) : 0, 7603
Telemetry [candidate] (7.551 ms) : 0, 7551
section appsec
BytebuddyAgent [baseline] (677.455 ms) : 0, 677455
BytebuddyAgent [candidate] (674.539 ms) : 0, 674539
GlobalTracer [baseline] (299.552 ms) : 0, 299552
GlobalTracer [candidate] (297.133 ms) : 0, 297133
AppSec [baseline] (153.672 ms) : 0, 153672
AppSec [candidate] (153.678 ms) : 0, 153678
IAST [baseline] (22.534 ms) : 0, 22534
IAST [candidate] (19.679 ms) : 0, 19679
Remote Config [baseline] (640.109 µs) : 0, 640
Remote Config [candidate] (632.983 µs) : 0, 633
Telemetry [baseline] (9.388 ms) : 0, 9388
Telemetry [candidate] (9.472 ms) : 0, 9472
section iast
BytebuddyAgent [baseline] (778.652 ms) : 0, 778652
BytebuddyAgent [candidate] (789.398 ms) : 0, 789398
GlobalTracer [baseline] (293.965 ms) : 0, 293965
GlobalTracer [candidate] (298.78 ms) : 0, 298780
AppSec [baseline] (47.108 ms) : 0, 47108
AppSec [candidate] (47.93 ms) : 0, 47930
IAST [baseline] (27.389 ms) : 0, 27389
IAST [candidate] (29.214 ms) : 0, 29214
Remote Config [baseline] (618.245 µs) : 0, 618
Remote Config [candidate] (638.896 µs) : 0, 639
Telemetry [baseline] (6.963 ms) : 0, 6963
Telemetry [candidate] (7.046 ms) : 0, 7046
section profiling
BytebuddyAgent [baseline] (664.447 ms) : 0, 664447
BytebuddyAgent [candidate] (667.111 ms) : 0, 667111
GlobalTracer [baseline] (388.829 ms) : 0, 388829
GlobalTracer [candidate] (388.159 ms) : 0, 388159
AppSec [baseline] (51.162 ms) : 0, 51162
AppSec [candidate] (52.04 ms) : 0, 52040
Remote Config [baseline] (682.917 µs) : 0, 683
Remote Config [candidate] (728.541 µs) : 0, 729
Telemetry [baseline] (7.393 ms) : 0, 7393
Telemetry [candidate] (7.39 ms) : 0, 7390
ProfilingAgent [baseline] (96.092 ms) : 0, 96092
ProfilingAgent [candidate] (97.453 ms) : 0, 97453
Profiling [baseline] (96.116 ms) : 0, 96116
Profiling [candidate] (97.478 ms) : 0, 97478
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.36.0-SNAPSHOT~dc5f4fb76e, baseline=1.36.0-SNAPSHOT~2f63f03a0d
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.067 s) : 0, 1066952
Total [baseline] (8.532 s) : 0, 8532302
Agent [candidate] (1.062 s) : 0, 1061568
Total [candidate] (8.546 s) : 0, 8546189
section iast
Agent [baseline] (1.17 s) : 0, 1169588
Total [baseline] (8.972 s) : 0, 8972398
Agent [candidate] (1.166 s) : 0, 1166217
Total [candidate] (8.978 s) : 0, 8978144
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.174 s) : 0, 1174009
Total [baseline] (8.992 s) : 0, 8992252
Agent [candidate] (1.169 s) : 0, 1168522
Total [candidate] (9.006 s) : 0, 9005969
section iast_TELEMETRY_OFF
Agent [baseline] (1.163 s) : 0, 1163130
Total [baseline] (8.999 s) : 0, 8999461
Agent [candidate] (1.161 s) : 0, 1160820
Total [candidate] (8.969 s) : 0, 8968541
gantt
title insecure-bank - break down per module: candidate=1.36.0-SNAPSHOT~dc5f4fb76e, baseline=1.36.0-SNAPSHOT~2f63f03a0d
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (668.643 ms) : 0, 668643
BytebuddyAgent [candidate] (665.113 ms) : 0, 665113
GlobalTracer [baseline] (305.768 ms) : 0, 305768
GlobalTracer [candidate] (303.593 ms) : 0, 303593
AppSec [baseline] (49.91 ms) : 0, 49910
AppSec [candidate] (50.21 ms) : 0, 50210
Remote Config [baseline] (705.417 µs) : 0, 705
Remote Config [candidate] (685.338 µs) : 0, 685
Telemetry [baseline] (7.491 ms) : 0, 7491
Telemetry [candidate] (7.603 ms) : 0, 7603
section iast
BytebuddyAgent [baseline] (780.038 ms) : 0, 780038
BytebuddyAgent [candidate] (777.549 ms) : 0, 777549
GlobalTracer [baseline] (293.611 ms) : 0, 293611
GlobalTracer [candidate] (293.276 ms) : 0, 293276
AppSec [baseline] (47.124 ms) : 0, 47124
AppSec [candidate] (47.21 ms) : 0, 47210
IAST [baseline] (28.125 ms) : 0, 28125
IAST [candidate] (27.371 ms) : 0, 27371
Remote Config [baseline] (563.771 µs) : 0, 564
Remote Config [candidate] (599.165 µs) : 0, 599
Telemetry [baseline] (6.886 ms) : 0, 6886
Telemetry [candidate] (6.932 ms) : 0, 6932
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (782.049 ms) : 0, 782049
BytebuddyAgent [candidate] (778.512 ms) : 0, 778512
GlobalTracer [baseline] (294.839 ms) : 0, 294839
GlobalTracer [candidate] (293.582 ms) : 0, 293582
AppSec [baseline] (47.342 ms) : 0, 47342
AppSec [candidate] (47.228 ms) : 0, 47228
IAST [baseline] (27.436 ms) : 0, 27436
IAST [candidate] (28.242 ms) : 0, 28242
Remote Config [baseline] (582.728 µs) : 0, 583
Remote Config [candidate] (627.337 µs) : 0, 627
Telemetry [baseline] (8.443 ms) : 0, 8443
Telemetry [candidate] (7.021 ms) : 0, 7021
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (773.682 ms) : 0, 773682
BytebuddyAgent [candidate] (772.926 ms) : 0, 772926
GlobalTracer [baseline] (293.183 ms) : 0, 293183
GlobalTracer [candidate] (292.534 ms) : 0, 292534
AppSec [baseline] (46.924 ms) : 0, 46924
AppSec [candidate] (46.601 ms) : 0, 46601
IAST [baseline] (28.606 ms) : 0, 28606
IAST [candidate] (26.572 ms) : 0, 26572
Remote Config [baseline] (595.263 µs) : 0, 595
Remote Config [candidate] (584.976 µs) : 0, 585
Telemetry [baseline] (6.838 ms) : 0, 6838
Telemetry [candidate] (8.361 ms) : 0, 8361
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 13 metrics, 15 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.36.0-SNAPSHOT~dc5f4fb76e, baseline=1.36.0-SNAPSHOT~2f63f03a0d
dateFormat X
axisFormat %s
section baseline
no_agent (375.027 µs) : 355, 395
. : milestone, 375,
iast (497.908 µs) : 477, 519
. : milestone, 498,
iast_FULL (564.888 µs) : 544, 586
. : milestone, 565,
iast_GLOBAL (508.954 µs) : 488, 530
. : milestone, 509,
iast_HARDCODED_SECRET_DISABLED (494.157 µs) : 473, 516
. : milestone, 494,
iast_INACTIVE (463.611 µs) : 442, 485
. : milestone, 464,
iast_TELEMETRY_OFF (481.158 µs) : 460, 502
. : milestone, 481,
tracing (455.831 µs) : 434, 477
. : milestone, 456,
section candidate
no_agent (381.478 µs) : 359, 404
. : milestone, 381,
iast (497.175 µs) : 475, 519
. : milestone, 497,
iast_FULL (562.127 µs) : 541, 584
. : milestone, 562,
iast_GLOBAL (517.31 µs) : 495, 539
. : milestone, 517,
iast_HARDCODED_SECRET_DISABLED (490.801 µs) : 469, 512
. : milestone, 491,
iast_INACTIVE (460.543 µs) : 440, 482
. : milestone, 461,
iast_TELEMETRY_OFF (484.262 µs) : 463, 506
. : milestone, 484,
tracing (449.322 µs) : 429, 470
. : milestone, 449,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.36.0-SNAPSHOT~dc5f4fb76e, baseline=1.36.0-SNAPSHOT~2f63f03a0d
dateFormat X
axisFormat %s
section baseline
no_agent (1.379 ms) : 1360, 1398
. : milestone, 1379,
appsec (1.714 ms) : 1689, 1739
. : milestone, 1714,
appsec_no_iast (1.744 ms) : 1720, 1768
. : milestone, 1744,
iast (1.491 ms) : 1468, 1514
. : milestone, 1491,
profiling (1.507 ms) : 1483, 1532
. : milestone, 1507,
tracing (1.46 ms) : 1435, 1484
. : milestone, 1460,
section candidate
no_agent (1.332 ms) : 1313, 1351
. : milestone, 1332,
appsec (1.712 ms) : 1687, 1736
. : milestone, 1712,
appsec_no_iast (1.711 ms) : 1686, 1736
. : milestone, 1711,
iast (1.504 ms) : 1482, 1526
. : milestone, 1504,
profiling (1.521 ms) : 1494, 1547
. : milestone, 1521,
tracing (1.482 ms) : 1458, 1506
. : milestone, 1482,
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 biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.36.0-SNAPSHOT~dc5f4fb76e, baseline=1.36.0-SNAPSHOT~2f63f03a0d
dateFormat X
axisFormat %s
section baseline
no_agent (15.426 s) : 15426000, 15426000
. : milestone, 15426000,
appsec (15.17 s) : 15170000, 15170000
. : milestone, 15170000,
iast (18.995 s) : 18995000, 18995000
. : milestone, 18995000,
iast_GLOBAL (17.947 s) : 17947000, 17947000
. : milestone, 17947000,
profiling (15.047 s) : 15047000, 15047000
. : milestone, 15047000,
tracing (15.022 s) : 15022000, 15022000
. : milestone, 15022000,
section candidate
no_agent (15.075 s) : 15075000, 15075000
. : milestone, 15075000,
appsec (14.928 s) : 14928000, 14928000
. : milestone, 14928000,
iast (18.787 s) : 18787000, 18787000
. : milestone, 18787000,
iast_GLOBAL (17.687 s) : 17687000, 17687000
. : milestone, 17687000,
profiling (15.481 s) : 15481000, 15481000
. : milestone, 15481000,
tracing (14.997 s) : 14997000, 14997000
. : milestone, 14997000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.36.0-SNAPSHOT~dc5f4fb76e, baseline=1.36.0-SNAPSHOT~2f63f03a0d
dateFormat X
axisFormat %s
section baseline
no_agent (1.463 ms) : 1452, 1475
. : milestone, 1463,
appsec (2.218 ms) : 2184, 2253
. : milestone, 2218,
iast (1.975 ms) : 1934, 2016
. : milestone, 1975,
iast_GLOBAL (2.013 ms) : 1972, 2053
. : milestone, 2013,
profiling (2.322 ms) : 2144, 2501
. : milestone, 2322,
tracing (1.841 ms) : 1809, 1874
. : milestone, 1841,
section candidate
no_agent (1.468 ms) : 1456, 1479
. : milestone, 1468,
appsec (2.22 ms) : 2186, 2254
. : milestone, 2220,
iast (1.983 ms) : 1941, 2025
. : milestone, 1983,
iast_GLOBAL (2.011 ms) : 1970, 2052
. : milestone, 2011,
profiling (1.87 ms) : 1836, 1903
. : milestone, 1870,
tracing (1.845 ms) : 1812, 1877
. : milestone, 1845,
|
...avro/src/main/java/datadog/trace/instrumentation/avro/GenericDatumReaderInstrumentation.java
Outdated
Show resolved
Hide resolved
...avro/src/main/java/datadog/trace/instrumentation/avro/GenericDatumWriterInstrumentation.java
Outdated
Show resolved
Hide resolved
...avro/src/main/java/datadog/trace/instrumentation/avro/GenericDatumReaderInstrumentation.java
Outdated
Show resolved
Hide resolved
...t/instrumentation/avro/src/main/java/datadog/trace/instrumentation/avro/SchemaExtractor.java
Outdated
Show resolved
Hide resolved
...avro/src/main/java/datadog/trace/instrumentation/avro/GenericDatumReaderInstrumentation.java
Show resolved
Hide resolved
...avro/src/main/java/datadog/trace/instrumentation/avro/GenericDatumWriterInstrumentation.java
Show resolved
Hide resolved
...avro/src/main/java/datadog/trace/instrumentation/avro/GenericDatumReaderInstrumentation.java
Outdated
Show resolved
Hide resolved
...avro/src/main/java/datadog/trace/instrumentation/avro/GenericDatumWriterInstrumentation.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.
Glad to see compileOnly
solved your issue!
What Does This Do
Adds schema tracking functionality for Avro's serializing library
Motivation
Additional Notes
Jira ticket: AIT-10052