Remove remaining use of `JAVA_X_HOME` #10340

bric3 · 2026-01-12T21:25:51Z

What Does This Do

This PR allows to completely get rid of setting up JAVA_X_HOME on local dev environment.

The stable special word has special handling though, that is using Gradle internal toolchain registry, and fallback to env vars. However, this mainly affect CI.

Motivation

Easier project setup

Additional Notes

After this PR, setting up JAVA_X_HOME won't be needed anymore. One can use -PtestJvm=..., e.g. -PtestJvm=11, -PtestJvm=graalvm17, ...

pr-commenter · 2026-01-12T22:11:35Z

Benchmarks

Startup

Parameters

	Baseline	Candidate
baseline_or_candidate	baseline	candidate
git_branch	master	bdu/remove-remaining-use-of-JAVA_X_HOME
git_commit_date	1768337169	1768481313
git_commit_sha	`671cc8c`	`e55a0a4`
release_version	1.59.0-SNAPSHOT~671cc8cdbf	1.59.0-SNAPSHOT~e55a0a4bc5

See matching parameters

	Baseline	Candidate
application	insecure-bank	insecure-bank
ci_job_date	1768483759	1768483759
ci_job_id	1352569319	1352569319
ci_pipeline_id	90864383	90864383
cpu_model	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz
kernel_version	Linux runner-zfyrx7zua-project-304-concurrent-0-h1m8ow7q 6.8.0-1031-aws #33~22.04.1-Ubuntu SMP Thu Jun 26 14:22:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux	Linux runner-zfyrx7zua-project-304-concurrent-0-h1m8ow7q 6.8.0-1031-aws #33~22.04.1-Ubuntu SMP Thu Jun 26 14:22:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
module	Agent	Agent
parent	None	None

Summary

Found 0 performance improvements and 0 performance regressions! Performance is the same for 56 metrics, 9 unstable metrics.

Startup time reports for insecure-bank

gantt
    title insecure-bank - global startup overhead: candidate=1.59.0-SNAPSHOT~e55a0a4bc5, baseline=1.59.0-SNAPSHOT~671cc8cdbf

    dateFormat X
    axisFormat %s
section tracing
Agent [baseline] (1.084 s) : 0, 1083739
Total [baseline] (8.75 s) : 0, 8750130
Agent [candidate] (1.088 s) : 0, 1088127
Total [candidate] (8.783 s) : 0, 8783091
section iast
Agent [baseline] (1.219 s) : 0, 1219263
Total [baseline] (9.325 s) : 0, 9324805
Agent [candidate] (1.233 s) : 0, 1233069
Total [candidate] (9.367 s) : 0, 9366888

baseline results

Module	Variant	Duration	Δ tracing
Agent	tracing	1.084 s	-
Agent	iast	1.219 s	135.524 ms (12.5%)
Total	tracing	8.75 s	-
Total	iast	9.325 s	574.675 ms (6.6%)

candidate results

Module	Variant	Duration	Δ tracing
Agent	tracing	1.088 s	-
Agent	iast	1.233 s	144.942 ms (13.3%)
Total	tracing	8.783 s	-
Total	iast	9.367 s	583.797 ms (6.6%)

gantt
    title insecure-bank - break down per module: candidate=1.59.0-SNAPSHOT~e55a0a4bc5, baseline=1.59.0-SNAPSHOT~671cc8cdbf

    dateFormat X
    axisFormat %s
section tracing
crashtracking [baseline] (1.186 ms) : 0, 1186
crashtracking [candidate] (1.177 ms) : 0, 1177
BytebuddyAgent [baseline] (651.612 ms) : 0, 651612
BytebuddyAgent [candidate] (653.402 ms) : 0, 653402
GlobalTracer [baseline] (282.346 ms) : 0, 282346
GlobalTracer [candidate] (284.505 ms) : 0, 284505
AppSec [baseline] (32.666 ms) : 0, 32666
AppSec [candidate] (32.814 ms) : 0, 32814
Debugger [baseline] (67.182 ms) : 0, 67182
Debugger [candidate] (67.212 ms) : 0, 67212
Remote Config [baseline] (617.149 µs) : 0, 617
Remote Config [candidate] (635.57 µs) : 0, 636
Telemetry [baseline] (8.906 ms) : 0, 8906
Telemetry [candidate] (9.051 ms) : 0, 9051
Flare Poller [baseline] (3.768 ms) : 0, 3768
Flare Poller [candidate] (3.86 ms) : 0, 3860
section iast
crashtracking [baseline] (1.179 ms) : 0, 1179
crashtracking [candidate] (1.189 ms) : 0, 1189
BytebuddyAgent [baseline] (788.801 ms) : 0, 788801
BytebuddyAgent [candidate] (797.345 ms) : 0, 797345
GlobalTracer [baseline] (256.195 ms) : 0, 256195
GlobalTracer [candidate] (258.569 ms) : 0, 258569
IAST [baseline] (26.843 ms) : 0, 26843
IAST [candidate] (27.206 ms) : 0, 27206
AppSec [baseline] (34.079 ms) : 0, 34079
AppSec [candidate] (32.984 ms) : 0, 32984
Debugger [baseline] (64.532 ms) : 0, 64532
Debugger [candidate] (67.427 ms) : 0, 67427
Remote Config [baseline] (528.456 µs) : 0, 528
Remote Config [candidate] (594.684 µs) : 0, 595
Telemetry [baseline] (8.344 ms) : 0, 8344
Telemetry [candidate] (8.552 ms) : 0, 8552
Flare Poller [baseline] (3.542 ms) : 0, 3542
Flare Poller [candidate] (3.642 ms) : 0, 3642

Startup time reports for petclinic

gantt
    title petclinic - global startup overhead: candidate=1.59.0-SNAPSHOT~e55a0a4bc5, baseline=1.59.0-SNAPSHOT~671cc8cdbf

    dateFormat X
    axisFormat %s
section tracing
Agent [baseline] (1.084 s) : 0, 1083861
Total [baseline] (10.758 s) : 0, 10757551
Agent [candidate] (1.086 s) : 0, 1086088
Total [candidate] (10.866 s) : 0, 10866215
section appsec
Agent [baseline] (1.274 s) : 0, 1274143
Total [baseline] (11.064 s) : 0, 11063978
Agent [candidate] (1.271 s) : 0, 1270805
Total [candidate] (11.055 s) : 0, 11055371
section iast
Agent [baseline] (1.24 s) : 0, 1239852
Total [baseline] (11.248 s) : 0, 11248229
Agent [candidate] (1.225 s) : 0, 1224701
Total [candidate] (11.189 s) : 0, 11188674
section profiling
Agent [baseline] (1.206 s) : 0, 1206280
Total [baseline] (10.946 s) : 0, 10945820
Agent [candidate] (1.215 s) : 0, 1214879
Total [candidate] (10.901 s) : 0, 10900864

baseline results

Module	Variant	Duration	Δ tracing
Agent	tracing	1.084 s	-
Agent	appsec	1.274 s	190.282 ms (17.6%)
Agent	iast	1.24 s	155.991 ms (14.4%)
Agent	profiling	1.206 s	122.419 ms (11.3%)
Total	tracing	10.758 s	-
Total	appsec	11.064 s	306.427 ms (2.8%)
Total	iast	11.248 s	490.678 ms (4.6%)
Total	profiling	10.946 s	188.269 ms (1.8%)

candidate results

Module	Variant	Duration	Δ tracing
Agent	tracing	1.086 s	-
Agent	appsec	1.271 s	184.718 ms (17.0%)
Agent	iast	1.225 s	138.613 ms (12.8%)
Agent	profiling	1.215 s	128.791 ms (11.9%)
Total	tracing	10.866 s	-
Total	appsec	11.055 s	189.156 ms (1.7%)
Total	iast	11.189 s	322.459 ms (3.0%)
Total	profiling	10.901 s	34.649 ms (0.3%)

gantt
    title petclinic - break down per module: candidate=1.59.0-SNAPSHOT~e55a0a4bc5, baseline=1.59.0-SNAPSHOT~671cc8cdbf

    dateFormat X
    axisFormat %s
section tracing
crashtracking [baseline] (1.172 ms) : 0, 1172
crashtracking [candidate] (1.195 ms) : 0, 1195
BytebuddyAgent [baseline] (650.676 ms) : 0, 650676
BytebuddyAgent [candidate] (651.704 ms) : 0, 651704
GlobalTracer [baseline] (282.475 ms) : 0, 282475
GlobalTracer [candidate] (283.073 ms) : 0, 283073
AppSec [baseline] (32.72 ms) : 0, 32720
AppSec [candidate] (32.764 ms) : 0, 32764
Debugger [baseline] (68.1 ms) : 0, 68100
Debugger [candidate] (68.473 ms) : 0, 68473
Remote Config [baseline] (642.237 µs) : 0, 642
Remote Config [candidate] (636.683 µs) : 0, 637
Telemetry [baseline] (8.937 ms) : 0, 8937
Telemetry [candidate] (8.982 ms) : 0, 8982
Flare Poller [baseline] (3.777 ms) : 0, 3777
Flare Poller [candidate] (3.792 ms) : 0, 3792
section appsec
crashtracking [baseline] (1.189 ms) : 0, 1189
crashtracking [candidate] (1.187 ms) : 0, 1187
BytebuddyAgent [baseline] (696.321 ms) : 0, 696321
BytebuddyAgent [candidate] (694.594 ms) : 0, 694594
GlobalTracer [baseline] (260.072 ms) : 0, 260072
GlobalTracer [candidate] (259.596 ms) : 0, 259596
IAST [baseline] (24.906 ms) : 0, 24906
IAST [candidate] (24.85 ms) : 0, 24850
AppSec [baseline] (173.474 ms) : 0, 173474
AppSec [candidate] (172.494 ms) : 0, 172494
Debugger [baseline] (68.385 ms) : 0, 68385
Debugger [candidate] (68.548 ms) : 0, 68548
Remote Config [baseline] (780.274 µs) : 0, 780
Remote Config [candidate] (753.824 µs) : 0, 754
Telemetry [baseline] (9.553 ms) : 0, 9553
Telemetry [candidate] (9.418 ms) : 0, 9418
Flare Poller [baseline] (3.885 ms) : 0, 3885
Flare Poller [candidate] (3.83 ms) : 0, 3830
section iast
crashtracking [baseline] (1.2 ms) : 0, 1200
crashtracking [candidate] (1.175 ms) : 0, 1175
BytebuddyAgent [baseline] (803.154 ms) : 0, 803154
BytebuddyAgent [candidate] (791.301 ms) : 0, 791301
GlobalTracer [baseline] (259.137 ms) : 0, 259137
GlobalTracer [candidate] (256.459 ms) : 0, 256459
IAST [baseline] (27.322 ms) : 0, 27322
IAST [candidate] (26.951 ms) : 0, 26951
AppSec [baseline] (34.687 ms) : 0, 34687
AppSec [candidate] (34.54 ms) : 0, 34540
Debugger [baseline] (65.993 ms) : 0, 65993
Debugger [candidate] (66.129 ms) : 0, 66129
Remote Config [baseline] (583.808 µs) : 0, 584
Remote Config [candidate] (593.399 µs) : 0, 593
Telemetry [baseline] (8.6 ms) : 0, 8600
Telemetry [candidate] (8.552 ms) : 0, 8552
Flare Poller [baseline] (3.552 ms) : 0, 3552
Flare Poller [candidate] (3.594 ms) : 0, 3594
section profiling
crashtracking [baseline] (1.217 ms) : 0, 1217
crashtracking [candidate] (1.225 ms) : 0, 1225
BytebuddyAgent [baseline] (703.857 ms) : 0, 703857
BytebuddyAgent [candidate] (708.776 ms) : 0, 708776
GlobalTracer [baseline] (221.18 ms) : 0, 221180
GlobalTracer [candidate] (223.078 ms) : 0, 223078
AppSec [baseline] (32.195 ms) : 0, 32195
AppSec [candidate] (32.537 ms) : 0, 32537
Debugger [baseline] (67.888 ms) : 0, 67888
Debugger [candidate] (68.883 ms) : 0, 68883
Remote Config [baseline] (690.38 µs) : 0, 690
Remote Config [candidate] (675.434 µs) : 0, 675
Telemetry [baseline] (8.945 ms) : 0, 8945
Telemetry [candidate] (8.918 ms) : 0, 8918
Flare Poller [baseline] (3.757 ms) : 0, 3757
Flare Poller [candidate] (3.77 ms) : 0, 3770
ProfilingAgent [baseline] (96.716 ms) : 0, 96716
ProfilingAgent [candidate] (96.8 ms) : 0, 96800
Profiling [baseline] (97.295 ms) : 0, 97295
Profiling [candidate] (97.378 ms) : 0, 97378

Load

Parameters

	Baseline	Candidate
baseline_or_candidate	baseline	candidate
git_branch	master	bdu/remove-remaining-use-of-JAVA_X_HOME
git_commit_date	1768337169	1768481313
git_commit_sha	`671cc8c`	`e55a0a4`
release_version	1.59.0-SNAPSHOT~671cc8cdbf	1.59.0-SNAPSHOT~e55a0a4bc5

See matching parameters

	Baseline	Candidate
application	insecure-bank	insecure-bank
ci_job_date	1768484256	1768484256
ci_job_id	1352569320	1352569320
ci_pipeline_id	90864383	90864383
cpu_model	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz
kernel_version	Linux runner-zfyrx7zua-project-304-concurrent-1-066ekmz4 6.8.0-1031-aws #33~22.04.1-Ubuntu SMP Thu Jun 26 14:22:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux	Linux runner-zfyrx7zua-project-304-concurrent-1-066ekmz4 6.8.0-1031-aws #33~22.04.1-Ubuntu SMP Thu Jun 26 14:22:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux

Summary

Found 0 performance improvements and 1 performance regressions! Performance is the same for 16 metrics, 19 unstable metrics.

scenario	Δ mean agg_http_req_duration_p50	Δ mean agg_http_req_duration_p95	Δ mean throughput	candidate mean agg_http_req_duration_p50	candidate mean agg_http_req_duration_p95	candidate mean throughput	baseline mean agg_http_req_duration_p50	baseline mean agg_http_req_duration_p95	baseline mean throughput
scenario:load:insecure-bank:iast:high_load	worse [+122.661µs; +259.654µs] or [+4.902%; +10.378%]	unstable [-146.487µs; +787.209µs] or [-1.955%; +10.505%]	unstable [-231.951op/s; +52.326op/s] or [-16.220%; +3.659%]	2.693ms	7.814ms	1340.250op/s	2.502ms	7.494ms	1430.062op/s

Request duration reports for petclinic

gantt
    title petclinic - request duration [CI 0.99] : candidate=1.59.0-SNAPSHOT~e55a0a4bc5, baseline=1.59.0-SNAPSHOT~671cc8cdbf
    dateFormat X
    axisFormat %s
section baseline
no_agent (18.252 ms) : 18060, 18444
.   : milestone, 18252,
appsec (18.5 ms) : 18312, 18687
.   : milestone, 18500,
code_origins (18.174 ms) : 17993, 18355
.   : milestone, 18174,
iast (17.868 ms) : 17691, 18045
.   : milestone, 17868,
profiling (18.875 ms) : 18681, 19070
.   : milestone, 18875,
tracing (17.899 ms) : 17723, 18076
.   : milestone, 17899,
section candidate
no_agent (17.243 ms) : 17068, 17418
.   : milestone, 17243,
appsec (18.806 ms) : 18612, 19000
.   : milestone, 18806,
code_origins (17.649 ms) : 17473, 17825
.   : milestone, 17649,
iast (17.683 ms) : 17508, 17858
.   : milestone, 17683,
profiling (18.546 ms) : 18361, 18731
.   : milestone, 18546,
tracing (17.486 ms) : 17316, 17656
.   : milestone, 17486,

baseline results

Variant	Request duration [CI 0.99]	Δ no_agent
no_agent	18.252 ms [18.06 ms, 18.444 ms]	-
appsec	18.5 ms [18.312 ms, 18.687 ms]	247.595 µs (1.4%)
code_origins	18.174 ms [17.993 ms, 18.355 ms]	-77.824 µs (-0.4%)
iast	17.868 ms [17.691 ms, 18.045 ms]	-384.261 µs (-2.1%)
profiling	18.875 ms [18.681 ms, 19.07 ms]	623.376 µs (3.4%)
tracing	17.899 ms [17.723 ms, 18.076 ms]	-352.531 µs (-1.9%)

candidate results

Variant	Request duration [CI 0.99]	Δ no_agent
no_agent	17.243 ms [17.068 ms, 17.418 ms]	-
appsec	18.806 ms [18.612 ms, 19.0 ms]	1.563 ms (9.1%)
code_origins	17.649 ms [17.473 ms, 17.825 ms]	405.934 µs (2.4%)
iast	17.683 ms [17.508 ms, 17.858 ms]	440.088 µs (2.6%)
profiling	18.546 ms [18.361 ms, 18.731 ms]	1.303 ms (7.6%)
tracing	17.486 ms [17.316 ms, 17.656 ms]	243.472 µs (1.4%)

Request duration reports for insecure-bank

gantt
    title insecure-bank - request duration [CI 0.99] : candidate=1.59.0-SNAPSHOT~e55a0a4bc5, baseline=1.59.0-SNAPSHOT~671cc8cdbf
    dateFormat X
    axisFormat %s
section baseline
no_agent (1.186 ms) : 1175, 1198
.   : milestone, 1186,
iast (3.198 ms) : 3159, 3238
.   : milestone, 3198,
iast_FULL (5.71 ms) : 5654, 5767
.   : milestone, 5710,
iast_GLOBAL (3.592 ms) : 3539, 3645
.   : milestone, 3592,
profiling (2.048 ms) : 2030, 2065
.   : milestone, 2048,
tracing (1.804 ms) : 1789, 1819
.   : milestone, 1804,
section candidate
no_agent (1.196 ms) : 1184, 1208
.   : milestone, 1196,
iast (3.418 ms) : 3375, 3460
.   : milestone, 3418,
iast_FULL (5.765 ms) : 5708, 5822
.   : milestone, 5765,
iast_GLOBAL (3.621 ms) : 3568, 3673
.   : milestone, 3621,
profiling (1.986 ms) : 1969, 2002
.   : milestone, 1986,
tracing (1.802 ms) : 1787, 1817
.   : milestone, 1802,

baseline results

Variant	Request duration [CI 0.99]	Δ no_agent
no_agent	1.186 ms [1.175 ms, 1.198 ms]	-
iast	3.198 ms [3.159 ms, 3.238 ms]	2.012 ms (169.6%)
iast_FULL	5.71 ms [5.654 ms, 5.767 ms]	4.524 ms (381.4%)
iast_GLOBAL	3.592 ms [3.539 ms, 3.645 ms]	2.406 ms (202.8%)
profiling	2.048 ms [2.03 ms, 2.065 ms]	861.519 µs (72.6%)
tracing	1.804 ms [1.789 ms, 1.819 ms]	617.812 µs (52.1%)

candidate results

Variant	Request duration [CI 0.99]	Δ no_agent
no_agent	1.196 ms [1.184 ms, 1.208 ms]	-
iast	3.418 ms [3.375 ms, 3.46 ms]	2.222 ms (185.8%)
iast_FULL	5.765 ms [5.708 ms, 5.822 ms]	4.569 ms (382.0%)
iast_GLOBAL	3.621 ms [3.568 ms, 3.673 ms]	2.425 ms (202.7%)
profiling	1.986 ms [1.969 ms, 2.002 ms]	789.668 µs (66.0%)
tracing	1.802 ms [1.787 ms, 1.817 ms]	606.142 µs (50.7%)

Dacapo

Parameters

	Baseline	Candidate
baseline_or_candidate	baseline	candidate
git_branch	master	bdu/remove-remaining-use-of-JAVA_X_HOME
git_commit_date	1768337169	1768481313
git_commit_sha	`671cc8c`	`e55a0a4`
release_version	1.59.0-SNAPSHOT~671cc8cdbf	1.59.0-SNAPSHOT~e55a0a4bc5

See matching parameters

	Baseline	Candidate
application	biojava	biojava
ci_job_date	1768483779	1768483779
ci_job_id	1352569321	1352569321
ci_pipeline_id	90864383	90864383
cpu_model	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz	Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz
kernel_version	Linux runner-zfyrx7zua-project-304-concurrent-2-9u7zqfej 6.8.0-1031-aws #33~22.04.1-Ubuntu SMP Thu Jun 26 14:22:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux	Linux runner-zfyrx7zua-project-304-concurrent-2-9u7zqfej 6.8.0-1031-aws #33~22.04.1-Ubuntu SMP Thu Jun 26 14:22:30 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux

Summary

Found 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 1 unstable metrics.

Execution time for biojava

gantt
    title biojava - execution time [CI 0.99] : candidate=1.59.0-SNAPSHOT~e55a0a4bc5, baseline=1.59.0-SNAPSHOT~671cc8cdbf
    dateFormat X
    axisFormat %s
section baseline
no_agent (14.985 s) : 14985000, 14985000
.   : milestone, 14985000,
appsec (14.669 s) : 14669000, 14669000
.   : milestone, 14669000,
iast (18.153 s) : 18153000, 18153000
.   : milestone, 18153000,
iast_GLOBAL (17.905 s) : 17905000, 17905000
.   : milestone, 17905000,
profiling (15.627 s) : 15627000, 15627000
.   : milestone, 15627000,
tracing (14.591 s) : 14591000, 14591000
.   : milestone, 14591000,
section candidate
no_agent (15.149 s) : 15149000, 15149000
.   : milestone, 15149000,
appsec (14.457 s) : 14457000, 14457000
.   : milestone, 14457000,
iast (18.207 s) : 18207000, 18207000
.   : milestone, 18207000,
iast_GLOBAL (17.749 s) : 17749000, 17749000
.   : milestone, 17749000,
profiling (14.891 s) : 14891000, 14891000
.   : milestone, 14891000,
tracing (14.826 s) : 14826000, 14826000
.   : milestone, 14826000,

baseline results

Variant	Execution Time [CI 0.99]	Δ no_agent
no_agent	14.985 s [14.985 s, 14.985 s]	-
appsec	14.669 s [14.669 s, 14.669 s]	-316.0 ms (-2.1%)
iast	18.153 s [18.153 s, 18.153 s]	3.168 s (21.1%)
iast_GLOBAL	17.905 s [17.905 s, 17.905 s]	2.92 s (19.5%)
profiling	15.627 s [15.627 s, 15.627 s]	642.0 ms (4.3%)
tracing	14.591 s [14.591 s, 14.591 s]	-394.0 ms (-2.6%)

candidate results

Variant	Execution Time [CI 0.99]	Δ no_agent
no_agent	15.149 s [15.149 s, 15.149 s]	-
appsec	14.457 s [14.457 s, 14.457 s]	-692.0 ms (-4.6%)
iast	18.207 s [18.207 s, 18.207 s]	3.058 s (20.2%)
iast_GLOBAL	17.749 s [17.749 s, 17.749 s]	2.6 s (17.2%)
profiling	14.891 s [14.891 s, 14.891 s]	-258.0 ms (-1.7%)
tracing	14.826 s [14.826 s, 14.826 s]	-323.0 ms (-2.1%)

Execution time for tomcat

gantt
    title tomcat - execution time [CI 0.99] : candidate=1.59.0-SNAPSHOT~e55a0a4bc5, baseline=1.59.0-SNAPSHOT~671cc8cdbf
    dateFormat X
    axisFormat %s
section baseline
no_agent (1.47 ms) : 1459, 1481
.   : milestone, 1470,
appsec (3.646 ms) : 3430, 3863
.   : milestone, 3646,
iast (2.208 ms) : 2143, 2273
.   : milestone, 2208,
iast_GLOBAL (2.253 ms) : 2188, 2319
.   : milestone, 2253,
profiling (2.082 ms) : 2028, 2136
.   : milestone, 2082,
tracing (2.037 ms) : 1985, 2088
.   : milestone, 2037,
section candidate
no_agent (1.466 ms) : 1455, 1478
.   : milestone, 1466,
appsec (3.7 ms) : 3480, 3919
.   : milestone, 3700,
iast (2.199 ms) : 2135, 2264
.   : milestone, 2199,
iast_GLOBAL (2.24 ms) : 2175, 2306
.   : milestone, 2240,
profiling (2.078 ms) : 2024, 2131
.   : milestone, 2078,
tracing (2.036 ms) : 1985, 2088
.   : milestone, 2036,

baseline results

Variant	Execution Time [CI 0.99]	Δ no_agent
no_agent	1.47 ms [1.459 ms, 1.481 ms]	-
appsec	3.646 ms [3.43 ms, 3.863 ms]	2.176 ms (148.0%)
iast	2.208 ms [2.143 ms, 2.273 ms]	737.582 µs (50.2%)
iast_GLOBAL	2.253 ms [2.188 ms, 2.319 ms]	783.489 µs (53.3%)
profiling	2.082 ms [2.028 ms, 2.136 ms]	612.291 µs (41.7%)
tracing	2.037 ms [1.985 ms, 2.088 ms]	566.824 µs (38.6%)

candidate results

Variant	Execution Time [CI 0.99]	Δ no_agent
no_agent	1.466 ms [1.455 ms, 1.478 ms]	-
appsec	3.7 ms [3.48 ms, 3.919 ms]	2.234 ms (152.4%)
iast	2.199 ms [2.135 ms, 2.264 ms]	733.208 µs (50.0%)
iast_GLOBAL	2.24 ms [2.175 ms, 2.306 ms]	774.301 µs (52.8%)
profiling	2.078 ms [2.024 ms, 2.131 ms]	611.587 µs (41.7%)
tracing	2.036 ms [1.985 ms, 2.088 ms]	570.25 µs (38.9%)

…A_*_HOME env vars

…lib/jvm/8/jre)

BUILDING.md

dd-smoke-tests/spring-boot-2.7-webflux/build.gradle

docs/how_instrumentations_work.md

sarahchen6

Wow this is cool! Left a few small wording nits, but otherwise looks good!

I was wondering -- how were you able to test this? My guess is by removing local Java installations and confirming that non-JDK 21 tests can still run.

AlexeyKuznetsov-DD · 2026-01-14T23:59:15Z

buildSrc/src/main/kotlin/datadog/gradle/plugin/testJvmConstraints/TestJvmSpec.kt

    }

-    // "stable" is calculated as the largest X found in JAVA_X_HOME
+    // "stable" is calculated as the largest Java version found via toolchains or JAVA_X_HOME


misc: I still do not like stable name... can we rename it to latest or something similar?

I'd rather not change it in this PR.

AlexeyKuznetsov-DD

LGTM. Just have a question - would it be possible to have JVMs from various vendors to test for example Oracle 8 and Zulu on my laptop?

PerfectSlayer

Erf, I just notice my review was left in draft. Submitting now

PerfectSlayer · 2026-01-14T17:47:47Z

docs/how_instrumentations_work.md

 ```

-Running tests that require JDK-21 will require the `JAVA_21_HOME` env var set and can be done like this:
+Running tests that require JDK-21 to be installed, use the `-PtestJvm=21` flag, for example:


🎯 suggestion: ‏How will it work to run with a specific JVM?
For JDK version based criteria, we can set the version number. But what happens for the case where you want to run against a JDK installed in a specific path?

The toolchain will be automatically downloaded with a number or (zulu11) e.g. :

// -------------> 1% CONFIGURING [10s] // > :dd-java-agent > :dd-java-agent:agent-iast > Downloading toolchain from URI https://api.foojay.io/disco/v3.0/ids/dc9716d3e8f8baffc294fb182ceca2c2/redirect > https://api.foojay.io/disco/v3.0/ids/dc9716d3e8f8baffc294fb182ceca2c2/redirect > 80 MiB/104.5 MiB downloaded

That said provisioning won't work oracle as it cannot be provisonned automatically, so it has to be installed manually in a location that Gradle know about.

About JVM paths, this was already handled with the dd-trace-java.test-jvm-contraints plugin, so passing -PtestJvm=/path/to/fancy/jvm/ will use that jvm for tests.

dd-trace-java/buildSrc/src/main/kotlin/datadog/gradle/plugin/testJvmConstraints/TestJvmSpec.kt

Lines 159 to 161 in f13392e

private val Project.javaToolchains: JavaToolchainService

get() =

extensions.getByName("javaToolchains") as JavaToolchainService

PerfectSlayer · 2026-01-14T17:51:01Z

CONTRIBUTING.md

 For IntelliJ IDEA, we suggest the following settings and plugin.

-The default JVM to build and run tests from the command line should be Java 8.
+The default JVM to build and run tests from the command line should be Java 21.


❔ question: ‏Is this still needed? Won't it be picked automatically?

Your comment made me realize the gradle-daemon-jvm.properties was missing something for this to work. Next commit fixes that.

I'm removing this line as it shouldn't be a concern anymore.

PerfectSlayer · 2026-01-14T17:55:30Z

BUILDING.md

 #### macOS

+
+Use your JDK manager ([mise](https://mise.jdx.dev/), [sdkman](https://sdkman.io/), etc.) or set-up the required JDKs with `brew`


💭 thought: ‏This is confusing as the next section gives explanations for brew. We choose to explain the brew way as it is the simplest as already available tools for most MacOS users not accustomed to Java development. For the others, they already know how to install a JDK and use their preferred tool.

BUILDING.md

PerfectSlayer · 2026-01-15T10:00:01Z

setup.sh

-check-jvm "JAVA_21_HOME" "21"
-check-jvm "JAVA_25_HOME" "25"
-check-jvm "JAVA_GRAALVM17_HOME" "17"
+echo "ℹ️ Other JDK versions will be automatically downloaded by Gradle toolchain resolver."


Looks like some part of my review were lost at some point but I had a comment about the onboarded script that was lost.
Here is a follow up PR to address what I originally posted: #10384
We can proceed with yours first and I will update / resolve conflict on mine after.

Actually, is this even necessary as JAVA_HOME is not strictly required anymore ?

bric3 · 2026-01-15T10:36:00Z

gradle/java_no_deps.gradle

  return launcher
 } as Closure<Provider<JavaLauncher>>

+ext.getLazyJavaHomeFor = (int javaVersionInteger, boolean hasNativeImage = false) -> {


note: Eventually this should be moved to the test-jvm-contraints plugin.

bric3 · 2026-01-15T12:34:23Z

LGTM. Just have a question - would it be possible to have JVMs from various vendors to test for example Oracle 8 and Zulu on my laptop?

Just use -PtestJvm=zulu11. However for oracle you need to install it yourself, since no tool can provision them. You can also pass a path to testJvm property. I amended the doc.
BTW this was possible before this PR.

…targets

URLs are acquired from the foojay resolver, when running this task ./gradlew updateDaemonJvm --jvm-version=21 See https://docs.gradle.org/8.14.3/userguide/gradle_daemon.html#configuring_the_daemon_jvm

…eing not enough

bric3 added type: enhancement Enhancements and improvements tag: no release notes Changes to exclude from release notes comp: tooling Build & Tooling labels Jan 12, 2026

bric3 temporarily deployed to ci-build January 12, 2026 21:25 — with GitHub Actions Inactive

bric3 force-pushed the bdu/remove-remaining-use-of-JAVA_X_HOME branch from a1e732b to e6e93dd Compare January 13, 2026 15:57

bric3 added 7 commits January 13, 2026 21:50

fix: Remove remaining use of JAVA_X_HOME in build files

34aa680

fix: Remove remaining use of JAVA_X_HOME in test

10a114f

fix: Remove remaining use of JAVA_X_HOME in native build tests

331caaf

fix: Move similar JAVA_HOME usage to our lazy toolchain util

f27e5c1

fix: Remove JAVA_X_HOME usage in tests

0ee2ca4

fix: Use Gradle toolchain registry in favor of JAVA_X_HOME for stable

a49b7a3

chore: Update documentation and seetup script to lift the need of JAV…

bea5f86

…A_*_HOME env vars

bric3 force-pushed the bdu/remove-remaining-use-of-JAVA_X_HOME branch from ad686ad to bea5f86 Compare January 13, 2026 20:50

fix: In CI for JDK 8, java.home may point to the JRE directory (/usr/…

f13392e

…lib/jvm/8/jre)

bric3 marked this pull request as ready for review January 14, 2026 17:43

bric3 requested review from a team as code owners January 14, 2026 17:43

bric3 requested review from PerfectSlayer, daniel-mohedano and sarahchen6 and removed request for a team January 14, 2026 17:44

sarahchen6 reviewed Jan 14, 2026

View reviewed changes

BUILDING.md Outdated Show resolved Hide resolved

sarahchen6 reviewed Jan 14, 2026

View reviewed changes

BUILDING.md Outdated Show resolved Hide resolved

sarahchen6 reviewed Jan 14, 2026

View reviewed changes

dd-smoke-tests/spring-boot-2.7-webflux/build.gradle Outdated Show resolved Hide resolved

sarahchen6 reviewed Jan 14, 2026

View reviewed changes

docs/how_instrumentations_work.md Outdated Show resolved Hide resolved

sarahchen6 approved these changes Jan 14, 2026

View reviewed changes

AlexeyKuznetsov-DD reviewed Jan 14, 2026

View reviewed changes

AlexeyKuznetsov-DD approved these changes Jan 15, 2026

View reviewed changes

PerfectSlayer reviewed Jan 15, 2026

View reviewed changes

daniel-mohedano approved these changes Jan 15, 2026

View reviewed changes

PerfectSlayer mentioned this pull request Jan 15, 2026

Update onboarding script #10384

Open

PerfectSlayer reviewed Jan 15, 2026

View reviewed changes

bric3 commented Jan 15, 2026

View reviewed changes

bric3 added 4 commits January 15, 2026 13:48

chore: Replace buildSrc reliance on toolchain by setting compilation …

5ba7a07

…targets

chore: Allow Gradle Daemon auto-provisioning to work out of the box

b54568a

URLs are acquired from the foojay resolver, when running this task ./gradlew updateDaemonJvm --jvm-version=21 See https://docs.gradle.org/8.14.3/userguide/gradle_daemon.html#configuring_the_daemon_jvm

chore: Improves doc wording

2c12ef8

chore: consistent setup actionable items across setup.sh and setup.ps1

e55a0a4

bric3 force-pushed the bdu/remove-remaining-use-of-JAVA_X_HOME branch from fd1d2a7 to e55a0a4 Compare January 15, 2026 12:54

chore: Tweak Test JVM spec to cover for ground for Gradle Toolchain b…

e6645d3

…eing not enough

bric3 enabled auto-merge (squash) January 15, 2026 14:32

	private val Project.javaToolchains: JavaToolchainService
	get() =
	extensions.getByName("javaToolchains") as JavaToolchainService

		#### macOS


		Use your JDK manager ([mise](https://mise.jdx.dev/), [sdkman](https://sdkman.io/), etc.) or set-up the required JDKs with `brew`

Remove remaining use of JAVA_X_HOME #10340

Are you sure you want to change the base?

Remove remaining use of JAVA_X_HOME #10340

Conversation

bric3 commented Jan 12, 2026

What Does This Do

Motivation

Additional Notes

Uh oh!

pr-commenter bot commented Jan 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Benchmarks

Startup

Parameters

Summary

Load

Parameters

Summary

Dacapo

Parameters

Summary

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

sarahchen6 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

AlexeyKuznetsov-DD left a comment

Choose a reason for hiding this comment

Uh oh!

PerfectSlayer left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bric3 Jan 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bric3 commented Jan 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Remove remaining use of `JAVA_X_HOME` #10340

Remove remaining use of `JAVA_X_HOME` #10340

pr-commenter bot commented Jan 12, 2026 •

edited

Loading

bric3 Jan 15, 2026 •

edited

Loading