-
Notifications
You must be signed in to change notification settings - Fork 290
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 JSON component #7973
base: master
Are you sure you want to change the base?
Add JSON component #7973
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. |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 54 metrics, 9 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.43.0-SNAPSHOT~ab3a7c0181, baseline=1.43.0-SNAPSHOT~603ef416b6
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.087 s) : 0, 1087183
Total [baseline] (8.63 s) : 0, 8630205
Agent [candidate] (1.105 s) : 0, 1104755
Total [candidate] (8.702 s) : 0, 8702442
section iast
Agent [baseline] (1.224 s) : 0, 1224479
Total [baseline] (9.202 s) : 0, 9201627
Agent [candidate] (1.216 s) : 0, 1216491
Total [candidate] (9.199 s) : 0, 9199318
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.217 s) : 0, 1216949
Total [baseline] (9.156 s) : 0, 9156236
Agent [candidate] (1.227 s) : 0, 1227278
Total [candidate] (9.212 s) : 0, 9212244
section iast_TELEMETRY_OFF
Agent [baseline] (1.221 s) : 0, 1220929
Total [baseline] (9.185 s) : 0, 9185498
Agent [candidate] (1.226 s) : 0, 1226418
Total [candidate] (9.22 s) : 0, 9220026
gantt
title insecure-bank - break down per module: candidate=1.43.0-SNAPSHOT~ab3a7c0181, baseline=1.43.0-SNAPSHOT~603ef416b6
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (692.748 ms) : 0, 692748
BytebuddyAgent [candidate] (705.078 ms) : 0, 705078
GlobalTracer [baseline] (316.758 ms) : 0, 316758
GlobalTracer [candidate] (320.149 ms) : 0, 320149
AppSec [baseline] (54.654 ms) : 0, 54654
AppSec [candidate] (55.405 ms) : 0, 55405
Remote Config [baseline] (681.84 µs) : 0, 682
Remote Config [candidate] (704.687 µs) : 0, 705
Telemetry [baseline] (8.568 ms) : 0, 8568
Telemetry [candidate] (9.386 ms) : 0, 9386
section iast
BytebuddyAgent [baseline] (815.519 ms) : 0, 815519
BytebuddyAgent [candidate] (809.316 ms) : 0, 809316
GlobalTracer [baseline] (307.739 ms) : 0, 307739
GlobalTracer [candidate] (306.68 ms) : 0, 306680
AppSec [baseline] (55.846 ms) : 0, 55846
AppSec [candidate] (56.362 ms) : 0, 56362
Remote Config [baseline] (624.322 µs) : 0, 624
Remote Config [candidate] (615.785 µs) : 0, 616
Telemetry [baseline] (7.458 ms) : 0, 7458
Telemetry [candidate] (7.413 ms) : 0, 7413
IAST [baseline] (23.435 ms) : 0, 23435
IAST [candidate] (22.353 ms) : 0, 22353
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (809.199 ms) : 0, 809199
BytebuddyAgent [candidate] (815.737 ms) : 0, 815737
GlobalTracer [baseline] (307.032 ms) : 0, 307032
GlobalTracer [candidate] (309.927 ms) : 0, 309927
AppSec [baseline] (58.036 ms) : 0, 58036
AppSec [candidate] (57.589 ms) : 0, 57589
Remote Config [baseline] (622.963 µs) : 0, 623
Remote Config [candidate] (633.25 µs) : 0, 633
Telemetry [baseline] (7.514 ms) : 0, 7514
Telemetry [candidate] (7.518 ms) : 0, 7518
IAST [baseline] (20.8 ms) : 0, 20800
IAST [candidate] (21.976 ms) : 0, 21976
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (811.736 ms) : 0, 811736
BytebuddyAgent [candidate] (816.934 ms) : 0, 816934
GlobalTracer [baseline] (307.546 ms) : 0, 307546
GlobalTracer [candidate] (308.163 ms) : 0, 308163
AppSec [baseline] (56.389 ms) : 0, 56389
AppSec [candidate] (58.754 ms) : 0, 58754
Remote Config [baseline] (629.376 µs) : 0, 629
Remote Config [candidate] (629.725 µs) : 0, 630
Telemetry [baseline] (7.511 ms) : 0, 7511
Telemetry [candidate] (7.491 ms) : 0, 7491
IAST [baseline] (23.315 ms) : 0, 23315
IAST [candidate] (20.538 ms) : 0, 20538
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.43.0-SNAPSHOT~ab3a7c0181, baseline=1.43.0-SNAPSHOT~603ef416b6
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.098 s) : 0, 1097552
Total [baseline] (10.47 s) : 0, 10469694
Agent [candidate] (1.085 s) : 0, 1085267
Total [candidate] (10.462 s) : 0, 10461976
section appsec
Agent [baseline] (1.222 s) : 0, 1221659
Total [baseline] (10.724 s) : 0, 10723957
Agent [candidate] (1.225 s) : 0, 1224557
Total [candidate] (10.709 s) : 0, 10709007
section iast
Agent [baseline] (1.217 s) : 0, 1217405
Total [baseline] (10.977 s) : 0, 10976999
Agent [candidate] (1.217 s) : 0, 1216586
Total [candidate] (10.945 s) : 0, 10945436
section profiling
Agent [baseline] (1.288 s) : 0, 1287995
Total [baseline] (10.84 s) : 0, 10840116
Agent [candidate] (1.285 s) : 0, 1284547
Total [candidate] (10.777 s) : 0, 10777078
gantt
title petclinic - break down per module: candidate=1.43.0-SNAPSHOT~ab3a7c0181, baseline=1.43.0-SNAPSHOT~603ef416b6
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (697.513 ms) : 0, 697513
BytebuddyAgent [candidate] (691.085 ms) : 0, 691085
GlobalTracer [baseline] (320.958 ms) : 0, 320958
GlobalTracer [candidate] (316.662 ms) : 0, 316662
AppSec [baseline] (55.194 ms) : 0, 55194
AppSec [candidate] (54.565 ms) : 0, 54565
Remote Config [baseline] (688.829 µs) : 0, 689
Remote Config [candidate] (681.527 µs) : 0, 682
Telemetry [baseline] (9.386 ms) : 0, 9386
Telemetry [candidate] (8.572 ms) : 0, 8572
section appsec
BytebuddyAgent [baseline] (708.462 ms) : 0, 708462
BytebuddyAgent [candidate] (709.848 ms) : 0, 709848
GlobalTracer [baseline] (313.533 ms) : 0, 313533
GlobalTracer [candidate] (315.185 ms) : 0, 315185
AppSec [baseline] (166.475 ms) : 0, 166475
AppSec [candidate] (165.856 ms) : 0, 165856
Remote Config [baseline] (641.502 µs) : 0, 642
Remote Config [candidate] (656.284 µs) : 0, 656
Telemetry [baseline] (8.944 ms) : 0, 8944
Telemetry [candidate] (8.936 ms) : 0, 8936
IAST [baseline] (19.809 ms) : 0, 19809
IAST [candidate] (20.623 ms) : 0, 20623
section iast
BytebuddyAgent [baseline] (810.481 ms) : 0, 810481
BytebuddyAgent [candidate] (809.704 ms) : 0, 809704
GlobalTracer [baseline] (306.633 ms) : 0, 306633
GlobalTracer [candidate] (306.306 ms) : 0, 306306
AppSec [baseline] (57.045 ms) : 0, 57045
AppSec [candidate] (56.36 ms) : 0, 56360
Remote Config [baseline] (616.577 µs) : 0, 617
Remote Config [candidate] (614.233 µs) : 0, 614
Telemetry [baseline] (7.361 ms) : 0, 7361
Telemetry [candidate] (7.426 ms) : 0, 7426
IAST [baseline] (21.564 ms) : 0, 21564
IAST [candidate] (22.437 ms) : 0, 22437
section profiling
BytebuddyAgent [baseline] (687.27 ms) : 0, 687270
BytebuddyAgent [candidate] (685.096 ms) : 0, 685096
GlobalTracer [baseline] (400.948 ms) : 0, 400948
GlobalTracer [candidate] (399.954 ms) : 0, 399954
AppSec [baseline] (55.245 ms) : 0, 55245
AppSec [candidate] (55.067 ms) : 0, 55067
Remote Config [baseline] (675.475 µs) : 0, 675
Remote Config [candidate] (672.243 µs) : 0, 672
Telemetry [baseline] (11.441 ms) : 0, 11441
Telemetry [candidate] (12.769 ms) : 0, 12769
ProfilingAgent [baseline] (93.433 ms) : 0, 93433
ProfilingAgent [candidate] (91.966 ms) : 0, 91966
Profiling [baseline] (93.457 ms) : 0, 93457
Profiling [candidate] (91.989 ms) : 0, 91989
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 16 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.43.0-SNAPSHOT~ab3a7c0181, baseline=1.43.0-SNAPSHOT~603ef416b6
dateFormat X
axisFormat %s
section baseline
no_agent (377.749 µs) : 358, 398
. : milestone, 378,
iast (486.229 µs) : 465, 508
. : milestone, 486,
iast_FULL (646.58 µs) : 625, 668
. : milestone, 647,
iast_GLOBAL (515.836 µs) : 494, 538
. : milestone, 516,
iast_HARDCODED_SECRET_DISABLED (486.296 µs) : 465, 507
. : milestone, 486,
iast_INACTIVE (445.373 µs) : 425, 466
. : milestone, 445,
iast_TELEMETRY_OFF (474.033 µs) : 453, 495
. : milestone, 474,
tracing (444.834 µs) : 424, 465
. : milestone, 445,
section candidate
no_agent (373.272 µs) : 354, 393
. : milestone, 373,
iast (485.01 µs) : 464, 506
. : milestone, 485,
iast_FULL (647.442 µs) : 626, 669
. : milestone, 647,
iast_GLOBAL (525.456 µs) : 503, 548
. : milestone, 525,
iast_HARDCODED_SECRET_DISABLED (490.694 µs) : 469, 513
. : milestone, 491,
iast_INACTIVE (443.909 µs) : 423, 465
. : milestone, 444,
iast_TELEMETRY_OFF (475.055 µs) : 454, 497
. : milestone, 475,
tracing (449.011 µs) : 428, 470
. : milestone, 449,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.43.0-SNAPSHOT~ab3a7c0181, baseline=1.43.0-SNAPSHOT~603ef416b6
dateFormat X
axisFormat %s
section baseline
no_agent (1.342 ms) : 1322, 1363
. : milestone, 1342,
appsec (1.756 ms) : 1732, 1781
. : milestone, 1756,
appsec_no_iast (1.747 ms) : 1722, 1772
. : milestone, 1747,
iast (1.511 ms) : 1489, 1533
. : milestone, 1511,
profiling (1.556 ms) : 1531, 1581
. : milestone, 1556,
tracing (1.504 ms) : 1480, 1529
. : milestone, 1504,
section candidate
no_agent (1.344 ms) : 1324, 1364
. : milestone, 1344,
appsec (1.738 ms) : 1713, 1763
. : milestone, 1738,
appsec_no_iast (1.728 ms) : 1703, 1754
. : milestone, 1728,
iast (1.499 ms) : 1476, 1521
. : milestone, 1499,
profiling (1.512 ms) : 1489, 1535
. : milestone, 1512,
tracing (1.492 ms) : 1469, 1516
. : milestone, 1492,
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.43.0-SNAPSHOT~ab3a7c0181, baseline=1.43.0-SNAPSHOT~603ef416b6
dateFormat X
axisFormat %s
section baseline
no_agent (15.357 s) : 15357000, 15357000
. : milestone, 15357000,
appsec (15.068 s) : 15068000, 15068000
. : milestone, 15068000,
iast (18.826 s) : 18826000, 18826000
. : milestone, 18826000,
iast_GLOBAL (18.3 s) : 18300000, 18300000
. : milestone, 18300000,
profiling (14.779 s) : 14779000, 14779000
. : milestone, 14779000,
tracing (14.869 s) : 14869000, 14869000
. : milestone, 14869000,
section candidate
no_agent (15.186 s) : 15186000, 15186000
. : milestone, 15186000,
appsec (14.943 s) : 14943000, 14943000
. : milestone, 14943000,
iast (18.532 s) : 18532000, 18532000
. : milestone, 18532000,
iast_GLOBAL (18.048 s) : 18048000, 18048000
. : milestone, 18048000,
profiling (15.324 s) : 15324000, 15324000
. : milestone, 15324000,
tracing (15.093 s) : 15093000, 15093000
. : milestone, 15093000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.43.0-SNAPSHOT~ab3a7c0181, baseline=1.43.0-SNAPSHOT~603ef416b6
dateFormat X
axisFormat %s
section baseline
no_agent (1.463 ms) : 1451, 1474
. : milestone, 1463,
appsec (2.33 ms) : 2289, 2372
. : milestone, 2330,
iast (2.068 ms) : 2016, 2120
. : milestone, 2068,
iast_GLOBAL (2.116 ms) : 2064, 2169
. : milestone, 2116,
profiling (1.956 ms) : 1914, 1999
. : milestone, 1956,
tracing (1.909 ms) : 1869, 1948
. : milestone, 1909,
section candidate
no_agent (1.462 ms) : 1451, 1474
. : milestone, 1462,
appsec (2.314 ms) : 2274, 2355
. : milestone, 2314,
iast (2.074 ms) : 2022, 2126
. : milestone, 2074,
iast_GLOBAL (2.111 ms) : 2059, 2163
. : milestone, 2111,
profiling (1.942 ms) : 1900, 1984
. : milestone, 1942,
tracing (1.914 ms) : 1873, 1954
. : milestone, 1914,
|
9eff3df
to
0a2b5c0
Compare
components/json/json-io/src/main/java/datadog/json/io/JsonWriter.java
Outdated
Show resolved
Hide resolved
b6f3e76
to
546b7c8
Compare
546b7c8
to
ab3a7c0
Compare
What Does This Do
This PR introduces a platform module to craft JSON payload.
It provides a minimalist writer to craft JSON objects and arrays, and mapper to convert simple Java object.
Motivation
The goals is to provide common capability to create simple JSON payload without relying on a JSON library that can conflict with the instrumented application.
As more and more products and features relies on JSON, gathering this capability into a dedicated reusable platform component allows to simplify the code base and consolidate the feature (in term of capability and testing) in one place.
Additional Notes
This PR is split to easily review the impact on each features relying on JSON that were migrated to this new component. It will be squashed on merge.
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]