It's easy to go bananas over one single configuration item such as sp_xpcmdshell. Of course noone should enable it wo having a good and well documented case for it.

And...

With risk to duplicate what Gaz mentinoned, we need to understand the basic processing/binding order (FROM, ON, JOIN and so on) and realize that the Query processor is the final...

...and I guess it's

SELECT * FROM #waitstats

and not

SELECT * FROM ..sysprocesses

right? 🙂

Excellent script, just had to adjust the first row to:

WITH Waits (servername, wait_type, wait_time_s, pct, rn) AS....

to include the columns!

Will use this in the daily reporting!

This article was written in a good form, but if the goal was clarifying the logic of inner join the content MUST be 100% correct. I can live with one...