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

sp_QuickieStore: Made last_ columns report the actual lasts, rather than the maximum of them #492

Merged
merged 1 commit into from
Oct 28, 2024

Conversation

ReeceGoding
Copy link
Contributor

Closes #474 . There wasn't really much to this. I just had to add in some calls to LAST_VALUE.

This was surprisingly hard to test. I'm fully confident in the runtime_stats table, but the table that pulls from the plan cache is inherently noisy and testing the tables showing the wait stats was so difficult that I started to question the other columns in them. Shockingly often, the wait stats tables would give identical outputs both for this version of sp_QuickieStore and for an older version.

@erikdarlingdata erikdarlingdata added bug Something isn't working sp_QuickieStore For the loving of Query Store labels Oct 28, 2024
@erikdarlingdata erikdarlingdata merged commit fe726cc into erikdarlingdata:dev Oct 28, 2024
1 check passed
erikdarlingdata added a commit that referenced this pull request Oct 29, 2024
Fixes a big introduced in #492 where a column in the qsrs_with_lasts derived table was missing a column alias ( = NULL) on older versions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working sp_QuickieStore For the loving of Query Store
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants