-
Notifications
You must be signed in to change notification settings - Fork 278
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
remove dependency on abstract message in schema extractor #7260
Open
piochelepiotr
wants to merge
1
commit into
master
Choose a base branch
from
piotr-wolski/data-streams-fix-crash-schema-tracking
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
remove dependency on abstract message in schema extractor #7260
piochelepiotr
wants to merge
1
commit into
master
from
piotr-wolski/data-streams-fix-crash-schema-tracking
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 49 metrics, 14 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.37.0-SNAPSHOT~fec8d9fd8e, baseline=1.37.0-SNAPSHOT~08f09790be
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.07 s) : 0, 1069727
Total [baseline] (8.548 s) : 0, 8548144
Agent [candidate] (1.074 s) : 0, 1074011
Total [candidate] (8.57 s) : 0, 8569510
section iast
Agent [baseline] (1.169 s) : 0, 1169157
Total [baseline] (9.028 s) : 0, 9028343
Agent [candidate] (1.17 s) : 0, 1169708
Total [candidate] (9.007 s) : 0, 9006540
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.169 s) : 0, 1169223
Total [baseline] (8.976 s) : 0, 8975794
Agent [candidate] (1.176 s) : 0, 1175784
Total [candidate] (8.968 s) : 0, 8968194
section iast_TELEMETRY_OFF
Agent [baseline] (1.165 s) : 0, 1164877
Total [baseline] (8.998 s) : 0, 8997649
Agent [candidate] (1.166 s) : 0, 1165953
Total [candidate] (9.002 s) : 0, 9001694
gantt
title insecure-bank - break down per module: candidate=1.37.0-SNAPSHOT~fec8d9fd8e, baseline=1.37.0-SNAPSHOT~08f09790be
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (670.045 ms) : 0, 670045
BytebuddyAgent [candidate] (672.212 ms) : 0, 672212
GlobalTracer [baseline] (306.269 ms) : 0, 306269
GlobalTracer [candidate] (307.891 ms) : 0, 307891
AppSec [baseline] (50.313 ms) : 0, 50313
AppSec [candidate] (50.762 ms) : 0, 50762
Remote Config [baseline] (708.094 µs) : 0, 708
Remote Config [candidate] (687.666 µs) : 0, 688
Telemetry [baseline] (7.679 ms) : 0, 7679
Telemetry [candidate] (7.764 ms) : 0, 7764
section iast
BytebuddyAgent [baseline] (778.29 ms) : 0, 778290
BytebuddyAgent [candidate] (779.111 ms) : 0, 779111
GlobalTracer [baseline] (293.22 ms) : 0, 293220
GlobalTracer [candidate] (294.404 ms) : 0, 294404
AppSec [baseline] (47.3 ms) : 0, 47300
AppSec [candidate] (47.259 ms) : 0, 47259
Remote Config [baseline] (588.492 µs) : 0, 588
Remote Config [candidate] (597.487 µs) : 0, 597
Telemetry [baseline] (6.922 ms) : 0, 6922
Telemetry [candidate] (6.945 ms) : 0, 6945
IAST [baseline] (29.46 ms) : 0, 29460
IAST [candidate] (27.963 ms) : 0, 27963
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (778.586 ms) : 0, 778586
BytebuddyAgent [candidate] (782.09 ms) : 0, 782090
GlobalTracer [baseline] (294.005 ms) : 0, 294005
GlobalTracer [candidate] (295.629 ms) : 0, 295629
AppSec [baseline] (47.423 ms) : 0, 47423
AppSec [candidate] (47.582 ms) : 0, 47582
Remote Config [baseline] (627.749 µs) : 0, 628
Remote Config [candidate] (626.503 µs) : 0, 627
Telemetry [baseline] (7.777 ms) : 0, 7777
Telemetry [candidate] (7.059 ms) : 0, 7059
IAST [baseline] (27.385 ms) : 0, 27385
IAST [candidate] (29.3 ms) : 0, 29300
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (776.035 ms) : 0, 776035
BytebuddyAgent [candidate] (775.61 ms) : 0, 775610
GlobalTracer [baseline] (294.293 ms) : 0, 294293
GlobalTracer [candidate] (293.546 ms) : 0, 293546
AppSec [baseline] (46.899 ms) : 0, 46899
AppSec [candidate] (47.331 ms) : 0, 47331
Remote Config [baseline] (573.885 µs) : 0, 574
Remote Config [candidate] (584.761 µs) : 0, 585
Telemetry [baseline] (7.585 ms) : 0, 7585
Telemetry [candidate] (8.364 ms) : 0, 8364
IAST [baseline] (26.075 ms) : 0, 26075
IAST [candidate] (27.085 ms) : 0, 27085
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.37.0-SNAPSHOT~fec8d9fd8e, baseline=1.37.0-SNAPSHOT~08f09790be
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.064 s) : 0, 1064162
Total [baseline] (10.379 s) : 0, 10379469
Agent [candidate] (1.062 s) : 0, 1061981
Total [candidate] (10.385 s) : 0, 10385492
section appsec
Agent [baseline] (1.184 s) : 0, 1184231
Total [baseline] (10.47 s) : 0, 10470327
Agent [candidate] (1.189 s) : 0, 1189132
Total [candidate] (10.454 s) : 0, 10453750
section iast
Agent [baseline] (1.17 s) : 0, 1169523
Total [baseline] (10.727 s) : 0, 10727225
Agent [candidate] (1.19 s) : 0, 1189869
Total [candidate] (10.785 s) : 0, 10784967
section profiling
Agent [baseline] (1.262 s) : 0, 1261661
Total [baseline] (10.588 s) : 0, 10588259
Agent [candidate] (1.263 s) : 0, 1263441
Total [candidate] (10.614 s) : 0, 10614127
gantt
title petclinic - break down per module: candidate=1.37.0-SNAPSHOT~fec8d9fd8e, baseline=1.37.0-SNAPSHOT~08f09790be
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (666.766 ms) : 0, 666766
BytebuddyAgent [candidate] (664.707 ms) : 0, 664707
GlobalTracer [baseline] (304.47 ms) : 0, 304470
GlobalTracer [candidate] (304.34 ms) : 0, 304340
AppSec [baseline] (50.073 ms) : 0, 50073
AppSec [candidate] (50.189 ms) : 0, 50189
Remote Config [baseline] (768.181 µs) : 0, 768
Remote Config [candidate] (714.756 µs) : 0, 715
Telemetry [baseline] (7.505 ms) : 0, 7505
Telemetry [candidate] (7.526 ms) : 0, 7526
section appsec
BytebuddyAgent [baseline] (676.947 ms) : 0, 676947
BytebuddyAgent [candidate] (680.337 ms) : 0, 680337
GlobalTracer [baseline] (297.876 ms) : 0, 297876
GlobalTracer [candidate] (299.326 ms) : 0, 299326
AppSec [baseline] (153.953 ms) : 0, 153953
AppSec [candidate] (154.409 ms) : 0, 154409
Remote Config [baseline] (628.064 µs) : 0, 628
Remote Config [candidate] (630.681 µs) : 0, 631
Telemetry [baseline] (9.011 ms) : 0, 9011
Telemetry [candidate] (8.348 ms) : 0, 8348
IAST [baseline] (22.209 ms) : 0, 22209
IAST [candidate] (22.546 ms) : 0, 22546
section iast
BytebuddyAgent [baseline] (778.829 ms) : 0, 778829
BytebuddyAgent [candidate] (792.052 ms) : 0, 792052
GlobalTracer [baseline] (294.107 ms) : 0, 294107
GlobalTracer [candidate] (299.363 ms) : 0, 299363
AppSec [baseline] (47.263 ms) : 0, 47263
AppSec [candidate] (48.266 ms) : 0, 48266
Remote Config [baseline] (616.787 µs) : 0, 617
Remote Config [candidate] (608.83 µs) : 0, 609
Telemetry [baseline] (7.748 ms) : 0, 7748
Telemetry [candidate] (7.099 ms) : 0, 7099
IAST [baseline] (27.54 ms) : 0, 27540
IAST [candidate] (28.848 ms) : 0, 28848
section profiling
BytebuddyAgent [baseline] (661.658 ms) : 0, 661658
BytebuddyAgent [candidate] (662.699 ms) : 0, 662699
GlobalTracer [baseline] (387.361 ms) : 0, 387361
GlobalTracer [candidate] (387.705 ms) : 0, 387705
AppSec [baseline] (51.727 ms) : 0, 51727
AppSec [candidate] (51.702 ms) : 0, 51702
Remote Config [baseline] (663.933 µs) : 0, 664
Remote Config [candidate] (660.376 µs) : 0, 660
Telemetry [baseline] (7.368 ms) : 0, 7368
Telemetry [candidate] (7.274 ms) : 0, 7274
ProfilingAgent [baseline] (95.814 ms) : 0, 95814
ProfilingAgent [candidate] (96.25 ms) : 0, 96250
Profiling [baseline] (95.84 ms) : 0, 95840
Profiling [candidate] (96.275 ms) : 0, 96275
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 17 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.37.0-SNAPSHOT~fec8d9fd8e, baseline=1.37.0-SNAPSHOT~08f09790be
dateFormat X
axisFormat %s
section baseline
no_agent (366.944 µs) : 347, 387
. : milestone, 367,
iast (483.899 µs) : 463, 505
. : milestone, 484,
iast_FULL (547.999 µs) : 527, 569
. : milestone, 548,
iast_GLOBAL (505.913 µs) : 484, 528
. : milestone, 506,
iast_HARDCODED_SECRET_DISABLED (476.051 µs) : 455, 497
. : milestone, 476,
iast_INACTIVE (450.836 µs) : 430, 472
. : milestone, 451,
iast_TELEMETRY_OFF (466.843 µs) : 445, 488
. : milestone, 467,
tracing (436.393 µs) : 416, 457
. : milestone, 436,
section candidate
no_agent (363.522 µs) : 344, 383
. : milestone, 364,
iast (481.741 µs) : 461, 503
. : milestone, 482,
iast_FULL (548.342 µs) : 527, 570
. : milestone, 548,
iast_GLOBAL (498.58 µs) : 478, 520
. : milestone, 499,
iast_HARDCODED_SECRET_DISABLED (480.253 µs) : 459, 502
. : milestone, 480,
iast_INACTIVE (446.832 µs) : 426, 468
. : milestone, 447,
iast_TELEMETRY_OFF (465.589 µs) : 444, 487
. : milestone, 466,
tracing (441.549 µs) : 421, 462
. : milestone, 442,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.37.0-SNAPSHOT~fec8d9fd8e, baseline=1.37.0-SNAPSHOT~08f09790be
dateFormat X
axisFormat %s
section baseline
no_agent (1.346 ms) : 1327, 1365
. : milestone, 1346,
appsec (1.704 ms) : 1679, 1730
. : milestone, 1704,
appsec_no_iast (1.734 ms) : 1709, 1758
. : milestone, 1734,
iast (1.468 ms) : 1445, 1491
. : milestone, 1468,
profiling (1.47 ms) : 1446, 1494
. : milestone, 1470,
tracing (1.452 ms) : 1427, 1476
. : milestone, 1452,
section candidate
no_agent (1.354 ms) : 1334, 1375
. : milestone, 1354,
appsec (1.722 ms) : 1697, 1746
. : milestone, 1722,
appsec_no_iast (1.709 ms) : 1685, 1734
. : milestone, 1709,
iast (1.486 ms) : 1463, 1509
. : milestone, 1486,
profiling (1.481 ms) : 1457, 1506
. : milestone, 1481,
tracing (1.439 ms) : 1415, 1463
. : milestone, 1439,
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.37.0-SNAPSHOT~fec8d9fd8e, baseline=1.37.0-SNAPSHOT~08f09790be
dateFormat X
axisFormat %s
section baseline
no_agent (1.459 ms) : 1447, 1470
. : milestone, 1459,
appsec (2.202 ms) : 2168, 2236
. : milestone, 2202,
iast (1.966 ms) : 1924, 2008
. : milestone, 1966,
iast_GLOBAL (2.004 ms) : 1962, 2045
. : milestone, 2004,
profiling (1.857 ms) : 1823, 1891
. : milestone, 1857,
tracing (1.827 ms) : 1795, 1859
. : milestone, 1827,
section candidate
no_agent (1.46 ms) : 1448, 1471
. : milestone, 1460,
appsec (2.207 ms) : 2172, 2242
. : milestone, 2207,
iast (1.956 ms) : 1915, 1997
. : milestone, 1956,
iast_GLOBAL (2.011 ms) : 1970, 2053
. : milestone, 2011,
profiling (2.307 ms) : 2132, 2482
. : milestone, 2307,
tracing (1.826 ms) : 1794, 1857
. : milestone, 1826,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.37.0-SNAPSHOT~fec8d9fd8e, baseline=1.37.0-SNAPSHOT~08f09790be
dateFormat X
axisFormat %s
section baseline
no_agent (15.535 s) : 15535000, 15535000
. : milestone, 15535000,
appsec (14.953 s) : 14953000, 14953000
. : milestone, 14953000,
iast (19.032 s) : 19032000, 19032000
. : milestone, 19032000,
iast_GLOBAL (17.787 s) : 17787000, 17787000
. : milestone, 17787000,
profiling (15.8 s) : 15800000, 15800000
. : milestone, 15800000,
tracing (14.848 s) : 14848000, 14848000
. : milestone, 14848000,
section candidate
no_agent (15.002 s) : 15002000, 15002000
. : milestone, 15002000,
appsec (15.084 s) : 15084000, 15084000
. : milestone, 15084000,
iast (18.78 s) : 18780000, 18780000
. : milestone, 18780000,
iast_GLOBAL (17.927 s) : 17927000, 17927000
. : milestone, 17927000,
profiling (15.266 s) : 15266000, 15266000
. : milestone, 15266000,
tracing (14.918 s) : 14918000, 14918000
. : milestone, 14918000,
|
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
Motivation
Trying to solve this error:
My theory is that the dynamic message instrumentation calls this class, but dynamic message has nothing to do with
AbstractMessage
, so maybe in some cases DynamicMessage exists, but not AbstractMessage?But that doesn't seem to be 100% correlated with the error.
Additional Notes
I'm not confident this will work. Since the error says the whole
protobuf
module is not exported.But I only instrument classes extending protobuf classes, so I don't understand how that can happen.
Jira ticket: [PROJ-IDENT]