-
Notifications
You must be signed in to change notification settings - Fork 315
add a feature flag to disable extra calls to the DB in JDBC instrumentation #9774
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
base: master
Are you sure you want to change the base?
Conversation
|
🎯 Code Coverage 🔗 Commit SHA: a4ec186 | Docs | Datadog PR Page | Was this helpful? Give us feedback! |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 1 performance regressions! Performance is the same for 50 metrics, 14 unstable metrics.
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.55.0-SNAPSHOT~a4ec186c19, baseline=1.55.0-SNAPSHOT~364bb8b68f
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.028 s) : 0, 1027741
Total [baseline] (8.716 s) : 0, 8715994
Agent [candidate] (1.02 s) : 0, 1020213
Total [candidate] (8.67 s) : 0, 8670007
section iast
Agent [baseline] (1.17 s) : 0, 1170020
Total [baseline] (9.369 s) : 0, 9368754
Agent [candidate] (1.154 s) : 0, 1153631
Total [candidate] (9.349 s) : 0, 9349234
gantt
title insecure-bank - break down per module: candidate=1.55.0-SNAPSHOT~a4ec186c19, baseline=1.55.0-SNAPSHOT~364bb8b68f
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.49 ms) : 0, 1490
crashtracking [candidate] (1.466 ms) : 0, 1466
BytebuddyAgent [baseline] (700.71 ms) : 0, 700710
BytebuddyAgent [candidate] (694.581 ms) : 0, 694581
GlobalTracer [baseline] (244.608 ms) : 0, 244608
GlobalTracer [candidate] (243.081 ms) : 0, 243081
AppSec [baseline] (32.327 ms) : 0, 32327
AppSec [candidate] (32.087 ms) : 0, 32087
Debugger [baseline] (6.368 ms) : 0, 6368
Debugger [candidate] (6.251 ms) : 0, 6251
Remote Config [baseline] (692.506 µs) : 0, 693
Remote Config [candidate] (668.513 µs) : 0, 669
Telemetry [baseline] (13.86 ms) : 0, 13860
Telemetry [candidate] (9.2 ms) : 0, 9200
Flare Poller [baseline] (6.514 ms) : 0, 6514
Flare Poller [candidate] (11.659 ms) : 0, 11659
section iast
crashtracking [baseline] (1.498 ms) : 0, 1498
crashtracking [candidate] (1.489 ms) : 0, 1489
BytebuddyAgent [baseline] (832.906 ms) : 0, 832906
BytebuddyAgent [candidate] (816.764 ms) : 0, 816764
GlobalTracer [baseline] (233.674 ms) : 0, 233674
GlobalTracer [candidate] (232.293 ms) : 0, 232293
IAST [baseline] (32.482 ms) : 0, 32482
IAST [candidate] (26.726 ms) : 0, 26726
AppSec [baseline] (28.673 ms) : 0, 28673
AppSec [candidate] (35.178 ms) : 0, 35178
Debugger [baseline] (6.118 ms) : 0, 6118
Debugger [candidate] (6.163 ms) : 0, 6163
Remote Config [baseline] (590.317 µs) : 0, 590
Remote Config [candidate] (674.937 µs) : 0, 675
Telemetry [baseline] (8.397 ms) : 0, 8397
Telemetry [candidate] (8.695 ms) : 0, 8695
Flare Poller [baseline] (4.164 ms) : 0, 4164
Flare Poller [candidate] (4.214 ms) : 0, 4214
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.55.0-SNAPSHOT~a4ec186c19, baseline=1.55.0-SNAPSHOT~364bb8b68f
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.029 s) : 0, 1029108
Total [baseline] (10.778 s) : 0, 10777805
Agent [candidate] (1.024 s) : 0, 1023606
Total [candidate] (10.686 s) : 0, 10685771
section appsec
Agent [baseline] (1.201 s) : 0, 1200779
Total [baseline] (10.963 s) : 0, 10962704
Agent [candidate] (1.197 s) : 0, 1197437
Total [candidate] (10.813 s) : 0, 10813212
section iast
Agent [baseline] (1.164 s) : 0, 1163640
Total [baseline] (11.104 s) : 0, 11103796
Agent [candidate] (1.159 s) : 0, 1159361
Total [candidate] (11.138 s) : 0, 11137742
section profiling
Agent [baseline] (1.172 s) : 0, 1171974
Total [baseline] (10.968 s) : 0, 10968264
Agent [candidate] (1.165 s) : 0, 1164900
Total [candidate] (10.808 s) : 0, 10807721
gantt
title petclinic - break down per module: candidate=1.55.0-SNAPSHOT~a4ec186c19, baseline=1.55.0-SNAPSHOT~364bb8b68f
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.468 ms) : 0, 1468
crashtracking [candidate] (1.466 ms) : 0, 1466
BytebuddyAgent [baseline] (701.552 ms) : 0, 701552
BytebuddyAgent [candidate] (697.901 ms) : 0, 697901
GlobalTracer [baseline] (244.813 ms) : 0, 244813
GlobalTracer [candidate] (244.546 ms) : 0, 244546
AppSec [baseline] (32.201 ms) : 0, 32201
AppSec [candidate] (32.575 ms) : 0, 32575
Debugger [baseline] (6.37 ms) : 0, 6370
Debugger [candidate] (6.377 ms) : 0, 6377
Remote Config [baseline] (683.439 µs) : 0, 683
Remote Config [candidate] (684.273 µs) : 0, 684
Telemetry [baseline] (15.799 ms) : 0, 15799
Telemetry [candidate] (9.44 ms) : 0, 9440
Flare Poller [baseline] (4.987 ms) : 0, 4987
Flare Poller [candidate] (9.408 ms) : 0, 9408
section appsec
crashtracking [baseline] (1.463 ms) : 0, 1463
crashtracking [candidate] (1.455 ms) : 0, 1455
BytebuddyAgent [baseline] (724.766 ms) : 0, 724766
BytebuddyAgent [candidate] (720.259 ms) : 0, 720259
GlobalTracer [baseline] (235.868 ms) : 0, 235868
GlobalTracer [candidate] (235.344 ms) : 0, 235344
IAST [baseline] (24.862 ms) : 0, 24862
IAST [candidate] (24.896 ms) : 0, 24896
AppSec [baseline] (174.049 ms) : 0, 174049
AppSec [candidate] (175.209 ms) : 0, 175209
Debugger [baseline] (5.851 ms) : 0, 5851
Debugger [candidate] (6.077 ms) : 0, 6077
Remote Config [baseline] (626.339 µs) : 0, 626
Remote Config [candidate] (654.879 µs) : 0, 655
Telemetry [baseline] (8.33 ms) : 0, 8330
Telemetry [candidate] (8.469 ms) : 0, 8469
Flare Poller [baseline] (3.861 ms) : 0, 3861
Flare Poller [candidate] (3.872 ms) : 0, 3872
section iast
crashtracking [baseline] (1.466 ms) : 0, 1466
crashtracking [candidate] (1.481 ms) : 0, 1481
BytebuddyAgent [baseline] (825.751 ms) : 0, 825751
BytebuddyAgent [candidate] (821.222 ms) : 0, 821222
GlobalTracer [baseline] (234.32 ms) : 0, 234320
GlobalTracer [candidate] (233.269 ms) : 0, 233269
IAST [baseline] (32.463 ms) : 0, 32463
IAST [candidate] (26.843 ms) : 0, 26843
AppSec [baseline] (28.846 ms) : 0, 28846
AppSec [candidate] (35.251 ms) : 0, 35251
Debugger [baseline] (6.175 ms) : 0, 6175
Debugger [candidate] (6.182 ms) : 0, 6182
Remote Config [baseline] (598.769 µs) : 0, 599
Remote Config [candidate] (599.886 µs) : 0, 600
Telemetry [baseline] (8.473 ms) : 0, 8473
Telemetry [candidate] (8.707 ms) : 0, 8707
Flare Poller [baseline] (4.177 ms) : 0, 4177
Flare Poller [candidate] (4.24 ms) : 0, 4240
section profiling
crashtracking [baseline] (1.45 ms) : 0, 1450
crashtracking [candidate] (1.468 ms) : 0, 1468
BytebuddyAgent [baseline] (724.023 ms) : 0, 724023
BytebuddyAgent [candidate] (721.593 ms) : 0, 721593
GlobalTracer [baseline] (220.918 ms) : 0, 220918
GlobalTracer [candidate] (219.375 ms) : 0, 219375
AppSec [baseline] (33.212 ms) : 0, 33212
AppSec [candidate] (32.236 ms) : 0, 32236
Debugger [baseline] (9.861 ms) : 0, 9861
Debugger [candidate] (6.676 ms) : 0, 6676
Remote Config [baseline] (717.864 µs) : 0, 718
Remote Config [candidate] (692.748 µs) : 0, 693
Telemetry [baseline] (12.165 ms) : 0, 12165
Telemetry [candidate] (15.707 ms) : 0, 15707
Flare Poller [baseline] (4.134 ms) : 0, 4134
Flare Poller [candidate] (4.204 ms) : 0, 4204
ProfilingAgent [baseline] (110.131 ms) : 0, 110131
ProfilingAgent [candidate] (109.077 ms) : 0, 109077
Profiling [baseline] (110.744 ms) : 0, 110744
Profiling [candidate] (109.702 ms) : 0, 109702
LoadParameters
See matching parameters
SummaryFound 4 performance improvements and 2 performance regressions! Performance is the same for 6 metrics, 12 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.55.0-SNAPSHOT~a4ec186c19, baseline=1.55.0-SNAPSHOT~364bb8b68f
dateFormat X
axisFormat %s
section baseline
no_agent (38.251 ms) : 37943, 38560
. : milestone, 38251,
appsec (47.984 ms) : 47568, 48400
. : milestone, 47984,
code_origins (44.342 ms) : 43951, 44733
. : milestone, 44342,
iast (44.228 ms) : 43849, 44606
. : milestone, 44228,
profiling (47.676 ms) : 47216, 48136
. : milestone, 47676,
tracing (43.96 ms) : 43574, 44345
. : milestone, 43960,
section candidate
no_agent (35.976 ms) : 35689, 36264
. : milestone, 35976,
appsec (46.899 ms) : 46496, 47303
. : milestone, 46899,
code_origins (44.161 ms) : 43774, 44548
. : milestone, 44161,
iast (43.97 ms) : 43586, 44354
. : milestone, 43970,
profiling (49.618 ms) : 49136, 50099
. : milestone, 49618,
tracing (41.818 ms) : 41445, 42191
. : milestone, 41818,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.55.0-SNAPSHOT~a4ec186c19, baseline=1.55.0-SNAPSHOT~364bb8b68f
dateFormat X
axisFormat %s
section baseline
no_agent (4.376 ms) : 4326, 4426
. : milestone, 4376,
iast (10.238 ms) : 10067, 10410
. : milestone, 10238,
iast_FULL (13.794 ms) : 13526, 14063
. : milestone, 13794,
iast_GLOBAL (11.119 ms) : 10919, 11320
. : milestone, 11119,
profiling (8.686 ms) : 8541, 8831
. : milestone, 8686,
tracing (7.988 ms) : 7871, 8104
. : milestone, 7988,
section candidate
no_agent (4.406 ms) : 4348, 4464
. : milestone, 4406,
iast (9.733 ms) : 9573, 9893
. : milestone, 9733,
iast_FULL (13.859 ms) : 13588, 14130
. : milestone, 13859,
iast_GLOBAL (10.823 ms) : 10633, 11013
. : milestone, 10823,
profiling (9.052 ms) : 8912, 9192
. : milestone, 9052,
tracing (7.572 ms) : 7464, 7679
. : milestone, 7572,
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.55.0-SNAPSHOT~a4ec186c19, baseline=1.55.0-SNAPSHOT~364bb8b68f
dateFormat X
axisFormat %s
section baseline
no_agent (1.478 ms) : 1467, 1490
. : milestone, 1478,
appsec (3.681 ms) : 3469, 3894
. : milestone, 3681,
iast (2.222 ms) : 2159, 2286
. : milestone, 2222,
iast_GLOBAL (2.258 ms) : 2194, 2322
. : milestone, 2258,
profiling (2.066 ms) : 2014, 2117
. : milestone, 2066,
tracing (2.038 ms) : 1988, 2088
. : milestone, 2038,
section candidate
no_agent (1.486 ms) : 1474, 1498
. : milestone, 1486,
appsec (3.647 ms) : 3433, 3861
. : milestone, 3647,
iast (2.205 ms) : 2141, 2269
. : milestone, 2205,
iast_GLOBAL (2.264 ms) : 2199, 2328
. : milestone, 2264,
profiling (2.05 ms) : 1999, 2101
. : milestone, 2050,
tracing (2.026 ms) : 1976, 2075
. : milestone, 2026,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.55.0-SNAPSHOT~a4ec186c19, baseline=1.55.0-SNAPSHOT~364bb8b68f
dateFormat X
axisFormat %s
section baseline
no_agent (15.594 s) : 15594000, 15594000
. : milestone, 15594000,
appsec (14.824 s) : 14824000, 14824000
. : milestone, 14824000,
iast (18.679 s) : 18679000, 18679000
. : milestone, 18679000,
iast_GLOBAL (18.049 s) : 18049000, 18049000
. : milestone, 18049000,
profiling (15.385 s) : 15385000, 15385000
. : milestone, 15385000,
tracing (15.117 s) : 15117000, 15117000
. : milestone, 15117000,
section candidate
no_agent (14.908 s) : 14908000, 14908000
. : milestone, 14908000,
appsec (14.7 s) : 14700000, 14700000
. : milestone, 14700000,
iast (18.522 s) : 18522000, 18522000
. : milestone, 18522000,
iast_GLOBAL (18.086 s) : 18086000, 18086000
. : milestone, 18086000,
profiling (14.947 s) : 14947000, 14947000
. : milestone, 14947000,
tracing (15.086 s) : 15086000, 15086000
. : milestone, 15086000,
|
internal-api/src/test/groovy/datadog/trace/api/ConfigTest.groovy
Outdated
Show resolved
Hide resolved
| try { | ||
| clientInfo = connection.getClientInfo(); | ||
| } catch (final Throwable ex) { | ||
| // getClientInfo is likely not allowed, we can still extract info from the url alone |
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.
Maybe make sense to put some debug/trace log? Just thinking out loud.
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.
you mean for investigation in the current case, or in general ?
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.
I think it could help for investigation in the current case! I have no scale for how often this method is called though -- would adding debug statements here be "okay" in a production environment where we don't want to spam logs?
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.
added a log, we'll need to make sure this is not merged to main if we want to keep the feature flags for further investigations
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.
Cool thanks! I'll add a do not merge tag for now to remind us
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
What Does This Do
the JDBC instrumentation does extra calls to the DB to fetch the metadata and client info. In case this might cause issues, we want to be able to disable this behavior to reduce the instrumentation to simply creating spans.
Motivation
customer investigation
Additional Notes
missing tests for now
Contributor Checklist
type:and (comp:orinst:) labels in addition to any useful labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: APMS-16143