Skip to content

Fix Job URL format for Github Actions#10514

Open
daniel-mohedano wants to merge 6 commits intomasterfrom
daniel.mohedano/gha-job-url
Open

Fix Job URL format for Github Actions#10514
daniel-mohedano wants to merge 6 commits intomasterfrom
daniel.mohedano/gha-job-url

Conversation

@daniel-mohedano
Copy link
Contributor

@daniel-mohedano daniel-mohedano commented Feb 3, 2026

What Does This Do

  • Fixes how we build the @ci.job.url tag for Github Actions with the appropriate @ci.job.id value. The new format is https://github.com/REPO_NAME/actions/runs/RUN_ID/job/JOB_ID while we previously were limited to https://github.com/REPO_NAME/commit/COMMIT_SHA/checks:
    • If the environment variable JOB_CHECK_RUN_ID is present, we use it as job id.
    • If not, we fall back to reading it from runner diagnostic files.
    • If none apply, we default to the previous behavior.

Motivation

These changes allow users to directly access CI logs with the URL provided, as well as improving the correlation between Test Optimization and CI Visibility.

Additional Notes

Validation in example project: https://github.com/DataDog/danm-testing-repo/pull/8

Contributor Checklist

Jira ticket: SDTEST-2999

@daniel-mohedano daniel-mohedano added type: enhancement Enhancements and improvements comp: ci visibility Continuous Integration Visibility labels Feb 3, 2026
@pr-commenter
Copy link

pr-commenter bot commented Feb 3, 2026

Benchmarks

Startup

Parameters

Baseline Candidate
baseline_or_candidate baseline candidate
git_branch master daniel.mohedano/gha-job-url
git_commit_date 1770217470 1770372627
git_commit_sha 786c771 52de604
release_version 1.60.0-SNAPSHOT~786c771187 1.59.0-SNAPSHOT~52de604f3b
See matching parameters
Baseline Candidate
application insecure-bank insecure-bank
ci_job_date 1770374368 1770374368
ci_job_id 1405795617 1405795617
ci_pipeline_id 94974592 94974592
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-4mylqk8x 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-4mylqk8x 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 63 metrics, 8 unstable metrics.

Startup time reports for insecure-bank
gantt
    title insecure-bank - global startup overhead: candidate=1.59.0-SNAPSHOT~52de604f3b, baseline=1.60.0-SNAPSHOT~786c771187

    dateFormat X
    axisFormat %s
section tracing
Agent [baseline] (1.075 s) : 0, 1075260
Total [baseline] (8.843 s) : 0, 8842811
Agent [candidate] (1.065 s) : 0, 1065068
Total [candidate] (8.735 s) : 0, 8734524
section iast
Agent [baseline] (1.231 s) : 0, 1231431
Total [baseline] (9.362 s) : 0, 9361652
Agent [candidate] (1.242 s) : 0, 1241704
Total [candidate] (9.367 s) : 0, 9367137
Loading
  • baseline results
Module Variant Duration Δ tracing
Agent tracing 1.075 s -
Agent iast 1.231 s 156.171 ms (14.5%)
Total tracing 8.843 s -
Total iast 9.362 s 518.842 ms (5.9%)
  • candidate results
Module Variant Duration Δ tracing
Agent tracing 1.065 s -
Agent iast 1.242 s 176.636 ms (16.6%)
Total tracing 8.735 s -
Total iast 9.367 s 632.613 ms (7.2%)
gantt
    title insecure-bank - break down per module: candidate=1.59.0-SNAPSHOT~52de604f3b, baseline=1.60.0-SNAPSHOT~786c771187

    dateFormat X
    axisFormat %s
section tracing
crashtracking [baseline] (1.202 ms) : 0, 1202
crashtracking [candidate] (1.192 ms) : 0, 1192
BytebuddyAgent [baseline] (633.969 ms) : 0, 633969
BytebuddyAgent [candidate] (629.727 ms) : 0, 629727
AgentMeter [baseline] (28.954 ms) : 0, 28954
AgentMeter [candidate] (28.812 ms) : 0, 28812
GlobalTracer [baseline] (259.888 ms) : 0, 259888
GlobalTracer [candidate] (258.448 ms) : 0, 258448
AppSec [baseline] (33.084 ms) : 0, 33084
AppSec [candidate] (32.947 ms) : 0, 32947
Debugger [baseline] (62.953 ms) : 0, 62953
Debugger [candidate] (62.677 ms) : 0, 62677
Remote Config [baseline] (630.728 µs) : 0, 631
Remote Config [candidate] (603.95 µs) : 0, 604
Telemetry [baseline] (12.645 ms) : 0, 12645
Telemetry [candidate] (10.662 ms) : 0, 10662
Flare Poller [baseline] (6.288 ms) : 0, 6288
Flare Poller [candidate] (4.497 ms) : 0, 4497
section iast
crashtracking [baseline] (1.189 ms) : 0, 1189
crashtracking [candidate] (1.199 ms) : 0, 1199
BytebuddyAgent [baseline] (796.08 ms) : 0, 796080
BytebuddyAgent [candidate] (803.659 ms) : 0, 803659
AgentMeter [baseline] (11.176 ms) : 0, 11176
AgentMeter [candidate] (11.43 ms) : 0, 11430
GlobalTracer [baseline] (248.139 ms) : 0, 248139
GlobalTracer [candidate] (249.577 ms) : 0, 249577
IAST [baseline] (27.042 ms) : 0, 27042
IAST [candidate] (27.302 ms) : 0, 27302
AppSec [baseline] (33.265 ms) : 0, 33265
AppSec [candidate] (33.598 ms) : 0, 33598
Debugger [baseline] (66.339 ms) : 0, 66339
Debugger [candidate] (66.636 ms) : 0, 66636
Remote Config [baseline] (550.17 µs) : 0, 550
Remote Config [candidate] (536.289 µs) : 0, 536
Telemetry [baseline] (8.758 ms) : 0, 8758
Telemetry [candidate] (8.702 ms) : 0, 8702
Flare Poller [baseline] (3.496 ms) : 0, 3496
Flare Poller [candidate] (3.496 ms) : 0, 3496
Loading
Startup time reports for petclinic
gantt
    title petclinic - global startup overhead: candidate=1.59.0-SNAPSHOT~52de604f3b, baseline=1.60.0-SNAPSHOT~786c771187

    dateFormat X
    axisFormat %s
section tracing
Agent [baseline] (1.066 s) : 0, 1065554
Total [baseline] (10.84 s) : 0, 10840460
Agent [candidate] (1.063 s) : 0, 1062640
Total [candidate] (10.903 s) : 0, 10902518
section appsec
Agent [baseline] (1.243 s) : 0, 1242922
Total [baseline] (11.144 s) : 0, 11143532
Agent [candidate] (1.241 s) : 0, 1240662
Total [candidate] (11.095 s) : 0, 11094734
section iast
Agent [baseline] (1.233 s) : 0, 1232668
Total [baseline] (11.241 s) : 0, 11240632
Agent [candidate] (1.241 s) : 0, 1240507
Total [candidate] (11.241 s) : 0, 11240794
section profiling
Agent [baseline] (1.197 s) : 0, 1197289
Total [baseline] (10.971 s) : 0, 10971376
Agent [candidate] (1.192 s) : 0, 1191665
Total [candidate] (10.996 s) : 0, 10995606
Loading
  • baseline results
Module Variant Duration Δ tracing
Agent tracing 1.066 s -
Agent appsec 1.243 s 177.368 ms (16.6%)
Agent iast 1.233 s 167.113 ms (15.7%)
Agent profiling 1.197 s 131.735 ms (12.4%)
Total tracing 10.84 s -
Total appsec 11.144 s 303.072 ms (2.8%)
Total iast 11.241 s 400.172 ms (3.7%)
Total profiling 10.971 s 130.916 ms (1.2%)
  • candidate results
Module Variant Duration Δ tracing
Agent tracing 1.063 s -
Agent appsec 1.241 s 178.022 ms (16.8%)
Agent iast 1.241 s 177.867 ms (16.7%)
Agent profiling 1.192 s 129.025 ms (12.1%)
Total tracing 10.903 s -
Total appsec 11.095 s 192.216 ms (1.8%)
Total iast 11.241 s 338.276 ms (3.1%)
Total profiling 10.996 s 93.088 ms (0.9%)
gantt
    title petclinic - break down per module: candidate=1.59.0-SNAPSHOT~52de604f3b, baseline=1.60.0-SNAPSHOT~786c771187

    dateFormat X
    axisFormat %s
section tracing
crashtracking [baseline] (1.182 ms) : 0, 1182
crashtracking [candidate] (1.18 ms) : 0, 1180
BytebuddyAgent [baseline] (629.051 ms) : 0, 629051
BytebuddyAgent [candidate] (628.228 ms) : 0, 628228
AgentMeter [baseline] (28.758 ms) : 0, 28758
AgentMeter [candidate] (28.735 ms) : 0, 28735
GlobalTracer [baseline] (257.879 ms) : 0, 257879
GlobalTracer [candidate] (257.838 ms) : 0, 257838
AppSec [baseline] (32.8 ms) : 0, 32800
AppSec [candidate] (32.846 ms) : 0, 32846
Debugger [baseline] (62.36 ms) : 0, 62360
Debugger [candidate] (61.461 ms) : 0, 61461
Remote Config [baseline] (606.968 µs) : 0, 607
Remote Config [candidate] (626.087 µs) : 0, 626
Telemetry [baseline] (11.567 ms) : 0, 11567
Telemetry [candidate] (11.789 ms) : 0, 11789
Flare Poller [baseline] (6.06 ms) : 0, 6060
Flare Poller [candidate] (4.531 ms) : 0, 4531
section appsec
crashtracking [baseline] (1.189 ms) : 0, 1189
crashtracking [candidate] (1.182 ms) : 0, 1182
BytebuddyAgent [baseline] (660.178 ms) : 0, 660178
BytebuddyAgent [candidate] (658.048 ms) : 0, 658048
AgentMeter [baseline] (11.772 ms) : 0, 11772
AgentMeter [candidate] (11.75 ms) : 0, 11750
GlobalTracer [baseline] (259.771 ms) : 0, 259771
GlobalTracer [candidate] (259.294 ms) : 0, 259294
AppSec [baseline] (168.644 ms) : 0, 168644
AppSec [candidate] (168.137 ms) : 0, 168137
Debugger [baseline] (67.248 ms) : 0, 67248
Debugger [candidate] (67.862 ms) : 0, 67862
Remote Config [baseline] (671.151 µs) : 0, 671
Remote Config [candidate] (685.13 µs) : 0, 685
Telemetry [baseline] (9.077 ms) : 0, 9077
Telemetry [candidate] (9.228 ms) : 0, 9228
Flare Poller [baseline] (3.623 ms) : 0, 3623
Flare Poller [candidate] (3.761 ms) : 0, 3761
IAST [baseline] (25.312 ms) : 0, 25312
IAST [candidate] (25.31 ms) : 0, 25310
section iast
crashtracking [baseline] (1.183 ms) : 0, 1183
crashtracking [candidate] (1.197 ms) : 0, 1197
BytebuddyAgent [baseline] (796.219 ms) : 0, 796219
BytebuddyAgent [candidate] (802.291 ms) : 0, 802291
AgentMeter [baseline] (11.142 ms) : 0, 11142
AgentMeter [candidate] (11.411 ms) : 0, 11411
GlobalTracer [baseline] (248.459 ms) : 0, 248459
GlobalTracer [candidate] (249.495 ms) : 0, 249495
AppSec [baseline] (34.929 ms) : 0, 34929
AppSec [candidate] (33.884 ms) : 0, 33884
Debugger [baseline] (65.843 ms) : 0, 65843
Debugger [candidate] (66.809 ms) : 0, 66809
Remote Config [baseline] (542.303 µs) : 0, 542
Remote Config [candidate] (539.966 µs) : 0, 540
Telemetry [baseline] (8.703 ms) : 0, 8703
Telemetry [candidate] (8.718 ms) : 0, 8718
Flare Poller [baseline] (3.427 ms) : 0, 3427
Flare Poller [candidate] (3.458 ms) : 0, 3458
IAST [baseline] (26.915 ms) : 0, 26915
IAST [candidate] (27.267 ms) : 0, 27267
section profiling
ProfilingAgent [baseline] (100.149 ms) : 0, 100149
ProfilingAgent [candidate] (99.914 ms) : 0, 99914
crashtracking [baseline] (1.22 ms) : 0, 1220
crashtracking [candidate] (1.216 ms) : 0, 1216
BytebuddyAgent [baseline] (685.736 ms) : 0, 685736
BytebuddyAgent [candidate] (682.709 ms) : 0, 682709
AgentMeter [baseline] (8.725 ms) : 0, 8725
AgentMeter [candidate] (8.624 ms) : 0, 8624
GlobalTracer [baseline] (217.472 ms) : 0, 217472
GlobalTracer [candidate] (215.89 ms) : 0, 215890
AppSec [baseline] (32.619 ms) : 0, 32619
AppSec [candidate] (32.44 ms) : 0, 32440
Debugger [baseline] (67.971 ms) : 0, 67971
Debugger [candidate] (67.65 ms) : 0, 67650
Remote Config [baseline] (604.45 µs) : 0, 604
Remote Config [candidate] (599.845 µs) : 0, 600
Telemetry [baseline] (8.936 ms) : 0, 8936
Telemetry [candidate] (8.861 ms) : 0, 8861
Flare Poller [baseline] (3.75 ms) : 0, 3750
Flare Poller [candidate] (3.806 ms) : 0, 3806
Profiling [baseline] (100.729 ms) : 0, 100729
Profiling [candidate] (100.504 ms) : 0, 100504
Loading

Load

Parameters

Baseline Candidate
baseline_or_candidate baseline candidate
git_branch master daniel.mohedano/gha-job-url
git_commit_date 1770217470 1770372627
git_commit_sha 786c771 52de604
release_version 1.60.0-SNAPSHOT~786c771187 1.59.0-SNAPSHOT~52de604f3b
See matching parameters
Baseline Candidate
application insecure-bank insecure-bank
ci_job_date 1770374939 1770374939
ci_job_id 1405795619 1405795619
ci_pipeline_id 94974592 94974592
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-8mcovivw 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-8mcovivw 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 18 metrics, 17 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:petclinic:no_agent:high_load worse
[+0.796ms; +2.342ms] or [+4.859%; +14.294%]
unstable
[+0.657ms; +3.587ms] or [+2.353%; +12.840%]
unstable
[-49.925op/s; +8.362op/s] or [-18.036%; +3.021%]
17.953ms 30.056ms 256.031op/s 16.384ms 27.934ms 276.812op/s
Request duration reports for insecure-bank
gantt
    title insecure-bank - request duration [CI 0.99] : candidate=1.59.0-SNAPSHOT~52de604f3b, baseline=1.60.0-SNAPSHOT~786c771187
    dateFormat X
    axisFormat %s
section baseline
no_agent (1.184 ms) : 1174, 1194
.   : milestone, 1184,
iast (3.076 ms) : 3035, 3118
.   : milestone, 3076,
iast_FULL (6.033 ms) : 5971, 6095
.   : milestone, 6033,
iast_GLOBAL (3.386 ms) : 3334, 3439
.   : milestone, 3386,
profiling (2.009 ms) : 1991, 2028
.   : milestone, 2009,
tracing (1.76 ms) : 1747, 1774
.   : milestone, 1760,
section candidate
no_agent (1.179 ms) : 1167, 1190
.   : milestone, 1179,
iast (3.193 ms) : 3151, 3236
.   : milestone, 3193,
iast_FULL (5.916 ms) : 5856, 5976
.   : milestone, 5916,
iast_GLOBAL (3.5 ms) : 3447, 3554
.   : milestone, 3500,
profiling (2.203 ms) : 2182, 2223
.   : milestone, 2203,
tracing (1.855 ms) : 1839, 1872
.   : milestone, 1855,
Loading
  • baseline results
Variant Request duration [CI 0.99] Δ no_agent
no_agent 1.184 ms [1.174 ms, 1.194 ms] -
iast 3.076 ms [3.035 ms, 3.118 ms] 1.892 ms (159.8%)
iast_FULL 6.033 ms [5.971 ms, 6.095 ms] 4.849 ms (409.5%)
iast_GLOBAL 3.386 ms [3.334 ms, 3.439 ms] 2.202 ms (186.0%)
profiling 2.009 ms [1.991 ms, 2.028 ms] 825.269 µs (69.7%)
tracing 1.76 ms [1.747 ms, 1.774 ms] 576.361 µs (48.7%)
  • candidate results
Variant Request duration [CI 0.99] Δ no_agent
no_agent 1.179 ms [1.167 ms, 1.19 ms] -
iast 3.193 ms [3.151 ms, 3.236 ms] 2.014 ms (170.9%)
iast_FULL 5.916 ms [5.856 ms, 5.976 ms] 4.737 ms (401.8%)
iast_GLOBAL 3.5 ms [3.447 ms, 3.554 ms] 2.322 ms (196.9%)
profiling 2.203 ms [2.182 ms, 2.223 ms] 1.024 ms (86.8%)
tracing 1.855 ms [1.839 ms, 1.872 ms] 676.47 µs (57.4%)
Request duration reports for petclinic
gantt
    title petclinic - request duration [CI 0.99] : candidate=1.59.0-SNAPSHOT~52de604f3b, baseline=1.60.0-SNAPSHOT~786c771187
    dateFormat X
    axisFormat %s
section baseline
no_agent (16.852 ms) : 16685, 17019
.   : milestone, 16852,
appsec (18.199 ms) : 18018, 18380
.   : milestone, 18199,
code_origins (17.621 ms) : 17443, 17798
.   : milestone, 17621,
iast (17.392 ms) : 17221, 17563
.   : milestone, 17392,
profiling (19.526 ms) : 19327, 19725
.   : milestone, 19526,
tracing (17.715 ms) : 17539, 17891
.   : milestone, 17715,
section candidate
no_agent (18.227 ms) : 18038, 18416
.   : milestone, 18227,
appsec (18.493 ms) : 18307, 18679
.   : milestone, 18493,
code_origins (17.636 ms) : 17461, 17811
.   : milestone, 17636,
iast (17.673 ms) : 17500, 17847
.   : milestone, 17673,
profiling (18.847 ms) : 18657, 19037
.   : milestone, 18847,
tracing (17.932 ms) : 17754, 18110
.   : milestone, 17932,
Loading
  • baseline results
Variant Request duration [CI 0.99] Δ no_agent
no_agent 16.852 ms [16.685 ms, 17.019 ms] -
appsec 18.199 ms [18.018 ms, 18.38 ms] 1.347 ms (8.0%)
code_origins 17.621 ms [17.443 ms, 17.798 ms] 768.628 µs (4.6%)
iast 17.392 ms [17.221 ms, 17.563 ms] 540.334 µs (3.2%)
profiling 19.526 ms [19.327 ms, 19.725 ms] 2.674 ms (15.9%)
tracing 17.715 ms [17.539 ms, 17.891 ms] 863.277 µs (5.1%)
  • candidate results
Variant Request duration [CI 0.99] Δ no_agent
no_agent 18.227 ms [18.038 ms, 18.416 ms] -
appsec 18.493 ms [18.307 ms, 18.679 ms] 266.8 µs (1.5%)
code_origins 17.636 ms [17.461 ms, 17.811 ms] -590.642 µs (-3.2%)
iast 17.673 ms [17.5 ms, 17.847 ms] -553.463 µs (-3.0%)
profiling 18.847 ms [18.657 ms, 19.037 ms] 620.325 µs (3.4%)
tracing 17.932 ms [17.754 ms, 18.11 ms] -294.648 µs (-1.6%)

Dacapo

Parameters

Baseline Candidate
baseline_or_candidate baseline candidate
git_branch master daniel.mohedano/gha-job-url
git_commit_date 1770217470 1770372627
git_commit_sha 786c771 52de604
release_version 1.60.0-SNAPSHOT~786c771187 1.59.0-SNAPSHOT~52de604f3b
See matching parameters
Baseline Candidate
application biojava biojava
ci_job_date 1770374642 1770374642
ci_job_id 1405795622 1405795622
ci_pipeline_id 94974592 94974592
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-xr3clmgp 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-xr3clmgp 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 tomcat
gantt
    title tomcat - execution time [CI 0.99] : candidate=1.59.0-SNAPSHOT~52de604f3b, baseline=1.60.0-SNAPSHOT~786c771187
    dateFormat X
    axisFormat %s
section baseline
no_agent (1.471 ms) : 1459, 1482
.   : milestone, 1471,
appsec (3.745 ms) : 3522, 3967
.   : milestone, 3745,
iast (2.242 ms) : 2173, 2310
.   : milestone, 2242,
iast_GLOBAL (2.293 ms) : 2224, 2363
.   : milestone, 2293,
profiling (2.083 ms) : 2029, 2138
.   : milestone, 2083,
tracing (2.067 ms) : 2013, 2120
.   : milestone, 2067,
section candidate
no_agent (1.472 ms) : 1460, 1483
.   : milestone, 1472,
appsec (3.758 ms) : 3537, 3979
.   : milestone, 3758,
iast (2.243 ms) : 2175, 2312
.   : milestone, 2243,
iast_GLOBAL (2.293 ms) : 2224, 2362
.   : milestone, 2293,
profiling (2.095 ms) : 2039, 2151
.   : milestone, 2095,
tracing (2.062 ms) : 2009, 2116
.   : milestone, 2062,
Loading
  • baseline results
Variant Execution Time [CI 0.99] Δ no_agent
no_agent 1.471 ms [1.459 ms, 1.482 ms] -
appsec 3.745 ms [3.522 ms, 3.967 ms] 2.274 ms (154.6%)
iast 2.242 ms [2.173 ms, 2.31 ms] 770.959 µs (52.4%)
iast_GLOBAL 2.293 ms [2.224 ms, 2.363 ms] 822.448 µs (55.9%)
profiling 2.083 ms [2.029 ms, 2.138 ms] 612.413 µs (41.6%)
tracing 2.067 ms [2.013 ms, 2.12 ms] 595.671 µs (40.5%)
  • candidate results
Variant Execution Time [CI 0.99] Δ no_agent
no_agent 1.472 ms [1.46 ms, 1.483 ms] -
appsec 3.758 ms [3.537 ms, 3.979 ms] 2.286 ms (155.4%)
iast 2.243 ms [2.175 ms, 2.312 ms] 771.791 µs (52.4%)
iast_GLOBAL 2.293 ms [2.224 ms, 2.362 ms] 821.478 µs (55.8%)
profiling 2.095 ms [2.039 ms, 2.151 ms] 623.25 µs (42.4%)
tracing 2.062 ms [2.009 ms, 2.116 ms] 590.76 µs (40.1%)
Execution time for biojava
gantt
    title biojava - execution time [CI 0.99] : candidate=1.59.0-SNAPSHOT~52de604f3b, baseline=1.60.0-SNAPSHOT~786c771187
    dateFormat X
    axisFormat %s
section baseline
no_agent (15.452 s) : 15452000, 15452000
.   : milestone, 15452000,
appsec (14.985 s) : 14985000, 14985000
.   : milestone, 14985000,
iast (18.015 s) : 18015000, 18015000
.   : milestone, 18015000,
iast_GLOBAL (17.884 s) : 17884000, 17884000
.   : milestone, 17884000,
profiling (15.247 s) : 15247000, 15247000
.   : milestone, 15247000,
tracing (14.883 s) : 14883000, 14883000
.   : milestone, 14883000,
section candidate
no_agent (15.308 s) : 15308000, 15308000
.   : milestone, 15308000,
appsec (14.881 s) : 14881000, 14881000
.   : milestone, 14881000,
iast (17.98 s) : 17980000, 17980000
.   : milestone, 17980000,
iast_GLOBAL (18.164 s) : 18164000, 18164000
.   : milestone, 18164000,
profiling (14.888 s) : 14888000, 14888000
.   : milestone, 14888000,
tracing (14.848 s) : 14848000, 14848000
.   : milestone, 14848000,
Loading
  • baseline results
Variant Execution Time [CI 0.99] Δ no_agent
no_agent 15.452 s [15.452 s, 15.452 s] -
appsec 14.985 s [14.985 s, 14.985 s] -467.0 ms (-3.0%)
iast 18.015 s [18.015 s, 18.015 s] 2.563 s (16.6%)
iast_GLOBAL 17.884 s [17.884 s, 17.884 s] 2.432 s (15.7%)
profiling 15.247 s [15.247 s, 15.247 s] -205.0 ms (-1.3%)
tracing 14.883 s [14.883 s, 14.883 s] -569.0 ms (-3.7%)
  • candidate results
Variant Execution Time [CI 0.99] Δ no_agent
no_agent 15.308 s [15.308 s, 15.308 s] -
appsec 14.881 s [14.881 s, 14.881 s] -427.0 ms (-2.8%)
iast 17.98 s [17.98 s, 17.98 s] 2.672 s (17.5%)
iast_GLOBAL 18.164 s [18.164 s, 18.164 s] 2.856 s (18.7%)
profiling 14.888 s [14.888 s, 14.888 s] -420.0 ms (-2.7%)
tracing 14.848 s [14.848 s, 14.848 s] -460.0 ms (-3.0%)

@daniel-mohedano daniel-mohedano removed the comp: ci visibility Continuous Integration Visibility label Feb 6, 2026
@DataDog DataDog deleted a comment from cit-pr-commenter-54b7da bot Feb 6, 2026
@DataDog DataDog deleted a comment from cit-pr-commenter-54b7da bot Feb 6, 2026
@DataDog DataDog deleted a comment from cit-pr-commenter-54b7da bot Feb 6, 2026
@DataDog DataDog deleted a comment from cit-pr-commenter-54b7da bot Feb 6, 2026
@DataDog DataDog deleted a comment from cit-pr-commenter-54b7da bot Feb 6, 2026
@DataDog DataDog deleted a comment from cit-pr-commenter-54b7da bot Feb 6, 2026
@DataDog DataDog deleted a comment from cit-pr-commenter-54b7da bot Feb 6, 2026
@DataDog DataDog deleted a comment from cit-pr-commenter-54b7da bot Feb 6, 2026
@DataDog DataDog deleted a comment from cit-pr-commenter-54b7da bot Feb 6, 2026
@DataDog DataDog deleted a comment from cit-pr-commenter-54b7da bot Feb 6, 2026
@DataDog DataDog deleted a comment from cit-pr-commenter-54b7da bot Feb 6, 2026
@DataDog DataDog deleted a comment from cit-pr-commenter-54b7da bot Feb 6, 2026
@DataDog DataDog deleted a comment from cit-pr-commenter-54b7da bot Feb 6, 2026
@daniel-mohedano daniel-mohedano added the comp: ci visibility Continuous Integration Visibility label Feb 6, 2026
@daniel-mohedano daniel-mohedano marked this pull request as ready for review February 6, 2026 10:34
@daniel-mohedano daniel-mohedano requested a review from a team as a code owner February 6, 2026 10:34

// Sort by filename in descending order to get the most recent file first
workerFiles.sort(
Comparator.comparing(p -> p.getFileName().toString(), Comparator.reverseOrder()));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nitpitck: we don't need to sort the entire list to find the most recent log, a single O(N) pass is enough:

Path mostRecentLog = null;
for (Path workerLog : stream) {
    if (mostRecentLog == null ||
        workerLog.getFileName().toString().compareTo(mostRecentLog.getFileName().toString()) > 0) {
        mostRecentLog = workerLog;
    }
}
if (workerLog == null) { return null; }

if (value.contains(".")) {
try {
long longValue = (long) Double.parseDouble(value);
lastMatch = String.valueOf(longValue);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since you're converting it back to String, could we just do:

int pointIdx = value.indexOf(".");
if (pointIdx != -1) {
  lastMatch = value.substring(0, pointIdx);
}

I might be missing things, but looking at the regex it seems like a NumberFormatException is not possible here given how you match the group

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp: ci visibility Continuous Integration Visibility type: enhancement Enhancements and improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants