-
Notifications
You must be signed in to change notification settings - Fork 306
Cleanup of zstd dependency and jps functionality #9010
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
BenchmarksStartupParameters
See matching parameters
SummaryFound 2 performance improvements and 1 performance regressions! Performance is the same for 39 metrics, 11 unstable metrics.
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.50.0-SNAPSHOT~48ab511815, baseline=1.50.0-SNAPSHOT~e0a1fc0e75
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.031 s) : 0, 1031186
Total [baseline] (8.574 s) : 0, 8573630
Agent [candidate] (1.026 s) : 0, 1026108
Total [candidate] (8.541 s) : 0, 8540785
section iast
Agent [baseline] (1.15 s) : 0, 1149746
Total [baseline] (9.244 s) : 0, 9243877
Agent [candidate] (1.16 s) : 0, 1159588
Total [candidate] (9.209 s) : 0, 9208687
gantt
title insecure-bank - break down per module: candidate=1.50.0-SNAPSHOT~48ab511815, baseline=1.50.0-SNAPSHOT~e0a1fc0e75
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (688.937 ms) : 0, 688937
BytebuddyAgent [candidate] (686.885 ms) : 0, 686885
GlobalTracer [baseline] (242.169 ms) : 0, 242169
GlobalTracer [candidate] (242.593 ms) : 0, 242593
AppSec [baseline] (58.383 ms) : 0, 58383
AppSec [candidate] (56.76 ms) : 0, 56760
Debugger [baseline] (6.938 ms) : 0, 6938
Debugger [candidate] (8.616 ms) : 0, 8616
Remote Config [baseline] (738.885 µs) : 0, 739
Remote Config [candidate] (674.706 µs) : 0, 675
Telemetry [baseline] (10.463 ms) : 0, 10463
Telemetry [candidate] (9.644 ms) : 0, 9644
section iast
BytebuddyAgent [baseline] (802.369 ms) : 0, 802369
BytebuddyAgent [candidate] (812.235 ms) : 0, 812235
GlobalTracer [baseline] (230.667 ms) : 0, 230667
GlobalTracer [candidate] (232.831 ms) : 0, 232831
IAST [baseline] (27.656 ms) : 0, 27656
IAST [candidate] (26.187 ms) : 0, 26187
AppSec [baseline] (51.183 ms) : 0, 51183
AppSec [candidate] (52.807 ms) : 0, 52807
Debugger [baseline] (5.947 ms) : 0, 5947
Debugger [candidate] (5.944 ms) : 0, 5944
Remote Config [baseline] (590.755 µs) : 0, 591
Remote Config [candidate] (616.732 µs) : 0, 617
Telemetry [baseline] (7.954 ms) : 0, 7954
Telemetry [candidate] (8.093 ms) : 0, 8093
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.50.0-SNAPSHOT~48ab511815, baseline=1.50.0-SNAPSHOT~e0a1fc0e75
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.028 s) : 0, 1027802
Total [baseline] (10.57 s) : 0, 10570131
Agent [candidate] (1.026 s) : 0, 1026243
Total [candidate] (10.551 s) : 0, 10551349
section appsec
Agent [baseline] (1.185 s) : 0, 1184941
Total [baseline] (10.74 s) : 0, 10739839
Agent [candidate] (1.176 s) : 0, 1176076
Total [candidate] (10.724 s) : 0, 10724304
section iast
Agent [baseline] (1.159 s) : 0, 1159327
Total [baseline] (10.934 s) : 0, 10934160
Agent [candidate] (1.16 s) : 0, 1160373
Total [candidate] (10.909 s) : 0, 10909063
section profiling
Agent [baseline] (1.264 s) : 0, 1264283
Total [baseline] (10.947 s) : 0, 10946649
Agent [candidate] (1.264 s) : 0, 1263839
Total [candidate] (10.949 s) : 0, 10949105
gantt
title petclinic - break down per module: candidate=1.50.0-SNAPSHOT~48ab511815, baseline=1.50.0-SNAPSHOT~e0a1fc0e75
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (687.625 ms) : 0, 687625
BytebuddyAgent [candidate] (687.34 ms) : 0, 687340
GlobalTracer [baseline] (241.157 ms) : 0, 241157
GlobalTracer [candidate] (243.129 ms) : 0, 243129
AppSec [baseline] (57.595 ms) : 0, 57595
AppSec [candidate] (58.982 ms) : 0, 58982
Debugger [baseline] (6.824 ms) : 0, 6824
Debugger [candidate] (6.284 ms) : 0, 6284
Remote Config [baseline] (724.573 µs) : 0, 725
Remote Config [candidate] (667.593 µs) : 0, 668
Telemetry [baseline] (10.399 ms) : 0, 10399
Telemetry [candidate] (8.968 ms) : 0, 8968
section appsec
BytebuddyAgent [baseline] (712.09 ms) : 0, 712090
BytebuddyAgent [candidate] (708.979 ms) : 0, 708979
GlobalTracer [baseline] (236.644 ms) : 0, 236644
GlobalTracer [candidate] (234.943 ms) : 0, 234943
IAST [baseline] (22.007 ms) : 0, 22007
IAST [candidate] (21.863 ms) : 0, 21863
AppSec [baseline] (176.691 ms) : 0, 176691
AppSec [candidate] (175.795 ms) : 0, 175795
Debugger [baseline] (5.911 ms) : 0, 5911
Debugger [candidate] (5.907 ms) : 0, 5907
Remote Config [baseline] (624.45 µs) : 0, 624
Remote Config [candidate] (616.471 µs) : 0, 616
Telemetry [baseline] (7.383 ms) : 0, 7383
Telemetry [candidate] (7.24 ms) : 0, 7240
section iast
BytebuddyAgent [baseline] (808.813 ms) : 0, 808813
BytebuddyAgent [candidate] (810.858 ms) : 0, 810858
GlobalTracer [baseline] (232.428 ms) : 0, 232428
GlobalTracer [candidate] (233.838 ms) : 0, 233838
IAST [baseline] (28.764 ms) : 0, 28764
IAST [candidate] (25.594 ms) : 0, 25594
AppSec [baseline] (51.029 ms) : 0, 51029
AppSec [candidate] (54.393 ms) : 0, 54393
Debugger [baseline] (6.042 ms) : 0, 6042
Debugger [candidate] (5.993 ms) : 0, 5993
Remote Config [baseline] (612.128 µs) : 0, 612
Remote Config [candidate] (643.209 µs) : 0, 643
Telemetry [baseline] (8.023 ms) : 0, 8023
Telemetry [candidate] (8.078 ms) : 0, 8078
section profiling
BytebuddyAgent [baseline] (673.943 ms) : 0, 673943
BytebuddyAgent [candidate] (674.601 ms) : 0, 674601
GlobalTracer [baseline] (360.233 ms) : 0, 360233
GlobalTracer [candidate] (359.355 ms) : 0, 359355
AppSec [baseline] (61.778 ms) : 0, 61778
AppSec [candidate] (62.169 ms) : 0, 62169
Debugger [baseline] (6.152 ms) : 0, 6152
Debugger [candidate] (6.112 ms) : 0, 6112
Remote Config [baseline] (641.421 µs) : 0, 641
Remote Config [candidate] (711.441 µs) : 0, 711
Telemetry [baseline] (8.161 ms) : 0, 8161
Telemetry [candidate] (8.099 ms) : 0, 8099
ProfilingAgent [baseline] (102.69 ms) : 0, 102690
ProfilingAgent [candidate] (104.37 ms) : 0, 104370
Profiling [baseline] (102.715 ms) : 0, 102715
Profiling [candidate] (104.395 ms)
8000
: 0, 104395
LoadParameters
See matching parameters
SummaryFound 3 performance improvements and 0 performance regressions! Performance is the same for 9 metrics, 12 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.50.0-SNAPSHOT~48ab511815, baseline=1.50.0-SNAPSHOT~e0a1fc0e75
dateFormat X
axisFormat %s
section baseline
no_agent (4.53 ms) : 4480, 4581
. : milestone, 4530,
iast (9.571 ms) : 9412, 9730
. : milestone, 9571,
iast_FULL (13.757 ms) : 13486, 14029
. : milestone, 13757,
iast_GLOBAL (10.423 ms) : 10239, 10607
. : milestone, 10423,
profiling (8.78 ms) : 8625, 8936
. : milestone, 8780,
tracing (7.37 ms) : 7261, 7478
. : milestone, 7370,
section candidate
no_agent (4.364 ms) : 4315, 4413
. : milestone, 4364,
iast (9.247 ms) : 9093, 9402
. : milestone, 9247,
iast_FULL (13.958 ms) : 13676, 14240
. : milestone, 13958,
iast_GLOBAL (10.418 ms) : 10236, 10601
. : milestone, 10418,
profiling (9.019 ms) : 8879, 9160
. : milestone, 9019,
tracing (7.586 ms) : 7481, 7691
. : milestone, 7586,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.50.0-SNAPSHOT~48ab511815, baseline=1.50.0-SNAPSHOT~e0a1fc0e75
dateFormat X
axisFormat %s
section baseline
no_agent (36.776 ms) : 36472, 37081
. : milestone, 36776,
appsec (46.14 ms) : 45734, 46545
. : milestone, 46140,
code_origins (44.332 ms) : 43980, 44685
. : milestone, 44332,
iast (43.288 ms) : 42923, 43653
. : milestone, 43288,
profiling (49.097 ms) : 48641, 49552
. : milestone, 49097,
tracing (45.889 ms) : 45495, 46282
. : milestone, 45889,
section candidate
no_agent (36.786 ms) : 36487, 37085
. : milestone, 36786,
appsec (44.05 ms) : 43673, 44426
. : milestone, 44050,
code_origins (44.223 ms) : 43878, 44569
. : milestone, 44223,
iast (43.944 ms) : 43573, 44314
. : milestone, 43944,
profiling (49.172 ms) : 48727, 49617
. : milestone, 49172,
tracing (44.07 ms) : 43696, 44443
. : milestone, 44070,
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.50.0-SNAPSHOT~48ab511815, baseline=1.50.0-SNAPSHOT~e0a1fc0e75
dateFormat X
axisFormat %s
section baseline
no_agent (1.475 ms) : 1463, 1486
. : milestone, 1475,
appsec (2.394 ms) : 2346, 2443
. : milestone, 2394,
iast (2.18 ms) : 2118, 2241
. : milestone, 2180,
iast_GLOBAL (2.233 ms) : 2171, 2294
. : milestone, 2233,
profiling (2.021 ms) : 1972, 2070
. : milestone, 2021,
tracing (2.004 ms) : 1956, 2051
. : milestone, 2004,
section candidate
no_agent (1.473 ms) : 1462, 1485
. : milestone, 1473,
appsec (2.399 ms) : 2350, 2447
. : milestone, 2399,
iast (2.185 ms) : 2124, 2246
. : milestone, 2185,
iast_GLOBAL (2.226 ms) : 2165, 2288
. : milestone, 2226,
profiling (2.039 ms) : 1989, 2089
. : milestone, 2039,
tracing (2.006 ms) : 1958, 2054
. : milestone, 2006,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.50.0-SNAPSHOT~48ab511815, baseline=1.50.0-SNAPSHOT~e0a1fc0e75
dateFormat X
axisFormat %s
section baseline
no_agent (15.547 s) : 15547000, 15547000
. : milestone, 15547000,
appsec (15.149 s) : 15149000, 15149000
. : milestone, 15149000,
iast (18.581 s) : 18581000, 18581000
. : milestone, 18581000,
iast_GLOBAL (17.947 s) : 17947000, 17947000
. : milestone, 17947000,
profiling (15.126 s) : 15126000, 15126000
. : milestone, 15126000,
tracing (14.839 s) : 14839000, 14839000
. : milestone, 14839000,
section candidate
no_agent (15.589 s) : 15589000, 15589000
. : milestone, 15589000,
appsec (14.732 s) : 14732000, 14732000
. : milestone, 14732000,
iast (18.594 s) : 18594000, 18594000
. : milestone, 18594000,
iast_GLOBAL (18.046 s) : 18046000, 18046000
. : milestone, 18046000,
profiling (15.121 s) : 15121000, 15121000
. : milestone, 15121000,
tracing (14.981 s) : 14981000, 14981000
. : milestone, 14981000,
|
2868fa3
to
2587735
Compare
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.
LGTM
exclude '**/io/airlift/compress/deflate/**' | ||
exclude '**/io/airlift/compress/gzip/**' | ||
exclude '**/io/airlift/compress/hadoop/**' | ||
exclude '**/io/airlift/compress/lz4/**' |
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.
question: Out of curiosity is there a reason to prefer lz4-java over aircompressor lz4 ?
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.
Not really, it's just that we know lz4-java works and it didn't feel good to do the change as a part of last minute cleanup PR :)
But definitely we should followup o
8000
n this - it does not make sense pulling in two different libraries when we can handle the supported compression formats by a single one.
} | ||
} catch (Exception e) { | ||
log.debug("Unable to list java processes via 'jps'", e); | ||
log.debug("Unable to obtain Java PIDs", e); |
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.
suggestion: Might be worth to keep mentioning the source of the pid i.e. hsperfdata, so support can look why this didn't work.
Thanks! |
What Does This Do
This cleans up the aircompressor dependency not to pull in all possible compressor implementations when we are interested only in zstd.
It also does minor cleanup in the jps-like helper where it does not make sense to use internal JVM classes via reflection when we already have a file based approach working.
Motivation
Reduce the agent size, cleanup the jps utility implementation.
Additional Notes