-
Notifications
You must be signed in to change notification settings - Fork 320
remove colons from process tags values #10109
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
Conversation
|
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 58 metrics, 7 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.57.0-SNAPSHOT~9f819c356e, baseline=1.57.0-SNAPSHOT~aff04738f0
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.092 s) : 0, 1092050
Total [baseline] (8.772 s) : 0, 8772481
Agent [candidate] (1.081 s) : 0, 1081017
Total [candidate] (8.729 s) : 0, 8728924
section iast
Agent [baseline] (1.221 s) : 0, 1220758
Total [baseline] (9.445 s) : 0, 9445162
Agent [candidate] (1.221 s) : 0, 1220633
Total [candidate] (9.414 s) : 0, 9414225
gantt
title insecure-bank - break down per module: candidate=1.57.0-SNAPSHOT~9f819c356e, baseline=1.57.0-SNAPSHOT~aff04738f0
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.203 ms) : 0, 1203
crashtracking [candidate] (1.214 ms) : 0, 1214
BytebuddyAgent [baseline] (656.173 ms) : 0, 656173
BytebuddyAgent [candidate] (649.236 ms) : 0, 649236
GlobalTracer [baseline] (284.251 ms) : 0, 284251
GlobalTracer [candidate] (281.937 ms) : 0, 281937
AppSec [baseline] (32.814 ms) : 0, 32814
AppSec [candidate] (32.445 ms) : 0, 32445
Debugger [baseline] (68.169 ms) : 0, 68169
Debugger [candidate] (67.189 ms) : 0, 67189
Remote Config [baseline] (665.804 µs) : 0, 666
Remote Config [candidate] (654.994 µs) : 0, 655
Telemetry [baseline] (9.195 ms) : 0, 9195
Telemetry [candidate] (9.042 ms) : 0, 9042
Flare Poller [baseline] (3.846 ms) : 0, 3846
Flare Poller [candidate] (3.814 ms) : 0, 3814
section iast
crashtracking [baseline] (1.199 ms) : 0, 1199
crashtracking [candidate] (1.179 ms) : 0, 1179
BytebuddyAgent [baseline] (790.255 ms) : 0, 790255
BytebuddyAgent [candidate] (789.847 ms) : 0, 789847
GlobalTracer [baseline] (255.139 ms) : 0, 255139
GlobalTracer [candidate] (255.318 ms) : 0, 255318
IAST [baseline] (27.02 ms) : 0, 27020
IAST [candidate] (26.907 ms) : 0, 26907
AppSec [baseline] (35.152 ms) : 0, 35152
AppSec [candidate] (35.094 ms) : 0, 35094
Debugger [baseline] (64.405 ms) : 0, 64405
Debugger [candidate] (64.563 ms) : 0, 64563
Remote Config [baseline] (556.29 µs) : 0, 556
Remote Config [candidate] (578.716 µs) : 0, 579
Telemetry [baseline] (8.347 ms) : 0, 8347
Telemetry [candidate] (8.357 ms) : 0, 8357
Flare Poller [baseline] (3.42 ms) : 0, 3420
Flare Poller [candidate] (3.477 ms) : 0, 3477
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.57.0-SNAPSHOT~9f819c356e, baseline=1.57.0-SNAPSHOT~aff04738f0
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.079 s) : 0, 1078981
Total [baseline] (10.817 s) : 0, 10817110
Agent [candidate] (1.08 s) : 0, 1079670
Total [candidate] (10.855 s) : 0, 10854960
section appsec
Agent [baseline] (1.274 s) : 0, 1273549
Total [baseline] (11.14 s) : 0, 11139717
Agent [candidate] (1.262 s) : 0, 1261791
Total [candidate] (11.078 s) : 0, 11078405
section iast
Agent [baseline] (1.22 s) : 0, 1220167
Total [baseline] (11.139 s) : 0, 11139434
Agent [candidate] (1.219 s) : 0, 1219245
Total [candidate] (11.124 s) : 0, 11124242
section profiling
Agent [baseline] (1.204 s) : 0, 1203517
Total [baseline] (11.126 s) : 0, 11126390
Agent [candidate] (1.202 s) : 0, 1202216
Total [candidate] (11.029 s) : 0, 11028762
gantt
title petclinic - break down per module: candidate=1.57.0-SNAPSHOT~9f819c356e, baseline=1.57.0-SNAPSHOT~aff04738f0
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.204 ms) : 0, 1204
crashtracking [candidate] (1.186 ms) : 0, 1186
BytebuddyAgent [baseline] (647.153 ms) : 0, 647153
BytebuddyAgent [candidate] (647.831 ms) : 0, 647831
GlobalTracer [baseline] (280.831 ms) : 0, 280831
GlobalTracer [candidate] (281.665 ms) : 0, 281665
AppSec [baseline] (32.347 ms) : 0, 32347
AppSec [candidate] (32.018 ms) : 0, 32018
Debugger [baseline] (68.7 ms) : 0, 68700
Debugger [candidate] (68.006 ms) : 0, 68006
Remote Config [baseline] (624.38 µs) : 0, 624
Remote Config [candidate] (636.802 µs) : 0, 637
Telemetry [baseline] (8.999 ms) : 0, 8999
Telemetry [candidate] (9.111 ms) : 0, 9111
Flare Poller [baseline] (3.727 ms) : 0, 3727
Flare Poller [candidate] (3.78 ms) : 0, 3780
section appsec
crashtracking [baseline] (1.197 ms) : 0, 1197
crashtracking [candidate] (1.184 ms) : 0, 1184
BytebuddyAgent [baseline] (694.529 ms) : 0, 694529
BytebuddyAgent [candidate] (687.764 ms) : 0, 687764
GlobalTracer [baseline] (261.172 ms) : 0, 261172
GlobalTracer [candidate] (258.739 ms) : 0, 258739
IAST [baseline] (24.757 ms) : 0, 24757
IAST [candidate] (24.702 ms) : 0, 24702
AppSec [baseline] (175.637 ms) : 0, 175637
AppSec [candidate] (174.678 ms) : 0, 174678
Debugger [baseline] (66.603 ms) : 0, 66603
Debugger [candidate] (65.783 ms) : 0, 65783
Remote Config [baseline] (725.135 µs) : 0, 725
Remote Config [candidate] (688.235 µs) : 0, 688
Telemetry [baseline] (9.193 ms) : 0, 9193
Telemetry [candidate] (8.989 ms) : 0, 8989
Flare Poller [baseline] (4.039 ms) : 0, 4039
Flare Poller [candidate] (3.798 ms) : 0, 3798
section iast
crashtracking [baseline] (1.181 ms) : 0, 1181
crashtracking [candidate] (1.196 ms) : 0, 1196
BytebuddyAgent [baseline] (788.843 ms) : 0, 788843
BytebuddyAgent [candidate] (788.409 ms) : 0, 788409
GlobalTracer [baseline] (254.639 ms) : 0, 254639
GlobalTracer [candidate] (254.387 ms) : 0, 254387
IAST [baseline] (27.057 ms) : 0, 27057
IAST [candidate] (26.765 ms) : 0, 26765
AppSec [baseline] (35.35 ms) : 0, 35350
AppSec [candidate] (34.444 ms) : 0, 34444
Debugger [baseline] (65.448 ms) : 0, 65448
Debugger [candidate] (66.279 ms) : 0, 66279
Remote Config [baseline] (534.444 µs) : 0, 534
Remote Config [candidate] (587.777 µs) : 0, 588
Telemetry [baseline] (8.429 ms) : 0, 8429
Telemetry [candidate] (8.382 ms) : 0, 8382
Flare Poller [baseline] (3.455 ms) : 0, 3455
Flare Poller [candidate] (3.49 ms) : 0, 3490
section profiling
crashtracking [baseline] (1.178 ms) : 0, 1178
crashtracking [candidate] (1.178 ms) : 0, 1178
BytebuddyAgent [baseline] (701.927 ms) : 0, 701927
BytebuddyAgent [candidate] (700.538 ms) : 0, 700538
GlobalTracer [baseline] (220.217 ms) : 0, 220217
GlobalTracer [candidate] (220.788 ms) : 0, 220788
AppSec [baseline] (32.299 ms) : 0, 32299
AppSec [candidate] (32.199 ms) : 0, 32199
Debugger [baseline] (68.136 ms) : 0, 68136
Debugger [candidate] (67.917 ms) : 0, 67917
Remote Config [baseline] (675.714 µs) : 0, 676
Remote Config [candidate] (613.935 µs) : 0, 614
Telemetry [baseline] (9.103 ms) : 0, 9103
Telemetry [candidate] (8.932 ms) : 0, 8932
Flare Poller [baseline] (3.86 ms) : 0, 3860
Flare Poller [candidate] (3.778 ms) : 0, 3778
ProfilingAgent [baseline] (96.686 ms) : 0, 96686
ProfilingAgent [candidate] (96.894 ms) : 0, 96894
Profiling [baseline] (97.276 ms) : 0, 97276
Profiling [candidate] (97.459 ms) : 0, 97459
LoadParameters
See matching parameters
SummaryFound 4 performance improvements and 0 performance regressions! Performance is the same for 16 metrics, 16 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.57.0-SNAPSHOT~9f819c356e, baseline=1.57.0-SNAPSHOT~aff04738f0
dateFormat X
axisFormat %s
section baseline
no_agent (1.195 ms) : 1183, 1207
. : milestone, 1195,
iast (3.266 ms) : 3224, 3309
. : milestone, 3266,
iast_FULL (5.958 ms) : 5898, 6018
. : milestone, 5958,
iast_GLOBAL (3.896 ms) : 3839, 3954
. : milestone, 3896,
profiling (2.104 ms) : 2086, 2123
. : milestone, 2104,
tracing (1.776 ms) : 1762, 1791
. : milestone, 1776,
section candidate
no_agent (1.203 ms) : 1191, 1215
. : milestone, 1203,
iast (3.212 ms) : 3167, 3257
. : milestone, 3212,
iast_FULL (5.732 ms) : 5675, 5789
. : milestone, 5732,
iast_GLOBAL (3.614 ms) : 3557, 3671
. : milestone, 3614,
profiling (2.195 ms) : 2175, 2215
. : milestone, 2195,
tracing (1.884 ms) : 1867, 1901
. : milestone, 1884,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.57.0-SNAPSHOT~9f819c356e, baseline=1.57.0-SNAPSHOT~aff04738f0
dateFormat X
axisFormat %s
section baseline
no_agent (19.64 ms) : 19437, 19844
. : milestone, 19640,
appsec (19.055 ms) : 18859, 19251
. : milestone, 19055,
code_origins (17.846 ms) : 17668, 18024
. : milestone, 17846,
iast (17.936 ms) : 17758, 18115
. : milestone, 17936,
profiling (18.729 ms) : 18539, 18920
. : milestone, 18729,
tracing (17.656 ms) : 17482, 17830
. : milestone, 17656,
section candidate
no_agent (19.407 ms) : 19209, 19605
. : milestone, 19407,
appsec (18.866 ms) : 18674, 19059
. : milestone, 18866,
code_origins (17.962 ms) : 17785, 18138
. : milestone, 17962,
iast (17.737 ms) : 17562, 17911
. : milestone, 17737,
profiling (18.614 ms) : 18428, 18799
. : milestone, 18614,
tracing (17.843 ms) : 17663, 18023
. : milestone, 17843,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 1 unstable metrics. Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.57.0-SNAPSHOT~9f819c356e, baseline=1.57.0-SNAPSHOT~aff04738f0
dateFormat X
axisFormat %s
section baseline
no_agent (1.479 ms) : 1468, 1491
. : milestone, 1479,
appsec (3.736 ms) : 3514, 3958
. : milestone, 3736,
iast (2.24 ms) : 2175, 2304
. : milestone, 2240,
iast_GLOBAL (2.28 ms) : 2214, 2345
. : milestone, 2280,
profiling (2.084 ms) : 2031, 2136
. : milestone, 2084,
tracing (2.054 ms) : 2003, 2105
. : milestone, 2054,
section candidate
no_agent (1.483 ms) : 1472, 1495
. : milestone, 1483,
appsec (3.71 ms) : 3493, 3927
. : milestone, 3710,
iast (2.229 ms) : 2164, 2293
. : milestone, 2229,
iast_GLOBAL (2.277 ms) : 2212, 2343
. : milestone, 2277,
profiling (2.109 ms) : 2055, 2164
. : milestone, 2109,
tracing (2.06 ms) : 2010, 2111
. : milestone, 2060,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.57.0-SNAPSHOT~9f819c356e, baseline=1.57.0-SNAPSHOT~aff04738f0
dateFormat X
axisFormat %s
section baseline
no_agent (14.976 s) : 14976000, 14976000
. : milestone, 14976000,
appsec (14.693 s) : 14693000, 14693000
. : milestone, 14693000,
iast (18.179 s) : 18179000, 18179000
. : milestone, 18179000,
iast_GLOBAL (17.877 s) : 17877000, 17877000
. : milestone, 17877000,
profiling (14.576 s) : 14576000, 14576000
. : milestone, 14576000,
tracing (14.538 s) : 14538000, 14538000
. : milestone, 14538000,
section candidate
no_agent (14.948 s) : 14948000, 14948000
. : milestone, 14948000,
appsec (14.723 s) : 14723000, 14723000
. : milestone, 14723000,
iast (18.393 s) : 18393000, 18393000
. : milestone, 18393000,
iast_GLOBAL (18.054 s) : 18054000, 18054000
. : milestone, 18054000,
profiling (14.57 s) : 14570000, 14570000
. : milestone, 14570000,
tracing (14.944 s) : 14944000, 14944000
. : milestone, 14944000,
|
| entry.getKey() | ||
| + ":" | ||
| + TraceUtils.normalizeTagValue( | ||
| entry.getValue().replace(':', '_')))); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ideally it can also be factorised somewhere to have it reusable but not a hard requirement.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, I'm not sure it's something we're going to need somewhere else any time soon
amarziali
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the improvement
What Does This Do
I realized that Java "only" applied the normal tag value normalization, but for process tags, the spec states that colons are forbidden, so I added that extra step (before normalization to make sure consecutive
_are collapsed).Also added tests on normalization.
Motivation
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any useful labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]