erikdarlingdata / DarlingData

Open source SQL Server nonsense: sp_PressureDetector, sp_QuickieStore, sp_HumanEvents, etc.
https://www.erikdarling.com/
MIT License
443 stars 132 forks source link

sp_QuickieStore: Added parameter to hide the bottom table #431

Closed ReeceGoding closed 1 month ago

ReeceGoding commented 1 month ago

Closes #430

As always, I must warn that I have never taken a deep look at how this procedure works, so I may have totally messed it up. I have tried my best to make small changes that are true to the design and formatting.

I wasn't sure what to call the new parameter. The table that we're hiding doesn't seem to have any names. One YouTube video called it the "bottom table". I otherwise found nothing. In the end, I went with @hide_help_table. Communing with my inner Erik Darling indicates that I should have called it @helpless_mode.

No attempt has been made to update the documentation, because I know Erik already has plans for that. The @help = 1 text has been updated.

erikdarlingdata commented 1 month ago

@ReeceGoding hey, thanks! I'm going to merge this and your other PR into dev and review things.