-
Notifications
You must be signed in to change notification settings - Fork 4.1k
sql: include all session settings in statement bundle env.sql #72434
Copy link
Copy link
Closed
Labels
C-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)E-quick-winLikely to be a quick win for someone experienced.Likely to be a quick win for someone experienced.T-sql-queriesSQL Queries TeamSQL Queries Team
Description
We should include all session settings in the statement bundle so that we do not lack any information when debugging customer issues. If there are reasons why including all is problematic, we could consider including all session settings that do not have default values.
See the TODO here:
cockroach/pkg/sql/explain_bundle.go
Lines 522 to 523 in 87f79da
| // TODO(rytaft): Keeping this list up to date is a challenge. Consider just | |
| // printing all session settings. |
Jira issue: CRDB-11151
Epic CRDB-14510
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
C-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)E-quick-winLikely to be a quick win for someone experienced.Likely to be a quick win for someone experienced.T-sql-queriesSQL Queries TeamSQL Queries Team
Type
Projects
Status
Done