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

Updated the script with additional steps for killing the ongoing proc…#5372

Merged
ktoliver merged 4 commits intoMicrosoftDocs:livefrom
ronazi:patch-1
Oct 8, 2020
Merged

Updated the script with additional steps for killing the ongoing proc…#5372
ktoliver merged 4 commits intoMicrosoftDocs:livefrom
ronazi:patch-1

Conversation

@ronazi
Copy link
Copy Markdown
Contributor

@ronazi ronazi commented Aug 27, 2020

…esses and also sp_reconfigure options

Running just those (on the current doc) will error out with multiple errors and customers will need to go back and forth in troubleshooting the errors. Hence edited the doc with detailed step by step guide.

…esses and also sp_reconfigure options

Running just those (on the current doc) will error out with multiple errors and customers will need to go back and forth in troubleshooting the errors. Hence edited the doc with detailed step by step guide.
@ghost
Copy link
Copy Markdown

ghost commented Aug 27, 2020

CLA assistant check
All CLA requirements met.

@PRMerger15
Copy link
Copy Markdown
Contributor

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

@VanMSFT
Copy link
Copy Markdown
Member

VanMSFT commented Aug 27, 2020

Hi @ronazi - Thanks for the proposed changes.

@bluefooted - Hey Pam, do you mind taking a quick look at the proposed changes? Do you agree with providing instructions to kill all SPIDs above 50 before changing the containment option? At the very least, we will need to put a warning in place. Thanks!

@ronazi
Copy link
Copy Markdown
Contributor Author

ronazi commented Sep 10, 2020

@VanMSFT What is the status on this?

@ktoliver ktoliver requested a review from bluefooted September 10, 2020 16:36
@bluefooted
Copy link
Copy Markdown
Contributor

Sorry folks, just got back from vacation! No, the kill script isn't the best way to do this, we would normally recommend something like the following script:

ALTER DATABASE Test
SET SINGLE_USER -- or RESTRICTED_USER for Managed Instance
WITH ROLLBACK IMMEDIATE;

ALTER DATABASE Test
SET containment=partial;

ALTER DATABASE Test
SET MULTI_USER;

And yes I agree, if we add this, we should also add a warning.

@PRMerger7
Copy link
Copy Markdown
Contributor

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

@VanMSFT
Copy link
Copy Markdown
Member

VanMSFT commented Oct 8, 2020

Thanks Pam!

@ronazi - I made edits to your PR based on Pam's suggestion. Thanks for the help with our documentation. Let me know if there are any issues. I will approve this PR now.

#sign-off

Copy link
Copy Markdown
Contributor

@bluefooted bluefooted left a comment

Choose a reason for hiding this comment

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

Looks good, thank you!

@PRMerger16
Copy link
Copy Markdown
Contributor

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

@ktoliver ktoliver merged commit ae319cd into MicrosoftDocs:live Oct 8, 2020
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.

9 participants