Skip to content

Conversation

@bwiggin10
Copy link
Contributor

This adds additional context to the checks for Trace Flags 834, 7745 and 7752 that have messages dependent on @CheckUserDatabasesObjects = 1 (Or that and @BringThePain = 1 for database counts over 50).

I discovered this when running sp_Blitz @CheckUserDatabaseObjects = 1 and I forgot to specify @BringThePain = 1
That even when you run just sp_Blitz with no parameters, you could end up with Trace Flag 7745 telling you that no databases have query store enabled even if you do (Because the @QueryStoreInUse check can't run).

Looking at the code, it seemed the same issue would be possible for T834 and T7752.

This adds additional context to the checks for Trace Flags 834, 7745 and 7752 that have messages dependent on @CheckUserDatabasesObjects = 1 (Or that and @Bringthepain = 1 for database counts over 50).

So when you just ran sp_Blitz, you could end up with Trace Flag 7745 telling you that no databases have query store enabled even if you do (Because the check @QueryStoreInUse couldn't run)
Copy link
Member

@BrentOzar BrentOzar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great catch! I love it. Thanks for the pull request, merging into the dev branch, will be in the next release with credit to you in the release notes.

@BrentOzar BrentOzar added this to the 2025-12 Release milestone Oct 9, 2025
@BrentOzar BrentOzar merged commit e5d01d1 into BrentOzarULTD:dev Oct 9, 2025
0 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants