Conversation
Samplers were stored inside ProbeRateLimiter singleton into a concurrent map. and only one sampler per probe. Now to allow different samplers for probe, we are storing them directly into the probe instance. Samplers are created when probes are received from the configuration through initSamplers method from Sampled interface.
|
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
Debugger benchmarksParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 9 metrics, 6 unstable metrics. See unchanged results
Request duration reports for reportsgantt
title reports - request duration [CI 0.99] : candidate=None, baseline=None
dateFormat X
axisFormat %s
section baseline
noprobe (357.34 µs) : 231, 484
. : milestone, 357,
basic (276.628 µs) : 266, 287
. : milestone, 277,
loop (8.963 ms) : 8955, 8970
. : milestone, 8963,
section candidate
noprobe (317.126 µs) : 282, 352
. : milestone, 317,
basic (274.696 µs) : 268, 281
. : milestone, 275,
loop (8.967 ms) : 8962, 8972
. : milestone, 8967,
|
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 62 metrics, 9 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.60.0-SNAPSHOT~5c5143d347, baseline=1.60.0-SNAPSHOT~bfa4714663
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.072 s) : 0, 1072354
Total [baseline] (8.751 s) : 0, 8750581
Agent [candidate] (1.065 s) : 0, 1065244
Total [candidate] (8.719 s) : 0, 8719082
section iast
Agent [baseline] (1.227 s) : 0, 1226692
Total [baseline] (9.324 s) : 0, 9323561
Agent [candidate] (1.229 s) : 0, 1229421
Total [candidate] (9.353 s) : 0, 9353135
gantt
title insecure-bank - break down per module: candidate=1.60.0-SNAPSHOT~5c5143d347, baseline=1.60.0-SNAPSHOT~bfa4714663
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.192 ms) : 0, 1192
crashtracking [candidate] (1.179 ms) : 0, 1179
BytebuddyAgent [baseline] (632.972 ms) : 0, 632972
BytebuddyAgent [candidate] (629.005 ms) : 0, 629005
AgentMeter [baseline] (29.19 ms) : 0, 29190
AgentMeter [candidate] (29.021 ms) : 0, 29021
GlobalTracer [baseline] (259.135 ms) : 0, 259135
GlobalTracer [candidate] (257.622 ms) : 0, 257622
AppSec [baseline] (33.081 ms) : 0, 33081
AppSec [candidate] (32.78 ms) : 0, 32780
Debugger [baseline] (62.785 ms) : 0, 62785
Debugger [candidate] (61.188 ms) : 0, 61188
Remote Config [baseline] (636.757 µs) : 0, 637
Remote Config [candidate] (624.667 µs) : 0, 625
Telemetry [baseline] (13.251 ms) : 0, 13251
Telemetry [candidate] (9.879 ms) : 0, 9879
Flare Poller [baseline] (4.637 ms) : 0, 4637
Flare Poller [candidate] (8.611 ms) : 0, 8611
section iast
crashtracking [baseline] (1.195 ms) : 0, 1195
crashtracking [candidate] (1.181 ms) : 0, 1181
BytebuddyAgent [baseline] (792.49 ms) : 0, 792490
BytebuddyAgent [candidate] (794.774 ms) : 0, 794774
AgentMeter [baseline] (11.187 ms) : 0, 11187
AgentMeter [candidate] (11.237 ms) : 0, 11237
GlobalTracer [baseline] (247.025 ms) : 0, 247025
GlobalTracer [candidate] (248.137 ms) : 0, 248137
AppSec [baseline] (35.827 ms) : 0, 35827
AppSec [candidate] (32.66 ms) : 0, 32660
Debugger [baseline] (64.258 ms) : 0, 64258
Debugger [candidate] (66.738 ms) : 0, 66738
Remote Config [baseline] (556.116 µs) : 0, 556
Remote Config [candidate] (534.784 µs) : 0, 535
Telemetry [baseline] (8.621 ms) : 0, 8621
Telemetry [candidate] (8.553 ms) : 0, 8553
Flare Poller [baseline] (3.444 ms) : 0, 3444
Flare Poller [candidate] (3.44 ms) : 0, 3440
IAST [baseline] (26.917 ms) : 0, 26917
IAST [candidate] (26.943 ms) : 0, 26943
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.60.0-SNAPSHOT~5c5143d347, baseline=1.60.0-SNAPSHOT~bfa4714663
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.071 s) : 0, 1070827
Total [baseline] (10.91 s) : 0, 10910045
Agent [candidate] (1.064 s) : 0, 1063974
Total [candidate] (10.865 s) : 0, 10864544
section appsec
Agent [baseline] (1.238 s) : 0, 1237969
Total [baseline] (11.066 s) : 0, 11066290
Agent [candidate] (1.239 s) : 0, 1239329
Total [candidate] (10.954 s) : 0, 10954264
section iast
Agent [baseline] (1.232 s) : 0, 1232432
Total [baseline] (11.136 s) : 0, 11136250
Agent [candidate] (1.23 s) : 0, 1229712
Total [candidate] (11.15 s) : 0, 11149767
section profiling
Agent [baseline] (1.189 s) : 0, 1189345
Total [baseline] (10.862 s) : 0, 10862434
Agent [candidate] (1.204 s) : 0, 1204129
Total [candidate] (11.029 s) : 0, 11029363
gantt
title petclinic - break down per module: candidate=1.60.0-SNAPSHOT~5c5143d347, baseline=1.60.0-SNAPSHOT~bfa4714663
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.176 ms) : 0, 1176
crashtracking [candidate] (1.177 ms) : 0, 1177
BytebuddyAgent [baseline] (631.68 ms) : 0, 631680
BytebuddyAgent [candidate] (627.665 ms) : 0, 627665
AgentMeter [baseline] (29.111 ms) : 0, 29111
AgentMeter [candidate] (28.935 ms) : 0, 28935
GlobalTracer [baseline] (258.668 ms) : 0, 258668
GlobalTracer [candidate] (257.031 ms) : 0, 257031
AppSec [baseline] (33.147 ms) : 0, 33147
AppSec [candidate] (32.732 ms) : 0, 32732
Debugger [baseline] (61.879 ms) : 0, 61879
Debugger [candidate] (62.734 ms) : 0, 62734
Remote Config [baseline] (633.127 µs) : 0, 633
Remote Config [candidate] (629.958 µs) : 0, 630
Telemetry [baseline] (11.601 ms) : 0, 11601
Telemetry [candidate] (14.042 ms) : 0, 14042
Flare Poller [baseline] (7.738 ms) : 0, 7738
Flare Poller [candidate] (3.819 ms) : 0, 3819
section appsec
crashtracking [baseline] (1.175 ms) : 0, 1175
crashtracking [candidate] (1.173 ms) : 0, 1173
BytebuddyAgent [baseline] (657.199 ms) : 0, 657199
BytebuddyAgent [candidate] (658.824 ms) : 0, 658824
AgentMeter [baseline] (11.934 ms) : 0, 11934
AgentMeter [candidate] (12.004 ms) : 0, 12004
GlobalTracer [baseline] (257.753 ms) : 0, 257753
GlobalTracer [candidate] (258.29 ms) : 0, 258290
AppSec [baseline] (167.532 ms) : 0, 167532
AppSec [candidate] (167.586 ms) : 0, 167586
Debugger [baseline] (67.603 ms) : 0, 67603
Debugger [candidate] (67.089 ms) : 0, 67089
Remote Config [baseline] (659.892 µs) : 0, 660
Remote Config [candidate] (646.175 µs) : 0, 646
Telemetry [baseline] (9.844 ms) : 0, 9844
Telemetry [candidate] (9.618 ms) : 0, 9618
Flare Poller [baseline] (3.78 ms) : 0, 3780
Flare Poller [candidate] (3.731 ms) : 0, 3731
IAST [baseline] (25.319 ms) : 0, 25319
IAST [candidate] (25.149 ms) : 0, 25149
section iast
crashtracking [baseline] (1.188 ms) : 0, 1188
crashtracking [candidate] (1.179 ms) : 0, 1179
BytebuddyAgent [baseline] (796.59 ms) : 0, 796590
BytebuddyAgent [candidate] (794.309 ms) : 0, 794309
AgentMeter [baseline] (11.24 ms) : 0, 11240
AgentMeter [candidate] (11.257 ms) : 0, 11257
GlobalTracer [baseline] (248.754 ms) : 0, 248754
GlobalTracer [candidate] (248.06 ms) : 0, 248060
AppSec [baseline] (32.96 ms) : 0, 32960
AppSec [candidate] (32.986 ms) : 0, 32986
Debugger [baseline] (67.253 ms) : 0, 67253
Debugger [candidate] (67.162 ms) : 0, 67162
Remote Config [baseline] (531.13 µs) : 0, 531
Remote Config [candidate] (534.107 µs) : 0, 534
Telemetry [baseline] (8.476 ms) : 0, 8476
Telemetry [candidate] (8.554 ms) : 0, 8554
Flare Poller [baseline] (3.375 ms) : 0, 3375
Flare Poller [candidate] (3.508 ms) : 0, 3508
IAST [baseline] (26.803 ms) : 0, 26803
IAST [candidate] (26.936 ms) : 0, 26936
section profiling
crashtracking [baseline] (1.21 ms) : 0, 1210
crashtracking [candidate] (1.22 ms) : 0, 1220
BytebuddyAgent [baseline] (681.878 ms) : 0, 681878
BytebuddyAgent [candidate] (689.751 ms) : 0, 689751
AgentMeter [baseline] (8.569 ms) : 0, 8569
AgentMeter [candidate] (8.726 ms) : 0, 8726
GlobalTracer [baseline] (215.977 ms) : 0, 215977
GlobalTracer [candidate] (219.037 ms) : 0, 219037
AppSec [baseline] (32.542 ms) : 0, 32542
AppSec [candidate] (33.327 ms) : 0, 33327
Debugger [baseline] (67.274 ms) : 0, 67274
Debugger [candidate] (68.13 ms) : 0, 68130
Remote Config [baseline] (619.336 µs) : 0, 619
Remote Config [candidate] (660.355 µs) : 0, 660
Telemetry [baseline] (8.878 ms) : 0, 8878
Telemetry [candidate] (9.011 ms) : 0, 9011
Flare Poller [baseline] (3.782 ms) : 0, 3782
Flare Poller [candidate] (3.837 ms) : 0, 3837
ProfilingAgent [baseline] (98.809 ms) : 0, 98809
ProfilingAgent [candidate] (100.101 ms) : 0, 100101
Profiling [baseline] (99.382 ms) : 0, 99382
Profiling [candidate] (100.678 ms) : 0, 100678
LoadParameters
See matching parameters
SummaryFound 2 performance improvements and 1 performance regressions! Performance is the same for 17 metrics, 16 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.60.0-SNAPSHOT~5c5143d347, baseline=1.60.0-SNAPSHOT~bfa4714663
dateFormat X
axisFormat %s
section baseline
no_agent (1.173 ms) : 1161, 1185
. : milestone, 1173,
iast (3.223 ms) : 3180, 3267
. : milestone, 3223,
iast_FULL (5.82 ms) : 5761, 5878
. : milestone, 5820,
iast_GLOBAL (3.564 ms) : 3511, 3616
. : milestone, 3564,
profiling (2.121 ms) : 2101, 2140
. : milestone, 2121,
tracing (1.754 ms) : 1740, 1769
. : milestone, 1754,
section candidate
no_agent (1.204 ms) : 1192, 1216
. : milestone, 1204,
iast (3.227 ms) : 3183, 3271
. : milestone, 3227,
iast_FULL (5.801 ms) : 5742, 5859
. : milestone, 5801,
iast_GLOBAL (3.427 ms) : 3370, 3484
. : milestone, 3427,
profiling (2.154 ms) : 2134, 2174
. : milestone, 2154,
tracing (1.817 ms) : 1802, 1833
. : milestone, 1817,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.60.0-SNAPSHOT~5c5143d347, baseline=1.60.0-SNAPSHOT~bfa4714663
dateFormat X
axisFormat %s
section baseline
no_agent (18.745 ms) : 18556, 18935
. : milestone, 18745,
appsec (18.615 ms) : 18425, 18805
. : milestone, 18615,
code_origins (17.854 ms) : 17676, 18032
. : milestone, 17854,
iast (17.864 ms) : 17689, 18040
. : milestone, 17864,
profiling (18.447 ms) : 18265, 18628
. : milestone, 18447,
tracing (18.142 ms) : 17961, 18322
. : milestone, 18142,
section candidate
no_agent (18.386 ms) : 18200, 18572
. : milestone, 18386,
appsec (18.451 ms) : 18264, 18638
. : milestone, 18451,
code_origins (17.577 ms) : 17403, 17751
. : milestone, 17577,
iast (17.589 ms) : 17413, 17764
. : milestone, 17589,
profiling (19.063 ms) : 18870, 19257
. : milestone, 19063,
tracing (17.52 ms) : 17345, 17694
. : milestone, 17520,
DacapoParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 0 unstable metrics.
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.60.0-SNAPSHOT~5c5143d347, baseline=1.60.0-SNAPSHOT~bfa4714663
dateFormat X
axisFormat %s
section baseline
no_agent (15.496 s) : 15496000, 15496000
. : milestone, 15496000,
appsec (14.901 s) : 14901000, 14901000
. : milestone, 14901000,
iast (18.571 s) : 18571000, 18571000
. : milestone, 18571000,
iast_GLOBAL (17.877 s) : 17877000, 17877000
. : milestone, 17877000,
profiling (15.474 s) : 15474000, 15474000
. : milestone, 15474000,
tracing (14.617 s) : 14617000, 14617000
. : milestone, 14617000,
section candidate
no_agent (15.601 s) : 15601000, 15601000
. : milestone, 15601000,
appsec (14.686 s) : 14686000, 14686000
. : milestone, 14686000,
iast (18.086 s) : 18086000, 18086000
. : milestone, 18086000,
iast_GLOBAL (17.889 s) : 17889000, 17889000
. : milestone, 17889000,
profiling (14.806 s) : 14806000, 14806000
. : milestone, 14806000,
tracing (14.661 s) : 14661000, 14661000
. : milestone, 14661000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.60.0-SNAPSHOT~5c5143d347, baseline=1.60.0-SNAPSHOT~bfa4714663
dateFormat X
axisFormat %s
section baseline
no_agent (1.481 ms) : 1469, 1493
. : milestone, 1481,
appsec (3.806 ms) : 3582, 4029
. : milestone, 3806,
iast (2.257 ms) : 2189, 2326
. : milestone, 2257,
iast_GLOBAL (2.292 ms) : 2223, 2361
. : milestone, 2292,
profiling (2.082 ms) : 2027, 2136
. : milestone, 2082,
tracing (2.072 ms) : 2018, 2126
. : milestone, 2072,
section candidate
no_agent (1.474 ms) : 1462, 1485
. : milestone, 1474,
appsec (2.524 ms) : 2470, 2579
. : milestone, 2524,
iast (2.256 ms) : 2187, 2325
. : milestone, 2256,
iast_GLOBAL (2.3 ms) : 2231, 2369
. : milestone, 2300,
profiling (2.076 ms) : 2022, 2131
. : milestone, 2076,
tracing (2.065 ms) : 2012, 2118
. : milestone, 2065,
|
| ? ProbeRateLimiter.DEFAULT_SNAPSHOT_RATE | ||
| : ProbeRateLimiter.DEFAULT_LOG_RATE); | ||
| sampler = ProbeRateLimiter.createSampler(rate); | ||
| } |
There was a problem hiding this comment.
This is going to be called when we receive a new config version for this probe from RC right? Does it re-init the sampler on any update even to unrelated fields? Does that mess with the sampling in any undesired way?
There was a problem hiding this comment.
This will be called on every new version of the probe. It re-inits the sampler. this was already the case before.
The sampling is done with time window that is already short (1s).
What Does This Do
Samplers were stored inside ProbeRateLimiter singleton into a concurrent map. and only one sampler per probe. Now to allow different samplers for probe, we are storing them directly into the probe instance.
Samplers are created when probes are received from the configuration through initSamplers method from Sampled interface.
Motivation
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any other useful labelsclose,fix, or any linking keywords when referencing an issueUse
solvesinstead, and assign the PR milestone to the issueJira ticket: DEBUG-5208
Note: Once your PR is ready to merge, add it to the merge queue by commenting
/merge./merge -ccancels the queue request./merge -f --reason "reason"skips all merge queue checks; please use this judiciously, as some checks do not run at the PR-level. For more information, see this doc.