-
Notifications
You must be signed in to change notification settings - Fork 155
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.
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
Query Store Service #2171
Query Store Service #2171
Conversation
… QSM when waitstats is an available statistic#
As part of updating the dependencies in Packages.props we require that any PRs opened also verify that Please respond to this comment verifying that you've done the appropriate validation (or explain why it's not necessary) before merging in the PR
|
Just a draft for now to take a look at how things are working as I finish adding the tests. At @cheenamalhotra's recommendation, I may end up swapping to instead construct and execute a SqlCommand, and return an ID that ADS can then use to retrieve the result set, as this would remove the need to roll my own .NET -> TSQL datatype conversion code. |
src/Microsoft.SqlTools.ServiceLayer/QueryStore/QueryStoreService.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.SqlTools.ServiceLayer/QueryStore/QueryStoreService.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.SqlTools.ServiceLayer/QueryStore/QueryStoreService.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.SqlTools.ServiceLayer/QueryStore/Contracts/QueryStoreReportParams.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.SqlTools.ServiceLayer/QueryStore/Contracts/GetForcedPlanQueriesReport.cs
Outdated
Show resolved
Hide resolved
…oolsservice into benjin/queryStore
looks like the other tests also need the line ending change made for the ForcedPlanQueries and TrackedQueries tests, now that they're getting past the timezone failure |
Adds the Query Store service, which generates queries needed to fetch the data for all query store reports