Skip to content

Update alter-workload-group-transact-sql.md#9547

Merged
rwestMSFT merged 2 commits intoMicrosoftDocs:livefrom
kendra-little:patch-3
Dec 15, 2023
Merged

Update alter-workload-group-transact-sql.md#9547
rwestMSFT merged 2 commits intoMicrosoftDocs:livefrom
kendra-little:patch-3

Conversation

@kendra-little
Copy link
Copy Markdown
Contributor

Adding note that to modify resource governor configuration in Azure SQL Managed instance, you must be in the context of the master database.

If I attempt to run a resource governor command in the context of another database, I receive the message:

Msg 40510, Level 16, State 2, Line 31
Statement 'ALTER RESOURCE GOVERNOR' is not supported in this version of SQL Server.

But it works when I use master.

I think probably this note should be integrated into an include file in some way, but I couldn't manage to do this through the web UI and I wanted to submit this before I forgot. If it's easier to close this PR and make the change another way, totally understand. Thanks in advance!

adding note that to modify resource governor configuration in Azure SQL Managed instance, you must be in the context of the master database.
@prmerger-automator
Copy link
Copy Markdown
Contributor

@kendra-little : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit c1a53a0:

✅ Validation status: passed

File Status Preview URL Details
docs/t-sql/statements/alter-workload-group-transact-sql.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@American-Dipper
Copy link
Copy Markdown
Contributor

@markingmyname - can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"

@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged tracking label for the PR review team label Dec 12, 2023
@rwestMSFT
Copy link
Copy Markdown
Contributor

@kendra-little Are you suggesting that this should be noted across all the Resource Governor articles in this section where we mention SQL Managed Instance?

@kendra-little
Copy link
Copy Markdown
Contributor Author

@kendra-little Are you suggesting that this should be noted across all the Resource Governor articles in this section where we mention SQL Managed Instance?

Hiya,

I don't have a super strong opinion -- I ran into the issue when altering the default workload group, which I think might be one of the more common operations for resource governor with Managed Instance.

But I can see how someone might run into it on other pages like https://learn.microsoft.com/en-us/sql/relational-databases/resource-governor/create-and-test-a-classifier-user-defined-function?view=sql-server-ver16, which might also be common for folks to use due to the handy scripts.

Whatever you think best, really :) Just took me a while to figure it out, especially because I don't see the same behavior on my local SQL Server install, which doesn't seem to care which database I'm using when running these commands. I was convinced for a brief time that it was broken or not allowed until I thought to try using master.

Thank you!

@kendra-little
Copy link
Copy Markdown
Contributor Author

ps -- i think i just realized why you asked. I made this PR using the "pencil" and I saw

[!INCLUDE alter-workload-group]

which seemed like where the modification would go, but I didn't know how to edit that directly through the UI :)

Sorry if that was confusing!

@rwestMSFT
Copy link
Copy Markdown
Contributor

@kendra-little I think this is what you had in mind. Check the files tab now.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 8f00d28:

✅ Validation status: passed

File Status Preview URL Details
docs/t-sql/includes/alter-workload-group.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@rwestMSFT rwestMSFT merged commit 36f2e6d into MicrosoftDocs:live Dec 15, 2023
@prmerger-automator
Copy link
Copy Markdown
Contributor

PR 9547 has been merged from kendra-little:patch-3 to MicrosoftDocs:live by rwestMSFT.

@kendra-little, @markingmyname, @rwestMSFT

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.

4 participants