Skip to content

Update alter-database-transact-sql.md#7639

Merged
jborsecnik merged 3 commits intoMicrosoftDocs:livefrom
charithcaldera:patch-10
Jun 2, 2022
Merged

Update alter-database-transact-sql.md#7639
jborsecnik merged 3 commits intoMicrosoftDocs:livefrom
charithcaldera:patch-10

Conversation

@charithcaldera
Copy link
Copy Markdown
Contributor

ALTER DATABASE MODIFY NAME does not work with Azure Synapse Analytics Dedicated SQL Pool
Test Scenario - Rename SQL Pools.docx
s

ALTER DATABASE MODIFY NAME does not work with Azure Synapse Analytics Dedicated SQL Pools
@PRMerger-2
Copy link
Copy Markdown
Contributor

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

@WilliamDAssafMSFT
Copy link
Copy Markdown
Contributor

@charithcaldera Does this apply to both legacy standalone dedicated SQL pools (formerly SQL DW) and dedicated SQL pools in Azure Synapse workspaces?

@charithcaldera
Copy link
Copy Markdown
Contributor Author

Hi @WilliamDAssafMSFT ,
ALTER DATABASE MODIFY NAME works for Legacy standalone Dedicated SQL Pools (formerly SQL DW), but it does not work for Dedicated SQL Pools in Azure Synapse workspace.
Document attached.
Test Scenario - Rename SQL Pools.docx

@jborsecnik
Copy link
Copy Markdown
Contributor

@WilliamDAssafMSFT, if you approve the changes, please add a #sign-off comment to signal that the PR is ready for the review team to merge. Thanks.

#label:"aq-pr-triaged"

@PRMerger7 PRMerger7 added the aq-pr-triaged tracking label for the PR review team label Jun 1, 2022
@WilliamDAssafMSFT
Copy link
Copy Markdown
Contributor

Hi @WilliamDAssafMSFT , ALTER DATABASE MODIFY NAME works for Legacy standalone Dedicated SQL Pools (formerly SQL DW), but it does not work for Dedicated SQL Pools in Azure Synapse workspace. Document attached. Test Scenario - Rename SQL Pools.docx

Based on that information the article is accurate. The comments on line 1044 are specific as to what scenarios ALTER DATABASE MODIFY NAME is supported. These changes as proposed would be too broad, as ALTER DATABASE MODIFY NAME is supported in some dedicated SQL pools.

@charithcaldera
Copy link
Copy Markdown
Contributor Author

@WilliamDAssafMSFT ,
This appears to be making a confusion here,

Azure Synapse SQL Dedicated Pool and Standalone Dedicated Pools (Formerly SQL DW) are treated as two.

In this public document, in line 966, its mentioned as follows which can make a confusion -
"In Azure Synapse, ALTER DATABASE modifies the name, maximum size, or service objective for a database."

Since Azure Synapse Analytics Dedicated SQL Pools does not supported on "ALTER DATABASE MODIFY NAME" according to line 1046, the two lines (966 and 1046) having a conflict there.

The problem is not about the Standalone Dedicated Pools (Formerly SQL DW), but the Dedicated Pools in Azure Synapse Workspace.
I hope its clear.

Copy link
Copy Markdown
Contributor

@WilliamDAssafMSFT WilliamDAssafMSFT left a comment

Choose a reason for hiding this comment

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

I understand. I've updated the line near the top of the article to be more general. I've reverted the change on line 966. Please commit these two Suggestions, thanks!

Comment thread docs/t-sql/statements/alter-database-transact-sql.md Outdated
Comment thread docs/t-sql/statements/alter-database-transact-sql.md Outdated
@PRMerger20
Copy link
Copy Markdown
Contributor

@charithcaldera: I'm sorry - only the author of this article, @WilliamDAssafMSFT, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@charithcaldera
Copy link
Copy Markdown
Contributor Author

Thank you @WilliamDAssafMSFT ,
I agree with your suggestions / changes.

@WilliamDAssafMSFT
Copy link
Copy Markdown
Contributor

#hold-off
@charithcaldera you should see options beneath each of my suggestions to "Commit suggestion". Go ahead and commit those changes. You'll know for sure they're in when they are reflected in the Files Changed tab.
image

Co-authored-by: William Assaf MSFT <wiassaf@microsoft.com>
@PRMerger14
Copy link
Copy Markdown
Contributor

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

Co-authored-by: William Assaf MSFT <wiassaf@microsoft.com>
@PRMerger19
Copy link
Copy Markdown
Contributor

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

@charithcaldera
Copy link
Copy Markdown
Contributor Author

Hi @WilliamDAssafMSFT ,
Done

@WilliamDAssafMSFT
Copy link
Copy Markdown
Contributor

Thanks @charithcaldera!!
#sign-off

@jborsecnik jborsecnik merged commit 315f7a9 into MicrosoftDocs:live Jun 2, 2022
@MicrosoftDocs MicrosoftDocs deleted a comment from PRMerger10 Jun 2, 2022
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.

10 participants