-
Notifications
You must be signed in to change notification settings - Fork 303
Use constructor as fallback for ExecutionRequest creation in JUnit5 instrumentation #8871
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
Merged
daniel-mohedano
merged 2 commits into
master
from
daniel.mohedano/junit5-execution-request-constructor
May 23, 2025
Merged
Use constructor as fallback for ExecutionRequest creation in JUnit5 instrumentation #8871
daniel-mohedano
merged 2 commits into
master
from
daniel.mohedano/junit5-execution-request-constructor
May 23, 2025
Conversation
This file contains hidden or 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
nikita-tkachenko-datadog
approved these changes
May 23, 2025
BenchmarksStartupParameters
See matching parameters
SummaryFound 3 performance improvements and 1 performance regressions! Performance is the same for 51 metrics, 16 unstable metrics.
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.50.0-SNAPSHOT~a30b565046, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.022 s) : 0, 1022068
Total [baseline] (10.487 s) : 0, 10486890
Agent [candidate] (1.023 s) : 0, 1023032
Total [candidate] (10.475 s) : 0, 10475427
section appsec
Agent [baseline] (1.168 s) : 0, 1167529
Total [baseline] (10.709 s) : 0, 10709480
Agent [candidate] (1.169 s) : 0, 1169384
Total [candidate] (10.732 s) : 0, 10731541
section iast
Agent [baseline] (1.15 s) : 0, 1150051
Total [baseline] (10.875 s) : 0, 10875182
Agent [candidate] (1.152 s) : 0, 1152427
Total [candidate] (10.931 s) : 0, 10931201
section profiling
Agent [baseline] (1.292 s) : 0, 1292116
Total [baseline] (11.005 s) : 0, 11004618
Agent [candidate] (1.275 s) : 0, 1274739
Total [candidate] (10.828 s) : 0, 10828183
gantt
title petclinic - break down per module: candidate=1.50.0-SNAPSHOT~a30b565046, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (683.929 ms) : 0, 683929
BytebuddyAgent [candidate] (683.541 ms) : 0, 683541
GlobalTracer [baseline] (240.208 ms) : 0, 240208
GlobalTracer [candidate] (241.105 ms) : 0, 241105
AppSec [baseline] (54.297 ms) : 0, 54297
AppSec [candidate] (54.572 ms) : 0, 54572
Debugger [baseline] (9.002 ms) : 0, 9002
Debugger [candidate] (9.661 ms) : 0, 9661
Remote Config [baseline] (682.986 µs) : 0, 683
Remote Config [candidate] (732.521 µs) : 0, 733
Telemetry [baseline] (10.371 ms) : 0, 10371
Telemetry [candidate] (9.849 ms) : 0, 9849
section appsec
BytebuddyAgent [baseline] (704.751 ms) : 0, 704751
BytebuddyAgent [candidate] (705.247 ms) : 0, 705247
GlobalTracer [baseline] (237.853 ms) : 0, 237853
GlobalTracer [candidate] (238.393 ms) : 0, 238393
IAST [baseline] (21.754 ms) : 0, 21754
IAST [candidate] (21.805 ms) : 0, 21805
AppSec [baseline] (176.022 ms) : 0, 176022
AppSec [candidate] (177.09 ms) : 0, 177090
Debugger [baseline] (6.316 ms) : 0, 6316
Debugger [candidate] (6.004 ms) : 0, 6004
Remote Config [baseline] (620.8 µs) : 0, 621
Remote Config [candidate] (626.764 µs) : 0, 627
Telemetry [baseline] (7.412 ms) : 0, 7412
Telemetry [candidate] (7.474 ms) : 0, 7474
section iast
BytebuddyAgent [baseline] (802.868 ms) : 0, 802868
BytebuddyAgent [candidate] (804.409 ms) : 0, 804409
GlobalTracer [baseline] (231.046 ms) : 0, 231046
GlobalTracer [candidate] (231.729 ms) : 0, 231729
IAST [baseline] (26.438 ms) : 0, 26438
IAST [candidate] (29.755 ms) : 0, 29755
AppSec [baseline] (49.36 ms) : 0, 49360
AppSec [candidate] (48.432 ms) : 0, 48432
Debugger [baseline] (5.89 ms) : 0, 5890
Debugger [candidate] (5.929 ms) : 0, 5929
Remote Config [baseline] (598.425 µs) : 0, 598
Remote Config [candidate] (704.753 µs) : 0, 705
Telemetry [baseline] (7.875 ms) : 0, 7875
Telemetry [candidate] (7.923 ms) : 0, 7923
section profiling
BytebuddyAgent [baseline] (678.981 ms) : 0, 678981
BytebuddyAgent [candidate] (679.178 ms) : 0, 679178
GlobalTracer [baseline] (381.101 ms) : 0, 381101
GlobalTracer [candidate] (362.469 ms) : 0, 362469
AppSec [baseline] (55.082 ms) : 0, 55082
AppSec [candidate] (62.442 ms) : 0, 62442
Debugger [baseline] (6.156 ms) : 0, 6156
Debugger [candidate] (6.248 ms) : 0, 6248
Remote Config [baseline] (672.006 µs) : 0, 672
Remote Config [candidate] (654.346 µs) : 0, 654
Telemetry [baseline] (8.182 ms) : 0, 8182
Telemetry [candidate] (8.255 ms) : 0, 8255
ProfilingAgent [baseline] (110.823 ms) : 0, 110823
ProfilingAgent [candidate] (104.112 ms) : 0, 104112
Profiling [baseline] (110.849 ms) : 0, 110849
Profiling [candidate] (104.136 ms) : 0, 104136
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.50.0-SNAPSHOT~a30b565046, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.033 s) : 0, 1032774
Total [baseline] (8.689 s) : 0, 8689224
Agent [candidate] (1.024 s) : 0, 1023761
Total [candidate] (8.65 s) : 0, 8650277
section iast
Agent [baseline] (1.154 s) : 0, 1153608
Total [baseline] (9.279 s) : 0, 9278810
Agent [candidate] (1.151 s) : 0, 1150849
Total [candidate] (9.237 s) : 0, 9237173
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.148 s) : 0, 1148381
Total [baseline] (9.215 s) : 0, 9214683
Agent [candidate] (1.151 s) : 0, 1151132
Total [candidate] (9.192 s) : 0, 9191697
section iast_TELEMETRY_OFF
Agent [baseline] (1.153 s) : 0, 1153429
Total [baseline] (9.227 s) : 0, 9227159
Agent [candidate] (1.146 s) : 0, 1146483
Total [candidate] (9.245 s) : 0, 9244507
gantt
title insecure-bank - break down per module: candidate=1.50.0-SNAPSHOT~a30b565046, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (692.477 ms) : 0, 692477
BytebuddyAgent [candidate] (684.56 ms) : 0, 684560
GlobalTracer [baseline] (241.647 ms) : 0, 241647
GlobalTracer [candidate] (241.469 ms) : 0, 241469
AppSec [baseline] (55.689 ms) : 0, 55689
AppSec [candidate] (56.048 ms) : 0, 56048
Debugger [baseline] (7.765 ms) : 0, 7765
Debugger [candidate] (8.371 ms) : 0, 8371
Remote Config [baseline] (700.792 µs) : 0, 701
Remote Config [candidate] (694.809 µs) : 0, 695
Telemetry [baseline] (10.719 ms) : 0, 10719
Telemetry [candidate] (9.033 ms) : 0, 9033
section iast
BytebuddyAgent [baseline] (804.448 ms) : 0, 804448
BytebuddyAgent [candidate] (803.107 ms) : 0, 803107
GlobalTracer [baseline] (232.057 ms) : 0, 232057
GlobalTracer [candidate] (231.355 ms) : 0, 231355
IAST [baseline] (27.792 ms) : 0, 27792
IAST [candidate] (28.263 ms) : 0, 28263
AppSec [baseline] (50.524 ms) : 0, 50524
AppSec [candidate] (50.196 ms) : 0, 50196
Debugger [baseline] (5.965 ms) : 0, 5965
Debugger [candidate] (5.908 ms) : 0, 5908
Remote Config [baseline] (597.827 µs) : 0, 598
Remote Config [candidate] (593.969 µs) : 0, 594
Telemetry [baseline] (7.906 ms) : 0, 7906
Telemetry [candidate] (7.914 ms) : 0, 7914
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (801.395 ms) : 0, 801395
BytebuddyAgent [candidate] (802.743 ms) : 0, 802743
GlobalTracer [baseline] (230.744 ms) : 0, 230744
GlobalTracer [candidate] (232.081 ms) : 0, 232081
IAST [baseline] (29.7 ms) : 0, 29700
IAST [candidate] (29.143 ms) : 0, 29143
AppSec [baseline] (48.656 ms) : 0, 48656
AppSec [candidate] (49.237 ms) : 0, 49237
Debugger [baseline] (5.927 ms) : 0, 5927
Debugger [candidate] (5.89 ms) : 0, 5890
Remote Config [baseline] (594.502 µs) : 0, 595
Remote Config [candidate] (595.851 µs) : 0, 596
Telemetry [baseline] (7.907 ms) : 0, 7907
Telemetry [candidate] (7.922 ms) : 0, 7922
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (805.147 ms) : 0, 805147
BytebuddyAgent [candidate] (799.137 ms) : 0, 799137
GlobalTracer [baseline] (231.833 ms) : 0, 231833
GlobalTracer [candidate] (230.904 ms) : 0, 230904
IAST [baseline] (23.321 ms) : 0, 23321
IAST [candidate] (23.152 ms) : 0, 23152
AppSec [baseline] (55.22 ms) : 0, 55220
AppSec [candidate] (55.373 ms) : 0, 55373
Debugger [baseline] (5.959 ms) : 0, 5959
Debugger [candidate] (5.969 ms) : 0, 5969
Remote Config [baseline] (624.206 µs) : 0, 624
Remote Config [candidate] (606.663 µs) : 0, 607
Telemetry [baseline] (7.746 ms) : 0, 7746
Telemetry [candidate] (7.849 ms) : 0, 7849
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 18 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.50.0-SNAPSHOT~a30b565046, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section baseline
no_agent (402.015 µs) : 364, 440
. : milestone, 402,
iast (516.439 µs) : 493, 540
. : milestone, 516,
iast_FULL (738.644 µs) : 717, 761
. : milestone, 739,
iast_GLOBAL (561.746 µs) : 540, 584
. : milestone, 562,
iast_HARDCODED_SECRET_DISABLED (518.353 µs) : 496, 540
. : milestone, 518,
iast_INACTIVE (466.52 µs) : 446, 488
. : milestone, 467,
iast_TELEMETRY_OFF (508.11 µs) : 485, 531
. : milestone, 508,
tracing (466.052 µs) : 444, 488
. : milestone, 466,
section candidate
no_agent (385.836 µs) : 366, 406
. : milestone, 386,
iast (535.452 µs) : 514, 557
. : milestone, 535,
iast_FULL (738.357 µs) : 716, 760
. : milestone, 738,
iast_GLOBAL (580.202 µs) : 557, 604
. : milestone, 580,
iast_HARDCODED_SECRET_DISABLED (531.008 µs) : 508, 554
. : milestone, 531,
iast_INACTIVE (476.494 µs) : 454, 499
. : milestone, 476,
iast_TELEMETRY_OFF (510.533 µs) : 487, 534
. : milestone, 511,
tracing (467.914 µs) : 445, 491
. : milestone, 468,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.50.0-SNAPSHOT~a30b565046, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section baseline
no_agent (1.364 ms) : 1344, 1384
. : milestone, 1364,
appsec (1.721 ms) : 1697, 1745
. : milestone, 1721,
appsec_no_iast (1.739 ms) : 1715, 1763
. : milestone, 1739,
code_origins (1.654 ms) : 1626, 1682
. : milestone, 1654,
iast (1.517 ms) : 1493, 1542
. : milestone, 1517,
profiling (1.506 ms) : 1483, 1530
. : milestone, 1506,
tracing (1.49 ms) : 1465, 1514
. : milestone, 1490,
section candidate
no_agent (1.354 ms) : 1335, 1373
. : milestone, 1354,
appsec (1.742 ms) : 1718, 1765
. : milestone, 1742,
appsec_no_iast (1.736 ms) : 1713, 1759
. : milestone, 1736,
code_origins (1.656 ms) : 1628, 1683
. : milestone, 1656,
iast (1.51 ms) : 1486, 1534
. : milestone, 1510,
profiling (1.504 ms) : 1480, 1527
. : milestone, 1504,
tracing (1.5 ms) : 1474, 1525
. : milestone, 1500,
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.50.0-SNAPSHOT~a30b565046, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section baseline
no_agent (14.915 s) : 14915000, 14915000
. : milestone, 14915000,
appsec (14.993 s) : 14993000, 14993000
. : milestone, 14993000,
iast (18.618 s) : 18618000, 18618000
. : milestone, 18618000,
iast_GLOBAL (17.977 s) : 17977000, 17977000
. : milestone, 17977000,
profiling (15.022 s) : 15022000, 15022000
. : milestone, 15022000,
tracing (14.797 s) : 14797000, 14797000
. : milestone, 14797000,
section candidate
no_agent (14.692 s) : 14692000, 14692000
. : milestone, 14692000,
appsec (14.997 s) : 14997000, 14997000
. : milestone, 14997000,
iast (18.237 s) : 18237000, 18237000
. : milestone, 18237000,
iast_GLOBAL (17.919 s) : 17919000, 17919000
. : milestone, 17919000,
profiling (15.025 s) : 15025000, 15025000
. : milestone, 15025000,
tracing (14.852 s) : 14852000, 14852000
. : milestone, 14852000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.50.0-SNAPSHOT~a30b565046, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section baseline
no_agent (1.473 ms) : 1462, 1485
. : milestone, 1473,
appsec (2.398 ms) : 2349, 2446
. : milestone, 2398,
iast (2.174 ms) : 2114, 2235
. : milestone, 2174,
iast_GLOBAL (2.218 ms) : 2156, 2279
. : milestone, 2218,
profiling (2.035 ms) : 1985, 2084
. : milestone, 2035,
tracing (2.018 ms) : 1971, 2066
. : milestone, 2018,
section candidate
no_agent (1.48 ms) : 1468, 1491
. : milestone, 1480,
appsec (2.391 ms) : 2343, 2440
. : milestone, 2391,
iast (2.175 ms) : 2114, 2237
. : milestone, 2175,
iast_GLOBAL (2.213 ms) : 2152, 2274
. : milestone, 2213,
profiling (2.032 ms) : 1982, 2082
. : milestone, 2032,
tracing (2.015 ms) : 1967, 2063
. : milestone, 2015,
|
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
org.junit.platform.engine.ExecutionRequest
fromcreate
to the constructor, as it was originally implemented in the instrumentation.Motivation
Fixes a bug introduced in #8865. It would only fail on the earliest version of
org.junit.platform:junit-platform-engine
, in which thecreate
method is not available.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: [PROJ-IDENT]