-
Notifications
You must be signed in to change notification settings - Fork 357
Closed
Labels
comp:instrumentation.sqlclientThings related to OpenTelemetry.Instrumentation.SqlClientThings related to OpenTelemetry.Instrumentation.SqlClient
Description
SetDbStatementForText controls setting db.query.summary and a sanitized version of db.query.text. The option is currently disabled by default.
db.query.summary and db.query.text are required by the semantic conventions. I propose we eliminate this configuration option. Other languages supporting these attributes have not provided a way to opt in or out of this functionality.
Line 81 in eddd79f
| public bool SetDbStatementForText { get; set; } |
Metadata
Metadata
Assignees
Labels
comp:instrumentation.sqlclientThings related to OpenTelemetry.Instrumentation.SqlClientThings related to OpenTelemetry.Instrumentation.SqlClient