-
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
prevent errors when dealing with MinimalStage in graphql #7560
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 50 metrics, 13 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.40.0-SNAPSHOT~9c17bb943d, baseline=1.40.0-SNAPSHOT~fb9b96a2fa
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.063 s) : 0, 1062708
Total [baseline] (8.553 s) : 0, 8553152
Agent [candidate] (1.051 s) : 0, 1050611
Total [candidate] (8.506 s) : 0, 8505866
section iast
Agent [baseline] (1.172 s) : 0, 1172345
Total [baseline] (9.016 s) : 0, 9015897
Agent [candidate] (1.17 s) : 0, 1170143
Total [candidate] (8.957 s) : 0, 8957443
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.178 s) : 0, 1177770
Total [baseline] (8.931 s) : 0, 8930791
Agent [candidate] (1.173 s) : 0, 1172955
Total [candidate] (8.959 s) : 0, 8958925
section iast_TELEMETRY_OFF
Agent [baseline] (1.169 s) : 0, 1168988
Total [baseline] (8.938 s) : 0, 8937964
Agent [candidate] (1.17 s) : 0, 1170347
Total [candidate] (8.989 s) : 0, 8989441
gantt
title insecure-bank - break down per module: candidate=1.40.0-SNAPSHOT~9c17bb943d, baseline=1.40.0-SNAPSHOT~fb9b96a2fa
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (678.936 ms) : 0, 678936
BytebuddyAgent [candidate] (670.338 ms) : 0, 670338
GlobalTracer [baseline] (309.809 ms) : 0, 309809
GlobalTracer [candidate] (307.362 ms) : 0, 307362
AppSec [baseline] (51.999 ms) : 0, 51999
AppSec [candidate] (51.215 ms) : 0, 51215
Remote Config [baseline] (692.914 µs) : 0, 693
Remote Config [candidate] (670.948 µs) : 0, 671
Telemetry [baseline] (7.555 ms) : 0, 7555
Telemetry [candidate] (7.445 ms) : 0, 7445
section iast
BytebuddyAgent [baseline] (780.679 ms) : 0, 780679
BytebuddyAgent [candidate] (777.687 ms) : 0, 777687
GlobalTracer [baseline] (295.453 ms) : 0, 295453
GlobalTracer [candidate] (295.132 ms) : 0, 295132
AppSec [baseline] (49.84 ms) : 0, 49840
AppSec [candidate] (53.945 ms) : 0, 53945
IAST [baseline] (24.94 ms) : 0, 24940
IAST [candidate] (21.93 ms) : 0, 21930
Remote Config [baseline] (654.758 µs) : 0, 655
Remote Config [candidate] (585.322 µs) : 0, 585
Telemetry [baseline] (7.218 ms) : 0, 7218
Telemetry [candidate] (7.304 ms) : 0, 7304
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (784.732 ms) : 0, 784732
BytebuddyAgent [candidate] (779.581 ms) : 0, 779581
GlobalTracer [baseline] (295.768 ms) : 0, 295768
GlobalTracer [candidate] (296.11 ms) : 0, 296110
AppSec [baseline] (52.386 ms) : 0, 52386
AppSec [candidate] (53.993 ms) : 0, 53993
IAST [baseline] (23.335 ms) : 0, 23335
IAST [candidate] (21.801 ms) : 0, 21801
Remote Config [baseline] (585.215 µs) : 0, 585
Remote Config [candidate] (577.555 µs) : 0, 578
Telemetry [baseline] (7.277 ms) : 0, 7277
Telemetry [candidate] (7.274 ms) : 0, 7274
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (777.478 ms) : 0, 777478
BytebuddyAgent [candidate] (777.362 ms) : 0, 777362
GlobalTracer [baseline] (295.76 ms) : 0, 295760
GlobalTracer [candidate] (296.295 ms) : 0, 296295
AppSec [baseline] (53.003 ms) : 0, 53003
AppSec [candidate] (52.328 ms) : 0, 52328
IAST [baseline] (21.411 ms) : 0, 21411
IAST [candidate] (23.019 ms) : 0, 23019
Remote Config [baseline] (577.666 µs) : 0, 578
Remote Config [candidate] (568.171 µs) : 0, 568
Telemetry [baseline] (7.162 ms) : 0, 7162
Telemetry [candidate] (7.155 ms) : 0, 7155
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.40.0-SNAPSHOT~9c17bb943d, baseline=1.40.0-SNAPSHOT~fb9b96a2fa
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.051 s) : 0, 1050635
Total [baseline] (10.375 s) : 0, 10375040
Agent [candidate] (1.066 s) : 0, 1065846
Total [candidate] (10.424 s) : 0, 10423657
section appsec
Agent [baseline] (1.179 s) : 0, 1179283
Total [baseline] (10.566 s) : 0, 10566250
Agent [candidate] (1.179 s) : 0, 1179089
Total [candidate] (10.6 s) : 0, 10599640
section iast
Agent [baseline] (1.172 s) : 0, 1172142
Total [baseline] (10.756 s) : 0, 10755753
Agent [candidate] (1.175 s) : 0, 1174835
Total [candidate] (10.806 s) : 0, 10806057
section profiling
Agent [baseline] (1.246 s) : 0, 1245935
Total [baseline] (10.623 s) : 0, 10623304
Agent [candidate] (1.248 s) : 0, 1247779
Total [candidate] (10.594 s) : 0, 10594393
gantt
title petclinic - break down per module: candidate=1.40.0-SNAPSHOT~9c17bb943d, baseline=1.40.0-SNAPSHOT~fb9b96a2fa
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (670.344 ms) : 0, 670344
BytebuddyAgent [candidate] (679.973 ms) : 0, 679973
GlobalTracer [baseline] (307.155 ms) : 0, 307155
GlobalTracer [candidate] (311.349 ms) : 0, 311349
AppSec [baseline] (51.427 ms) : 0, 51427
AppSec [candidate] (52.407 ms) : 0, 52407
Remote Config [baseline] (674.692 µs) : 0, 675
Remote Config [candidate] (687.572 µs) : 0, 688
Telemetry [baseline] (7.466 ms) : 0, 7466
Telemetry [candidate] (7.652 ms) : 0, 7652
section appsec
BytebuddyAgent [baseline] (688.114 ms) : 0, 688114
BytebuddyAgent [candidate] (687.736 ms) : 0, 687736
GlobalTracer [baseline] (299.131 ms) : 0, 299131
GlobalTracer [candidate] (299.695 ms) : 0, 299695
AppSec [baseline] (159.871 ms) : 0, 159871
AppSec [candidate] (158.777 ms) : 0, 158777
Remote Config [baseline] (623.726 µs) : 0, 624
Remote Config [candidate] (626.162 µs) : 0, 626
Telemetry [baseline] (8.19 ms) : 0, 8190
Telemetry [candidate] (8.536 ms) : 0, 8536
IAST [baseline] (19.693 ms) : 0, 19693
IAST [candidate] (20.386 ms) : 0, 20386
section iast
BytebuddyAgent [baseline] (779.947 ms) : 0, 779947
BytebuddyAgent [candidate] (781.549 ms) : 0, 781549
GlobalTracer [baseline] (295.375 ms) : 0, 295375
GlobalTracer [candidate] (296.288 ms) : 0, 296288
AppSec [baseline] (51.315 ms) : 0, 51315
AppSec [candidate] (49.727 ms) : 0, 49727
Remote Config [baseline] (2.046 ms) : 0, 2046
Remote Config [candidate] (585.817 µs) : 0, 586
Telemetry [baseline] (8.065 ms) : 0, 8065
Telemetry [candidate] (9.479 ms) : 0, 9479
IAST [baseline] (21.8 ms) : 0, 21800
IAST [candidate] (23.605 ms) : 0, 23605
section profiling
BytebuddyAgent [baseline] (664.227 ms) : 0, 664227
BytebuddyAgent [candidate] (664.116 ms) : 0, 664116
GlobalTracer [baseline] (388.047 ms) : 0, 388047
GlobalTracer [candidate] (389.335 ms) : 0, 389335
AppSec [baseline] (52.16 ms) : 0, 52160
AppSec [candidate] (52.284 ms) : 0, 52284
Remote Config [baseline] (677.852 µs) : 0, 678
Remote Config [candidate] (704.856 µs) : 0, 705
Telemetry [baseline] (7.365 ms) : 0, 7365
Telemetry [candidate] (7.364 ms) : 0, 7364
ProfilingAgent [baseline] (95.71 ms) : 0, 95710
ProfilingAgent [candidate] (96.235 ms) : 0, 96235
Profiling [baseline] (95.733 ms) : 0, 95733
Profiling [candidate] (96.258 ms) : 0, 96258
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 10 metrics, 18 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.40.0-SNAPSHOT~9c17bb943d, baseline=1.40.0-SNAPSHOT~fb9b96a2fa
dateFormat X
axisFormat %s
section baseline
no_agent (368.567 µs) : 349, 388
. : milestone, 369,
iast (482.809 µs) : 461, 505
. : milestone, 483,
iast_FULL (551.882 µs) : 531, 573
. : milestone, 552,
iast_GLOBAL (504.155 µs) : 482, 526
. : milestone, 504,
iast_HARDCODED_SECRET_DISABLED (481.144 µs) : 460, 503
. : milestone, 481,
iast_INACTIVE (446.749 µs) : 426, 468
. : milestone, 447,
iast_TELEMETRY_OFF (471.68 µs) : 449, 495
. : milestone, 472,
tracing (435.335 µs) : 415, 455
. : milestone, 435,
section candidate
no_agent (423.11 µs) : 404, 443
. : milestone, 423,
iast (481.498 µs) : 460, 503
. : milestone, 481,
iast_FULL (551.963 µs) : 531, 573
. : milestone, 552,
iast_GLOBAL (507.231 µs) : 486, 529
. : milestone, 507,
iast_HARDCODED_SECRET_DISABLED (486.221 µs) : 464, 508
. : milestone, 486,
iast_INACTIVE (445.768 µs) : 425, 467
. : milestone, 446,
iast_TELEMETRY_OFF (472.849 µs) : 451, 495
. : milestone, 473,
tracing (438.979 µs) : 418, 460
. : milestone, 439,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.40.0-SNAPSHOT~9c17bb943d, baseline=1.40.0-SNAPSHOT~fb9b96a2fa
dateFormat X
axisFormat %s
section baseline
no_agent (1.337 ms) : 1318, 1356
. : milestone, 1337,
appsec (1.705 ms) : 1680, 1729
. : milestone, 1705,
appsec_no_iast (1.72 ms) : 1696, 1744
. : milestone, 1720,
iast (1.471 ms) : 1448, 1493
. : milestone, 1471,
profiling (1.494 ms) : 1470, 1519
. : milestone, 1494,
tracing (1.462 ms) : 1438, 1486
. : milestone, 1462,
section candidate
no_agent (1.356 ms) : 1336, 1376
. : milestone, 1356,
appsec (1.728 ms) : 1704, 1751
. : milestone, 1728,
appsec_no_iast (1.736 ms) : 1712, 1759
. : milestone, 1736,
iast (1.492 ms) : 1470, 1514
. : milestone, 1492,
profiling (1.527 ms) : 1502, 1551
. : milestone, 1527,
tracing (1.473 ms) : 1449, 1497
. : milestone, 1473,
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 tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.40.0-SNAPSHOT~9c17bb943d, baseline=1.40.0-SNAPSHOT~fb9b96a2fa
dateFormat X
axisFormat %s
section baseline
no_agent (1.454 ms) : 1443, 1466
. : milestone, 1454,
appsec (2.221 ms) : 2186, 2256
. : milestone, 2221,
iast (1.946 ms) : 1904, 1988
. : milestone, 1946,
iast_GLOBAL (2.016 ms) : 1972, 2060
. : milestone, 2016,
profiling (1.857 ms) : 1822, 1893
. : milestone, 1857,
tracing (1.831 ms) : 1798, 1864
. : milestone, 1831,
section candidate
no_agent (1.457 ms) : 1445, 1468
. : milestone, 1457,
appsec (2.208 ms) : 2173, 2243
. : milestone, 2208,
iast (1.968 ms) : 1925, 2010
. : milestone, 1968,
iast_GLOBAL (2.013 ms) : 1969, 2057
. : milestone, 2013,
profiling (1.847 ms) : 1812, 1881
. : milestone, 1847,
tracing (1.823 ms) : 1790, 1856
. : milestone, 1823,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.40.0-SNAPSHOT~9c17bb943d, baseline=1.40.0-SNAPSHOT~fb9b96a2fa
dateFormat X
axisFormat %s
section baseline
no_agent (14.953 s) : 14953000, 14953000
. : milestone, 14953000,
appsec (15.168 s) : 15168000, 15168000
. : milestone, 15168000,
iast (18.502 s) : 18502000, 18502000
. : milestone, 18502000,
iast_GLOBAL (17.985 s) : 17985000, 17985000
. : milestone, 17985000,
profiling (15.047 s) : 15047000, 15047000
. : milestone, 15047000,
tracing (15.169 s) : 15169000, 15169000
. : milestone, 15169000,
section candidate
no_agent (14.969 s) : 14969000, 14969000
. : milestone, 14969000,
appsec (14.965 s) : 14965000, 14965000
. : milestone, 14965000,
iast (19.029 s) : 19029000, 19029000
. : milestone, 19029000,
iast_GLOBAL (17.959 s) : 17959000, 17959000
. : milestone, 17959000,
profiling (15.316 s) : 15316000, 15316000
. : milestone, 15316000,
tracing (14.815 s) : 14815000, 14815000
. : milestone, 14815000,
|
ygree
force-pushed
the
vandonr/minimalstage
branch
from
September 6, 2024 21:32
9c17bb9
to
0ff2257
Compare
amarziali
reviewed
Sep 9, 2024
...-common/src/main/java/datadog/trace/instrumentation/graphqljava/InstrumentedDataFetcher.java
Show resolved
Hide resolved
ygree
force-pushed
the
vandonr/minimalstage
branch
from
September 9, 2024 16:09
0ff2257
to
72c6056
Compare
amarziali
approved these changes
Sep 9, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What Does This Do
solves #6284
Additional Notes
I'd like to have a test on the behavior that was fixed
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 issue