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.

Sign up for GitHub

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

Jump to bottom

Check for plans with >128 levels of nesting #839

Merged
merged 3 commits into from
Apr 8, 2017
Merged

Check for plans with >128 levels of nesting #839

merged 3 commits into from
Apr 8, 2017

Conversation

BlitzErik
Copy link
Contributor

Closes #838

@BlitzErik
Closes #838
a28e2ee 
Closes #838
@BlitzErik BlitzErik mentioned this pull request Apr 7, 2017
Closed
@BlitzErik
Fixup
dde705b 
Duh. SPID.
@BlitzErik
Adding raiserror and spid filters
68384f2 
Also, this has to left join for the other message to bubble up via the
isnull. Duh.
@BlitzErik BlitzErik changed the title Closes #838 Check for plans with >128 levels of nesting Apr 8, 2017
@BlitzErik BlitzErik self-assigned this Apr 8, 2017
@BlitzErik BlitzErik added this to the 2017-05 milestone Apr 8, 2017
@BlitzErik BlitzErik merged commit 4621b34 into dev Apr 8, 2017
@BlitzErik BlitzErik deleted the sp_BlitzCache_838 branch April 8, 2017 10:31
@BrentOzar
Copy link
Member

Love it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@BlitzErik BlitzErik

Labels
enhancement sp_BlitzCache
Projects
None yet
Milestone
2017-05
Development

Successfully merging this pull request may close these issues.
2 participants
@BlitzErik @BrentOzar