-
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
make tasks wrapped by grpc-context unwrappable #6977
Merged
Merged
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 petclinicgantt
title petclinic - global startup overhead: candidate=1.34.0-SNAPSHOT~9955cc8f3d, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.084 s) : 0, 1083920
Total [baseline] (10.457 s) : 0, 10456614
Agent [candidate] (1.076 s) : 0, 1076297
Total [candidate] (10.419 s) : 0, 10418798
section appsec
Agent [baseline] (1.191 s) : 0, 1190820
Total [baseline] (10.497 s) : 0, 10497425
Agent [candidate] (1.187 s) : 0, 1187221
Total [candidate] (10.488 s) : 0, 10488340
section iast
Agent [baseline] (1.199 s) : 0, 1198873
Total [baseline] (10.743 s) : 0, 10743200
Agent [candidate] (1.207 s) : 0, 1207183
Total [candidate] (10.758 s) : 0, 10757550
section profiling
Agent [baseline] (1.283 s) : 0, 1282903
Total [baseline] (10.592 s) : 0, 10592118
Agent [candidate] (1.266 s) : 0, 1265823
Total [candidate] (10.615 s) : 0, 10615351
gantt
title petclinic - break down per module: candidate=1.34.0-SNAPSHOT~9955cc8f3d, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (678.917 ms) : 0, 678917
BytebuddyAgent [candidate] (673.958 ms) : 0, 673958
GlobalTracer [baseline] (312.054 ms) : 0, 312054
GlobalTracer [candidate] (310.232 ms) : 0, 310232
AppSec [baseline] (49.848 ms) : 0, 49848
AppSec [candidate] (49.445 ms) : 0, 49445
Remote Config [baseline] (670.723 µs) : 0, 671
Remote Config [candidate] (656.304 µs) : 0, 656
Telemetry [baseline] (7.703 ms) : 0, 7703
Telemetry [candidate] (7.51 ms) : 0, 7510
section appsec
BytebuddyAgent [baseline] (697.06 ms) : 0, 697060
BytebuddyAgent [candidate] (694.859 ms) : 0, 694859
GlobalTracer [baseline] (291.464 ms) : 0, 291464
GlobalTracer [candidate] (291.768 ms) : 0, 291768
AppSec [baseline] (149.243 ms) : 0, 149243
AppSec [candidate] (148.78 ms) : 0, 148780
IAST [baseline] (19.137 ms) : 0, 19137
IAST [candidate] (19.313 ms) : 0, 19313
Remote Config [baseline] (624.637 µs) : 0, 625
Remote Config [candidate] (619.432 µs) : 0, 619
Telemetry [baseline] (8.459 ms) : 0, 8459
Telemetry [candidate] (7.224 ms) : 0, 7224
section iast
BytebuddyAgent [baseline] (795.003 ms) : 0, 795003
BytebuddyAgent [candidate] (799.772 ms) : 0, 799772
GlobalTracer [baseline] (288.517 ms) : 0, 288517
GlobalTracer [candidate] (290.352 ms) : 0, 290352
AppSec [baseline] (48.885 ms) : 0, 48885
AppSec [candidate] (50.611 ms) : 0, 50611
IAST [baseline] (23.384 ms) : 0, 23384
IAST [candidate] (23.514 ms) : 0, 23514
Remote Config [baseline] (2.099 ms) : 0, 2099
Remote Config [candidate] (1.977 ms) : 0, 1977
Telemetry [baseline] (6.72 ms) : 0, 6720
Telemetry [candidate] (6.586 ms) : 0, 6586
section profiling
BytebuddyAgent [baseline] (686.456 ms) : 0, 686456
BytebuddyAgent [candidate] (676.71 ms) : 0, 676710
GlobalTracer [baseline] (383.964 ms) : 0, 383964
GlobalTracer [candidate] (379.304 ms) : 0, 379304
AppSec [baseline] (50.446 ms) : 0, 50446
AppSec [candidate] (50.006 ms) : 0, 50006
Remote Config [baseline] (727.951 µs) : 0, 728
Remote Config [candidate] (698.302 µs) : 0, 698
Telemetry [baseline] (7.613 ms) : 0, 7613
Telemetry [candidate] (7.477 ms) : 0, 7477
ProfilingAgent [baseline] (96.414 ms) : 0, 96414
ProfilingAgent [candidate] (95.458 ms) : 0, 95458
Profiling [baseline] (96.438 ms) : 0, 96438
Profiling [candidate] (95.482 ms) : 0, 95482
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.34.0-SNAPSHOT~9955cc8f3d, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.076 s) : 0, 1075725
Total [baseline] (8.567 s) : 0, 8567299
Agent [candidate] (1.076 s) : 0, 1076250
Total [candidate] (8.541 s) : 0, 8541197
section iast
Agent [baseline] (1.196 s) : 0, 1196093
Total [baseline] (9.017 s) : 0, 9017309
Agent [candidate] (1.198 s) : 0, 1197607
Total [candidate] (8.988 s) : 0, 8987782
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.201 s) : 0, 1201380
Total [baseline] (8.988 s) : 0, 8988189
Agent [candidate] (1.21 s) : 0, 1210169
Total [candidate] (9.03 s) : 0, 9030030
section iast_TELEMETRY_OFF
Agent [baseline] (1.205 s) : 0, 1204614
Total [baseline] (9.035 s) : 0, 9034696
Agent [candidate] (1.205 s) : 0, 1204668
Total [candidate] (9.087 s) : 0, 9087267
gantt
title insecure-bank - break down per module: candidate=1.34.0-SNAPSHOT~9955cc8f3d, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (673.515 ms) : 0, 673515
BytebuddyAgent [candidate] (674.87 ms) : 0, 674870
GlobalTracer [baseline] (309.83 ms) : 0, 309830
GlobalTracer [candidate] (309.138 ms) : 0, 309138
AppSec [baseline] (49.65 ms) : 0, 49650
AppSec [candidate] (49.408 ms) : 0, 49408
Remote Config [baseline] (663.471 µs) : 0, 663
Remote Config [candidate] (661.569 µs) : 0, 662
Telemetry [baseline] (7.638 ms) : 0, 7638
Telemetry [candidate] (7.585 ms) : 0, 7585
section iast
BytebuddyAgent [baseline] (792.188 ms) : 0, 792188
BytebuddyAgent [candidate] (793.489 ms) : 0, 793489
GlobalTracer [baseline] (287.684 ms) : 0, 287684
GlobalTracer [candidate] (287.683 ms) : 0, 287683
AppSec [baseline] (50.938 ms) : 0, 50938
AppSec [candidate] (50.036 ms) : 0, 50036
IAST [baseline] (23.952 ms) : 0, 23952
IAST [candidate] (24.953 ms) : 0, 24953
Remote Config [baseline] (587.038 µs) : 0, 587
Remote Config [candidate] (668.323 µs) : 0, 668
Telemetry [baseline] (6.604 ms) : 0, 6604
Telemetry [candidate] (6.604 ms) : 0, 6604
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (794.409 ms) : 0, 794409
BytebuddyAgent [candidate] (801.39 ms) : 0, 801390
GlobalTracer [baseline] (288.993 ms) : 0, 288993
GlobalTracer [candidate] (290.829 ms) : 0, 290829
AppSec [baseline] (50.228 ms) : 0, 50228
AppSec [candidate] (52.247 ms) : 0, 52247
IAST [baseline] (23.079 ms) : 0, 23079
IAST [candidate] (22.143 ms) : 0, 22143
Remote Config [baseline] (2.813 ms) : 0, 2813
Remote Config [candidate] (2.075 ms) : 0, 2075
Telemetry [baseline] (7.556 ms) : 0, 7556
Telemetry [candidate] (6.815 ms) : 0, 6815
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (797.6 ms) : 0, 797600
BytebuddyAgent [candidate] (797.658 ms) : 0, 797658
GlobalTracer [baseline] (290.02 ms) : 0, 290020
GlobalTracer [candidate] (289.816 ms) : 0, 289816
AppSec [baseline] (52.12 ms) : 0, 52120
AppSec [candidate] (50.63 ms) : 0, 50630
IAST [baseline] (22.27 ms) : 0, 22270
IAST [candidate] (23.982 ms) : 0, 23982
Remote Config [baseline] (606.846 µs) : 0, 607
Remote Config [candidate] (589.56 µs) : 0, 590
Telemetry [baseline] (7.403 ms) : 0, 7403
Telemetry [candidate] (7.422 ms) : 0, 7422
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~9955cc8f3d, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section baseline
no_agent (383.992 µs) : 364, 404
. : milestone, 384,
iast (486.333 µs) : 465, 507
. : milestone, 486,
iast_FULL (551.145 µs) : 531, 572
. : milestone, 551,
iast_GLOBAL (508.357 µs) : 487, 530
. : milestone, 508,
iast_HARDCODED_SECRET_DISABLED (478.573 µs) : 458, 499
. : milestone, 479,
iast_INACTIVE (459.328 µs) : 438, 480
. : milestone, 459,
iast_TELEMETRY_OFF (484.104 µs) : 463, 505
. : milestone, 484,
tracing (462.271 µs) : 441, 484
. : milestone, 462,
section candidate
no_agent (376.327 µs) : 357, 396
. : milestone, 376,
iast (484.269 µs) : 463, 505
. : milestone, 484,
iast_FULL (555.197 µs) : 534, 577
. : milestone, 555,
iast_GLOBAL (502.591 µs) : 482, 523
. : milestone, 503,
iast_HARDCODED_SECRET_DISABLED (485.885 µs) : 465, 507
. : milestone, 486,
iast_INACTIVE (458.598 µs) : 438, 479
. : milestone, 459,
iast_TELEMETRY_OFF (477.91 µs) : 457, 499
. : milestone, 478,
tracing (461.172 µs) : 439, 483
. : milestone, 461,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.34.0-SNAPSHOT~9955cc8f3d, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section baseline
no_agent (1.344 ms) : 1325, 1363
. : milestone, 1344,
appsec (1.729 ms) : 1705, 1753
. : milestone, 1729,
appsec_no_iast (1.713 ms) : 1688, 1738
. : milestone, 1713,
iast (1.514 ms) : 1492, 1537
. : milestone, 1514,
profiling (1.54 ms) : 1514, 1566
. : milestone, 1540,
tracing (1.495 ms) : 1471, 1520
. : milestone, 1495,
section candidate
no_agent (1.347 ms) : 1328, 1367
. : milestone, 1347,
appsec (1.748 ms) : 1725, 1772
. : milestone, 1748,
appsec_no_iast (1.749 ms) : 1724, 1773
. : milestone, 1749,
iast (1.496 ms) : 1473, 1519
. : milestone, 1496,
profiling (1.532 ms) : 1506, 1557
. : milestone, 1532,
tracing (1.504 ms) : 1479, 1528
. : milestone, 1504,
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.34.0-SNAPSHOT~9955cc8f3d, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section baseline
no_agent (1.456 ms) : 1445, 1468
. : milestone, 1456,
appsec (2.194 ms) : 2160, 2228
. : milestone, 2194,
iast (1.868 ms) : 1833, 1903
. : milestone, 1868,
iast_GLOBAL (1.899 ms) : 1864, 1933
. : milestone, 1899,
profiling (1.826 ms) : 1793, 1860
. : milestone, 1826,
tracing (1.815 ms) : 1784, 1847
. : milestone, 1815,
section candidate
no_agent (1.449 ms) : 1438, 1460
. : milestone, 1449,
appsec (2.19 ms) : 2157, 2224
. : milestone, 2190,
iast (1.875 ms) : 1840, 1910
. : milestone, 1875,
iast_GLOBAL (1.909 ms) : 1874, 1944
. : milestone, 1909,
profiling (1.834 ms) : 1800, 1868
. : milestone, 1834,
tracing (1.82 ms) : 1788, 1852
. : milestone, 1820,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.34.0-SNAPSHOT~9955cc8f3d, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section baseline
no_agent (15.291 s) : 15291000, 15291000
. : milestone, 15291000,
appsec (15.244 s) : 15244000, 15244000
. : milestone, 15244000,
iast (18.775 s) : 18775000, 18775000
. : milestone, 18775000,
iast_GLOBAL (18.012 s) : 18012000, 18012000
. : milestone, 18012000,
profiling (15.464 s) : 15464000, 15464000
. : milestone, 15464000,
tracing (15.151 s) : 15151000, 15151000
. : milestone, 15151000,
section candidate
no_agent (15.041 s) : 15041000, 15041000
. : milestone, 15041000,
appsec (15.071 s) : 15071000, 15071000
. : milestone, 15071000,
iast (19.009 s) : 19009000, 19009000
. : milestone, 19009000,
iast_GLOBAL (17.885 s) : 17885000, 17885000
. : milestone, 17885000,
profiling (15.15 s) : 15150000, 15150000
. : milestone, 15150000,
tracing (14.88 s) : 14880000, 14880000
. : milestone, 14880000,
|
jbachorik
approved these changes
May 1, 2024
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
Makes
Runnable
s andCallable
s wrapped by grpc-context unwrappable, so we can report any time they spend queueing against the user's task name.Motivation
Additional Notes
Jira ticket: PROF-9698