-
Notifications
You must be signed in to change notification settings - Fork 290
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
Increase spark listener max collection size #6979
Merged
paul-laffon-dd
merged 1 commit into
master
from
paul.laffon/increase-spark-listener-max-event-size
May 3, 2024
Merged
Increase spark listener max collection size #6979
paul-laffon-dd
merged 1 commit into
master
from
paul.laffon/increase-spark-listener-max-event-size
May 3, 2024
Conversation
This file contains 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
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 51 metrics, 12 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.34.0-SNAPSHOT~dbf06bb509, baseline=1.34.0-SNAPSHOT~5c8d55ca87
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.074 s) : 0, 1073636
Total [baseline] (10.376 s) : 0, 10376371
Agent [candidate] (1.075 s) : 0, 1074709
Total [candidate] (10.364 s) : 0, 10364005
section appsec
Agent [baseline] (1.192 s) : 0, 1191846
Total [baseline] (10.488 s) : 0, 10488474
Agent [candidate] (1.189 s) : 0, 1188807
Total [candidate] (10.512 s) : 0, 10511657
section iast
Agent [baseline] (1.214 s) : 0, 1213527
Total [baseline] (10.765 s) : 0, 10764755
Agent [candidate] (1.209 s) : 0, 1209234
Total [candidate] (10.79 s) : 0, 10789641
section profiling
Agent [baseline] (1.269 s) : 0, 1268883
Total [baseline] (10.609 s) : 0, 10608785
Agent [candidate] (1.27 s) : 0, 1269859
Total [candidate] (10.614 s) : 0, 10614198
gantt
title petclinic - break down per module: candidate=1.34.0-SNAPSHOT~dbf06bb509, baseline=1.34.0-SNAPSHOT~5c8d55ca87
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (673.372 ms) : 0, 673372
BytebuddyAgent [candidate] (673.189 ms) : 0, 673189
GlobalTracer [baseline] (308.323 ms) : 0, 308323
GlobalTracer [candidate] (309.591 ms) : 0, 309591
AppSec [baseline] (49.433 ms) : 0, 49433
AppSec [candidate] (49.389 ms) : 0, 49389
Remote Config [baseline] (658.718 µs) : 0, 659
Remote Config [candidate] (654.558 µs) : 0, 655
Telemetry [baseline] (7.516 ms) : 0, 7516
Telemetry [candidate] (7.525 ms) : 0, 7525
section appsec
BytebuddyAgent [baseline] (698.558 ms) : 0, 698558
BytebuddyAgent [candidate] (695.979 ms) : 0, 695979
GlobalTracer [baseline] (291.538 ms) : 0, 291538
GlobalTracer [candidate] (291.645 ms) : 0, 291645
AppSec [baseline] (149.109 ms) : 0, 149109
AppSec [candidate] (149.371 ms) : 0, 149371
IAST [baseline] (19.247 ms) : 0, 19247
IAST [candidate] (19.237 ms) : 0, 19237
Remote Config [baseline] (620.455 µs) : 0, 620
Remote Config [candidate] (617.559 µs) : 0, 618
Telemetry [baseline] (7.898 ms) : 0, 7898
Telemetry [candidate] (7.168 ms) : 0, 7168
section iast
BytebuddyAgent [baseline] (802.714 ms) : 0, 802714
BytebuddyAgent [candidate] (801.345 ms) : 0, 801345
GlobalTracer [baseline] (292.658 ms) : 0, 292658
GlobalTracer [candidate] (290.39 ms) : 0, 290390
AppSec [baseline] (53.439 ms) : 0, 53439
AppSec [candidate] (49.85 ms) : 0, 49850
IAST [baseline] (22.675 ms) : 0, 22675
IAST [candidate] (24.991 ms) : 0, 24991
Remote Config [baseline] (592.64 µs) : 0, 593
Remote Config [candidate] (1.391 ms) : 0, 1391
Telemetry [baseline] (6.729 ms) : 0, 6729
Telemetry [candidate] (6.674 ms) : 0, 6674
section profiling
BytebuddyAgent [baseline] (678.156 ms) : 0, 678156
BytebuddyAgent [candidate] (678.663 ms) : 0, 678663
GlobalTracer [baseline] (380.435 ms) : 0, 380435
GlobalTracer [candidate] (380.431 ms) : 0, 380431
AppSec [baseline] (50.348 ms) : 0, 50348
AppSec [candidate] (50.23 ms) : 0, 50230
Remote Config [baseline] (706.749 µs) : 0, 707
Remote Config [candidate] (713.212 µs) : 0, 713
Telemetry [baseline] (7.5 ms) : 0, 7500
Telemetry [candidate] (7.46 ms) : 0, 7460
ProfilingAgent [baseline] (95.356 ms) : 0, 95356
ProfilingAgent [candidate] (95.85 ms) : 0, 95850
Profiling [baseline] (95.379 ms) : 0, 95379
Profiling [candidate] (95.874 ms) : 0, 95874
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.34.0-SNAPSHOT~dbf06bb509, baseline=1.34.0-SNAPSHOT~5c8d55ca87
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.074 s) : 0, 1074445
Total [baseline] (8.535 s) : 0, 8534692
Agent [candidate] (1.079 s) : 0, 1079435
Total [candidate] (8.591 s) : 0, 8590732
section iast
Agent [baseline] (1.209 s) : 0, 1209142
Total [baseline] (9.025 s) : 0, 9025152
Agent [candidate] (1.212 s) : 0, 1212285
Total [candidate] (9.006 s) : 0, 9005611
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.208 s) : 0, 1207898
Total [baseline] (8.986 s) : 0, 8985751
Agent [candidate] (1.208 s) : 0, 1208308
Total [candidate] (8.983 s) : 0, 8983439
section iast_TELEMETRY_OFF
Agent [baseline] (1.205 s) : 0, 1205159
Total [baseline] (9.005 s) : 0, 9005357
Agent [candidate] (1.198 s) : 0, 1198326
Total [candidate] (8.993 s) : 0, 8992879
gantt
title insecure-bank - break down per module: candidate=1.34.0-SNAPSHOT~dbf06bb509, baseline=1.34.0-SNAPSHOT~5c8d55ca87
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (673.184 ms) : 0, 673184
BytebuddyAgent [candidate] (675.976 ms) : 0, 675976
GlobalTracer [baseline] (309.247 ms) : 0, 309247
GlobalTracer [candidate] (311.312 ms) : 0, 311312
AppSec [baseline] (49.527 ms) : 0, 49527
AppSec [candidate] (49.411 ms) : 0, 49411
Remote Config [baseline] (656.984 µs) : 0, 657
Remote Config [candidate] (662.309 µs) : 0, 662
Telemetry [baseline] (7.529 ms) : 0, 7529
Telemetry [candidate] (7.561 ms) : 0, 7561
section iast
BytebuddyAgent [baseline] (801.094 ms) : 0, 801094
BytebuddyAgent [candidate] (802.344 ms) : 0, 802344
GlobalTracer [baseline] (290.469 ms) : 0, 290469
GlobalTracer [candidate] (292.493 ms) : 0, 292493
AppSec [baseline] (51.549 ms) : 0, 51549
AppSec [candidate] (51.912 ms) : 0, 51912
IAST [baseline] (24.27 ms) : 0, 24270
IAST [candidate] (23.516 ms) : 0, 23516
Remote Config [baseline] (577.942 µs) : 0, 578
Remote Config [candidate] (584.252 µs) : 0, 584
Telemetry [baseline] (6.609 ms) : 0, 6609
Telemetry [candidate] (6.731 ms) : 0, 6731
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (800.667 ms) : 0, 800667
BytebuddyAgent [candidate] (800.66 ms) : 0, 800660
GlobalTracer [baseline] (289.925 ms) : 0, 289925
GlobalTracer [candidate] (290.681 ms) : 0, 290681
AppSec [baseline] (49.39 ms) : 0, 49390
AppSec [candidate] (53.163 ms) : 0, 53163
IAST [baseline] (24.487 ms) : 0, 24487
IAST [candidate] (21.899 ms) : 0, 21899
Remote Config [baseline] (2.057 ms) : 0, 2057
Remote Config [candidate] (583.268 µs) : 0, 583
Telemetry [baseline] (6.769 ms) : 0, 6769
Telemetry [candidate] (6.72 ms) : 0, 6720
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (798.472 ms) : 0, 798472
BytebuddyAgent [candidate] (793.179 ms) : 0, 793179
GlobalTracer [baseline] (290.055 ms) : 0, 290055
GlobalTracer [candidate] (288.808 ms) : 0, 288808
AppSec [baseline] (50.586 ms) : 0, 50586
AppSec [candidate] (50.372 ms) : 0, 50372
IAST [baseline] (23.496 ms) : 0, 23496
IAST [candidate] (24.431 ms) : 0, 24431
Remote Config [baseline] (590.154 µs) : 0, 590
Remote Config [candidate] (582.219 µs) : 0, 582
Telemetry [baseline] (7.367 ms) : 0, 7367
Telemetry [candidate] (6.643 ms) : 0, 6643
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 insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.34.0-SNAPSHOT~dbf06bb509, baseline=1.34.0-SNAPSHOT~5c8d55ca87
dateFormat X
axisFormat %s
section baseline
no_agent (372.69 µs) : 353, 392
. : milestone, 373,
iast (476.205 µs) : 455, 497
. : milestone, 476,
iast_FULL (542.026 µs) : 521, 564
. : milestone, 542,
iast_GLOBAL (496.996 µs) : 476, 518
. : milestone, 497,
iast_HARDCODED_SECRET_DISABLED (473.872 µs) : 454, 494
. : milestone, 474,
iast_INACTIVE (448.071 µs) : 428, 468
. : milestone, 448,
iast_TELEMETRY_OFF (470.763 µs) : 449, 492
. : milestone, 471,
tracing (444.969 µs) : 425, 465
. : milestone, 445,
section candidate
no_agent (374.532 µs) : 354, 395
. : milestone, 375,
iast (472.185 µs) : 451, 493
. : milestone, 472,
iast_FULL (547.796 µs) : 526, 570
. : milestone, 548,
iast_GLOBAL (503.919 µs) : 483, 525
. : milestone, 504,
iast_HARDCODED_SECRET_DISABLED (477.605 µs) : 456, 499
. : milestone, 478,
iast_INACTIVE (458.026 µs) : 437, 480
. : milestone, 458,
iast_TELEMETRY_OFF (475.232 µs) : 454, 497
. : milestone, 475,
tracing (439.424 µs) : 419, 459
. : milestone, 439,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.34.0-SNAPSHOT~dbf06bb509, baseline=1.34.0-SNAPSHOT~5c8d55ca87
dateFormat X
axisFormat %s
section baseline
no_agent (1.352 ms) : 1332, 1371
. : milestone, 1352,
appsec (1.749 ms) : 1724, 1773
. : milestone, 1749,
appsec_no_iast (1.713 ms) : 1688, 1738
. : milestone, 1713,
iast (1.498 ms) : 1476, 1520
. : milestone, 1498,
profiling (1.5 ms) : 1475, 1524
. : milestone, 1500,
tracing (1.474 ms) : 1450, 1499
. : milestone, 1474,
section candidate
no_agent (1.341 ms) : 1323, 1360
. : milestone, 1341,
appsec (1.728 ms) : 1702, 1753
. : milestone, 1728,
appsec_no_iast (1.72 ms) : 1696, 1744
. : milestone, 1720,
iast (1.488 ms) : 1465, 1510
. : milestone, 1488,
profiling (1.5 ms) : 1476, 1525
. : milestone, 1500,
tracing (1.479 ms) : 1454, 1505
. : milestone, 1479,
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 biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.34.0-SNAPSHOT~dbf06bb509, baseline=1.34.0-SNAPSHOT~5c8d55ca87
dateFormat X
axisFormat %s
section baseline
no_agent (15.431 s) : 15431000, 15431000
. : milestone, 15431000,
appsec (15.249 s) : 15249000, 15249000
. : milestone, 15249000,
iast (18.836 s) : 18836000, 18836000
. : milestone, 18836000,
iast_GLOBAL (17.867 s) : 17867000, 17867000
. : milestone, 17867000,
profiling (15.106 s) : 15106000, 15106000
. : milestone, 15106000,
tracing (14.856 s) : 14856000, 14856000
. : milestone, 14856000,
section candidate
no_agent (15.042 s) : 15042000, 15042000
. : milestone, 15042000,
appsec (14.981 s) : 14981000, 14981000
. : milestone, 14981000,
iast (18.651 s) : 18651000, 18651000
. : milestone, 18651000,
iast_GLOBAL (17.918 s) : 17918000, 17918000
. : milestone, 17918000,
profiling (14.88 s) : 14880000, 14880000
. : milestone, 14880000,
tracing (14.886 s) : 14886000, 14886000
. : milestone, 14886000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.34.0-SNAPSHOT~dbf06bb509, baseline=1.34.0-SNAPSHOT~5c8d55ca87
dateFormat X
axisFormat %s
section baseline
no_agent (1.467 ms) : 1455, 1478
. : milestone, 1467,
appsec (2.209 ms) : 2175, 2243
. : milestone, 2209,
iast (1.887 ms) : 1852, 1923
. : milestone, 1887,
iast_GLOBAL (1.923 ms) : 1888, 1958
. : milestone, 1923,
profiling (1.848 ms) : 1815, 1882
. : milestone, 1848,
tracing (1.844 ms) : 1811, 1877
. : milestone, 1844,
section candidate
no_agent (1.465 ms) : 1454, 1477
. : milestone, 1465,
appsec (2.207 ms) : 2173, 2241
. : milestone, 2207,
iast (1.89 ms) : 1854, 1925
. : milestone, 1890,
iast_GLOBAL (1.93 ms) : 1895, 1966
. : milestone, 1930,
profiling (1.86 ms) : 1828, 1893
. : milestone, 1860,
tracing (1.831 ms) : 1799, 1862
. : milestone, 1831,
|
yiliangzhou
approved these changes
May 3, 2024
paul-laffon-dd
deleted the
paul.laffon/increase-spark-listener-max-event-size
branch
May 3, 2024 15:08
cecile75
pushed a commit
that referenced
this pull request
May 6, 2024
Increase the maximum number of retained events for the injected spark listener. Big spark clusters are reaching this limit
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
Increase the maximum number of retained events for the injected spark listener
Assuming spans are taking up to ~2kB of memory, going from 1000 to 5000 events can increase the memory impact from 2MB up to 10MB. This should be acceptable since only one listener is injected per spark driver
Motivation
Customer with some big spark clusters reaching this limit
Additional Notes