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: Reset the value of @query_text_search before each loop, if escaping and hitting multiple databases #381

Merged
merged 1 commit into from Mar 25, 2024

Conversation

ReeceGoding
Copy link
Contributor

This is the smallest change that I could think of that would close #380. I have made an effort to be faithful to the original's style, but I must admit that I have not taken the time to become greatly familiar with this procedure. What I have done appears to work, but I have no idea if this is the best way.

…nd hitting multiple databases

This is the smallest change that I could think of that would close erikdarlingdata#380. I have made an effort to be faithful to the original's style, but I must admit that I have not taken the time to become greatly familiar with this procedure. What I have done appears to work, but I have no idea if this is the best way.
@erikdarlingdata erikdarlingdata added bug Something isn't working sp_QuickieStore For the loving of Query Store labels Mar 25, 2024
@erikdarlingdata
Copy link
Owner

@ReeceGoding no, this looks good. Thanks for the PR! Adding stuff to this proc is tricky because of the single/multiple database options. I'm always grateful when folks use it in different ways and find things I don't.

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.

None yet

2 participants