Remove Strings.join API in favor of JDK String.join#7974
Merged
PerfectSlayer merged 1 commit intomasterfrom Nov 20, 2024
Merged
Remove Strings.join API in favor of JDK String.join#7974PerfectSlayer merged 1 commit intomasterfrom
PerfectSlayer merged 1 commit intomasterfrom
Conversation
67bc957 to
b63f6b2
Compare
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 55 metrics, 8 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.43.0-SNAPSHOT~b4d3e03b18, baseline=1.43.0-SNAPSHOT~ff8ee85501
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.079 s) : 0, 1078999
Total [baseline] (10.468 s) : 0, 10467843
Agent [candidate] (1.083 s) : 0, 1082994
Total [candidate] (10.413 s) : 0, 10412518
section appsec
Agent [baseline] (1.215 s) : 0, 1214528
Total [baseline] (10.699 s) : 0, 10699386
Agent [candidate] (1.222 s) : 0, 1222422
Total [candidate] (10.733 s) : 0, 10732752
section iast
Agent [baseline] (1.215 s) : 0, 1214749
Total [baseline] (10.949 s) : 0, 10948973
Agent [candidate] (1.225 s) : 0, 1224865
Total [candidate] (10.896 s) : 0, 10896463
section profiling
Agent [baseline] (1.294 s) : 0, 1294473
Total [baseline] (10.837 s) : 0, 10837268
Agent [candidate] (1.276 s) : 0, 1275670
Total [candidate] (10.736 s) : 0, 10735563
gantt
title petclinic - break down per module: candidate=1.43.0-SNAPSHOT~b4d3e03b18, baseline=1.43.0-SNAPSHOT~ff8ee85501
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (686.376 ms) : 0, 686376
BytebuddyAgent [candidate] (689.781 ms) : 0, 689781
GlobalTracer [baseline] (315.047 ms) : 0, 315047
GlobalTracer [candidate] (314.067 ms) : 0, 314067
AppSec [baseline] (54.057 ms) : 0, 54057
AppSec [candidate] (54.118 ms) : 0, 54118
Remote Config [baseline] (689.483 µs) : 0, 689
Remote Config [candidate] (675.475 µs) : 0, 675
Telemetry [baseline] (9.2 ms) : 0, 9200
Telemetry [candidate] (10.637 ms) : 0, 10637
section appsec
BytebuddyAgent [baseline] (703.464 ms) : 0, 703464
BytebuddyAgent [candidate] (708.177 ms) : 0, 708177
GlobalTracer [baseline] (312.614 ms) : 0, 312614
GlobalTracer [candidate] (315.006 ms) : 0, 315006
AppSec [baseline] (166.286 ms) : 0, 166286
AppSec [candidate] (166.527 ms) : 0, 166527
IAST [baseline] (20.226 ms) : 0, 20226
IAST [candidate] (20.427 ms) : 0, 20427
Remote Config [baseline] (633.88 µs) : 0, 634
Remote Config [candidate] (639.176 µs) : 0, 639
Telemetry [baseline] (7.811 ms) : 0, 7811
Telemetry [candidate] (8.149 ms) : 0, 8149
section iast
BytebuddyAgent [baseline] (809.597 ms) : 0, 809597
BytebuddyAgent [candidate] (815.675 ms) : 0, 815675
GlobalTracer [baseline] (305.198 ms) : 0, 305198
GlobalTracer [candidate] (308.238 ms) : 0, 308238
AppSec [baseline] (57.135 ms) : 0, 57135
AppSec [candidate] (57.027 ms) : 0, 57027
IAST [baseline] (20.957 ms) : 0, 20957
IAST [candidate] (21.944 ms) : 0, 21944
Remote Config [baseline] (616.331 µs) : 0, 616
Remote Config [candidate] (626.711 µs) : 0, 627
Telemetry [baseline] (7.507 ms) : 0, 7507
Telemetry [candidate] (7.509 ms) : 0, 7509
section profiling
BytebuddyAgent [baseline] (690.69 ms) : 0, 690690
BytebuddyAgent [candidate] (680.617 ms) : 0, 680617
GlobalTracer [baseline] (402.437 ms) : 0, 402437
GlobalTracer [candidate] (397.321 ms) : 0, 397321
AppSec [baseline] (55.101 ms) : 0, 55101
AppSec [candidate] (54.662 ms) : 0, 54662
Remote Config [baseline] (678.717 µs) : 0, 679
Remote Config [candidate] (660.897 µs) : 0, 661
Telemetry [baseline] (12.934 ms) : 0, 12934
Telemetry [candidate] (12.615 ms) : 0, 12615
ProfilingAgent [baseline] (93.272 ms) : 0, 93272
ProfilingAgent [candidate] (91.086 ms) : 0, 91086
Profiling [baseline] (93.296 ms) : 0, 93296
Profiling [candidate] (91.11 ms) : 0, 91110
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.43.0-SNAPSHOT~b4d3e03b18, baseline=1.43.0-SNAPSHOT~ff8ee85501
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.096 s) : 0, 1096376
Total [baseline] (8.606 s) : 0, 8605831
Agent [candidate] (1.078 s) : 0, 1078036
Total [candidate] (8.59 s) : 0, 8589779
section iast
Agent [baseline] (1.212 s) : 0, 1212149
Total [baseline] (9.126 s) : 0, 9126031
Agent [candidate] (1.214 s) : 0, 1214415
Total [candidate] (9.182 s) : 0, 9181583
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.204 s) : 0, 1203647
Total [baseline] (9.107 s) : 0, 9106597
Agent [candidate] (1.216 s) : 0, 1216079
Total [candidate] (9.118 s) : 0, 9117772
section iast_TELEMETRY_OFF
Agent [baseline] (1.202 s) : 0, 1201952
Total [baseline] (9.15 s) : 0, 9150184
Agent [candidate] (1.203 s) : 0, 1203116
Total [candidate] (9.143 s) : 0, 9142760
gantt
title insecure-bank - break down per module: candidate=1.43.0-SNAPSHOT~b4d3e03b18, baseline=1.43.0-SNAPSHOT~ff8ee85501
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (696.599 ms) : 0, 696599
BytebuddyAgent [candidate] (686.004 ms) : 0, 686004
GlobalTracer [baseline] (318.923 ms) : 0, 318923
GlobalTracer [candidate] (314.154 ms) : 0, 314154
AppSec [baseline] (54.825 ms) : 0, 54825
AppSec [candidate] (54.318 ms) : 0, 54318
Remote Config [baseline] (694.682 µs) : 0, 695
Remote Config [candidate] (679.177 µs) : 0, 679
Telemetry [baseline] (11.486 ms) : 0, 11486
Telemetry [candidate] (9.269 ms) : 0, 9269
section iast
BytebuddyAgent [baseline] (808.575 ms) : 0, 808575
BytebuddyAgent [candidate] (808.268 ms) : 0, 808268
GlobalTracer [baseline] (304.128 ms) : 0, 304128
GlobalTracer [candidate] (305.715 ms) : 0, 305715
AppSec [baseline] (57.089 ms) : 0, 57089
AppSec [candidate] (57.622 ms) : 0, 57622
IAST [baseline] (20.583 ms) : 0, 20583
IAST [candidate] (20.995 ms) : 0, 20995
Remote Config [baseline] (605.01 µs) : 0, 605
Remote Config [candidate] (613.794 µs) : 0, 614
Telemetry [baseline] (7.411 ms) : 0, 7411
Telemetry [candidate] (7.475 ms) : 0, 7475
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (800.454 ms) : 0, 800454
BytebuddyAgent [candidate] (808.919 ms) : 0, 808919
GlobalTracer [baseline] (304.017 ms) : 0, 304017
GlobalTracer [candidate] (306.478 ms) : 0, 306478
AppSec [baseline] (57.311 ms) : 0, 57311
AppSec [candidate] (57.069 ms) : 0, 57069
IAST [baseline] (20.26 ms) : 0, 20260
IAST [candidate] (21.667 ms) : 0, 21667
Remote Config [baseline] (595.447 µs) : 0, 595
Remote Config [candidate] (623.336 µs) : 0, 623
Telemetry [baseline] (7.391 ms) : 0, 7391
Telemetry [candidate] (7.572 ms) : 0, 7572
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (798.908 ms) : 0, 798908
BytebuddyAgent [candidate] (798.877 ms) : 0, 798877
GlobalTracer [baseline] (303.92 ms) : 0, 303920
GlobalTracer [candidate] (304.283 ms) : 0, 304283
AppSec [baseline] (57.493 ms) : 0, 57493
AppSec [candidate] (58.05 ms) : 0, 58050
IAST [baseline] (20.073 ms) : 0, 20073
IAST [candidate] (20.243 ms) : 0, 20243
Remote Config [baseline] (605.376 µs) : 0, 605
Remote Config [candidate] (614.138 µs) : 0, 614
Telemetry [baseline] (7.348 ms) : 0, 7348
Telemetry [candidate] (7.455 ms) : 0, 7455
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 petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.43.0-SNAPSHOT~b4d3e03b18, baseline=1.43.0-SNAPSHOT~ff8ee85501
dateFormat X
axisFormat %s
section baseline
no_agent (1.337 ms) : 1318, 1357
. : milestone, 1337,
appsec (1.749 ms) : 1724, 1773
. : milestone, 1749,
appsec_no_iast (1.731 ms) : 1706, 1755
. : milestone, 1731,
iast (1.493 ms) : 1470, 1516
. : milestone, 1493,
profiling (1.524 ms) : 1500, 1548
. : milestone, 1524,
tracing (1.473 ms) : 1448, 1498
. : milestone, 1473,
section candidate
no_agent (1.344 ms) : 1324, 1364
. : milestone, 1344,
appsec (1.765 ms) : 1742, 1789
. : milestone, 1765,
appsec_no_iast (1.746 ms) : 1722, 1771
. : milestone, 1746,
iast (1.487 ms) : 1463, 1510
. : milestone, 1487,
profiling (1.538 ms) : 1514, 1561
. : milestone, 1538,
tracing (1.489 ms) : 1465, 1513
. : milestone, 1489,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.43.0-SNAPSHOT~b4d3e03b18, baseline=1.43.0-SNAPSHOT~ff8ee85501
dateFormat X
axisFormat %s
section baseline
no_agent (373.568 µs) : 354, 393
. : milestone, 374,
iast (494.835 µs) : 473, 517
. : milestone, 495,
iast_FULL (649.471 µs) : 628, 671
. : milestone, 649,
iast_GLOBAL (514.453 µs) : 494, 535
. : milestone, 514,
iast_HARDCODED_SECRET_DISABLED (489.348 µs) : 468, 511
. : milestone, 489,
iast_INACTIVE (453.458 µs) : 432, 475
. : milestone, 453,
iast_TELEMETRY_OFF (481.423 µs) : 460, 503
. : milestone, 481,
tracing (447.514 µs) : 427, 468
. : milestone, 448,
section candidate
no_agent (371.772 µs) : 352, 391
. : milestone, 372,
iast (493.74 µs) : 472, 515
. : milestone, 494,
iast_FULL (647.058 µs) : 626, 668
. : milestone, 647,
iast_GLOBAL (525.585 µs) : 503, 548
. : milestone, 526,
iast_HARDCODED_SECRET_DISABLED (487.229 µs) : 466, 508
. : milestone, 487,
iast_INACTIVE (444.804 µs) : 424, 465
. : milestone, 445,
iast_TELEMETRY_OFF (482.689 µs) : 461, 504
. : milestone, 483,
tracing (449.473 µs) : 428, 471
. : milestone, 449,
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.43.0-SNAPSHOT~b4d3e03b18, baseline=1.43.0-SNAPSHOT~ff8ee85501
dateFormat X
axisFormat %s
section baseline
no_agent (1.466 ms) : 1454, 1477
. : milestone, 1466,
appsec (2.331 ms) : 2289, 2372
. : milestone, 2331,
iast (2.053 ms) : 2002, 2104
. : milestone, 2053,
iast_GLOBAL (2.11 ms) : 2058, 2162
. : milestone, 2110,
profiling (1.938 ms) : 1896, 1981
. : milestone, 1938,
tracing (1.91 ms) : 1870, 1950
. : milestone, 1910,
section candidate
no_agent (1.467 ms) : 1456, 1479
. : milestone, 1467,
appsec (2.33 ms) : 2288, 2371
. : milestone, 2330,
iast (2.071 ms) : 2019, 2123
. : milestone, 2071,
iast_GLOBAL (2.113 ms) : 2060, 2165
. : milestone, 2113,
profiling (1.947 ms) : 1905, 1989
. : milestone, 1947,
tracing (1.914 ms) : 1875, 1953
. : milestone, 1914,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.43.0-SNAPSHOT~b4d3e03b18, baseline=1.43.0-SNAPSHOT~ff8ee85501
dateFormat X
axisFormat %s
section baseline
no_agent (15.106 s) : 15106000, 15106000
. : milestone, 15106000,
appsec (15.273 s) : 15273000, 15273000
. : milestone, 15273000,
iast (18.436 s) : 18436000, 18436000
. : milestone, 18436000,
iast_GLOBAL (18.24 s) : 18240000, 18240000
. : milestone, 18240000,
profiling (15.203 s) : 15203000, 15203000
. : milestone, 15203000,
tracing (14.975 s) : 14975000, 14975000
. : milestone, 14975000,
section candidate
no_agent (15.74 s) : 15740000, 15740000
. : milestone, 15740000,
appsec (15.418 s) : 15418000, 15418000
. : milestone, 15418000,
iast (18.854 s) : 18854000, 18854000
. : milestone, 18854000,
iast_GLOBAL (17.783 s) : 17783000, 17783000
. : milestone, 17783000,
profiling (15.569 s) : 15569000, 15569000
. : milestone, 15569000,
tracing (15.004 s) : 15004000, 15004000
. : milestone, 15004000,
|
b63f6b2 to
b4d3e03
Compare
nikita-tkachenko-datadog
approved these changes
Nov 20, 2024
manuel-alvarez-alvarez
approved these changes
Nov 20, 2024
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
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
This PR cleans up custom
Strings.jointo use the JDKString.joininstead.Motivation
This API should date from pre 1.0 which was Java 7 compatible while
String.joinwas introduced in Java 8.The JDK method pass our custom
Strings.jointests too.Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any usefull labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]