Skip to content
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

sql: include sql fingerprint id in workload_index_recs built-in #141947

Open
kyle-a-wong opened this issue Feb 24, 2025 · 0 comments
Open

sql: include sql fingerprint id in workload_index_recs built-in #141947

kyle-a-wong opened this issue Feb 24, 2025 · 0 comments
Labels
C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-observability

Comments

@kyle-a-wong
Copy link
Contributor

kyle-a-wong commented Feb 24, 2025

Is your feature request related to a problem? Please describe.
The workload_index_rec built-in returns a set of index recommendations, but it is not clear how these index recommendations will impact the workload or which fingerprints will benefit from them.

Describe the solution you'd like
The workload_index_rec built-in should include the fingerprint ids of the queries that will benefit from the index recommendations. Currently, the built-in returns a set of index recommendations (as strings). This could be updated to be a jsonb type with the following structure:

{
  "index_recommendation" : string,
  "impacted_fingerprint_ids" : []string
}

Jira issue: CRDB-48091

@kyle-a-wong kyle-a-wong added C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-observability labels Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-observability
Projects
None yet
Development

No branches or pull requests

1 participant