Lokasi ngalangkungan proxy:   [ UP ]  
[Ngawartoskeun bug]   [Panyetelan cookie]                
Skip to content

Update best-practice-with-the-query-store.md#107

Merged
craigg-msft merged 1 commit intoMicrosoftDocs:livefrom
ArthurDaniels:patch-1
Sep 20, 2017
Merged

Update best-practice-with-the-query-store.md#107
craigg-msft merged 1 commit intoMicrosoftDocs:livefrom
ArthurDaniels:patch-1

Conversation

@ArthurDaniels
Copy link
Copy Markdown
Contributor

I propose this change because I have found that these two traceflags are necessary for an optimal configuration of Query Store. During a failover, I saw the wait type QDS_LOADDB track 3 minutes of wait time, meaning that Query Store was waiting to load and queries could not run. This behavior is prevented by Traceflag 7752 and I propose that this documentation includes at least a mention of these, considering that best practices are often referenced by consultants and corporations who set up this new feature.

I hope I provided sufficient description of what they do.

I propose this change because I have found that these two traceflags are necessary for an optimal configuration of Query Store. During a failover, I saw the wait type QDS_LOADDB track 3 minutes of wait time, meaning that Query Store was waiting to load and queries could not run. This behavior is prevented by Traceflag 7752 and I propose that this documentation includes at least a mention of these, considering that best practices are often referenced by consultants and corporations who set up this new feature.

I hope I provided sufficient description of what they do.
@msftclas
Copy link
Copy Markdown

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@craigg-msft
Copy link
Copy Markdown
Contributor

Thank you @ArthurDaniels for the help. @BYHAM please review.

@BYHAM
Copy link
Copy Markdown
Contributor

BYHAM commented Sep 19, 2017

Referring to @AndrejsAnt for review.

@AndrejsAnt
Copy link
Copy Markdown

#sign-off

Copy link
Copy Markdown
Contributor

@BYHAM BYHAM left a comment

Choose a reason for hiding this comment

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

Thank you.

@BYHAM
Copy link
Copy Markdown
Contributor

BYHAM commented Sep 20, 2017

#sign-off

@craigg-msft craigg-msft merged commit 94c866f into MicrosoftDocs:live Sep 20, 2017
@ArthurDaniels ArthurDaniels deleted the patch-1 branch September 25, 2017 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants