Skip to content

Added connected workspace reference#7968

Merged
WilliamDAssafMSFT merged 4 commits intoMicrosoftDocs:livefrom
scott-epperly:patch-4
Aug 26, 2022
Merged

Added connected workspace reference#7968
WilliamDAssafMSFT merged 4 commits intoMicrosoftDocs:livefrom
scott-epperly:patch-4

Conversation

@scott-epperly
Copy link
Copy Markdown
Contributor

Recent SR made it clear that the feature disparity for renaming a dedicated SQL pool in a Synapse workspace is not clearly documented. Renaming cannot occur under a standard workspace (where the workspace is created first and then the dedicated SQL pool is created), but is supported in a connected workspace scenario.

Recent SR made it clear that the feature disparity for renaming a dedicated SQL pool in a Synapse workspace is not clearly documented.  Renaming cannot occur under a standard workspace (where the workspace is created first and then the dedicated SQL pool is created), but is supported in a connected workspace scenario.
@prmerger-automator
Copy link
Copy Markdown
Contributor

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

@opbld33
Copy link
Copy Markdown

opbld33 commented Aug 19, 2022

Docs Build status updates of commit 415c774:

✅ Validation status: passed

File Status Preview URL Details
docs/t-sql/statements/alter-database-transact-sql.md 💡Suggestion Details

docs/t-sql/statements/alter-database-transact-sql.md

  • Line 1050, Column 173: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/synapse-analytics/sql-data-warehouse/workspace-connected-create' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@Jak-MS
Copy link
Copy Markdown
Contributor

Jak-MS commented Aug 19, 2022

@WilliamDAssafMSFT - Can you review this PR?

IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#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 Aug 19, 2022
@prmerger-automator
Copy link
Copy Markdown
Contributor

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

@opbld33
Copy link
Copy Markdown

opbld33 commented Aug 26, 2022

Docs Build status updates of commit baf8df9:

✅ Validation status: passed

File Status Preview URL Details
docs/t-sql/statements/alter-database-transact-sql.md 💡Suggestion Details

docs/t-sql/statements/alter-database-transact-sql.md

  • Line 1048, Column 173: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/azure/synapse-analytics/sql-data-warehouse/workspace-connected-create' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

Copy link
Copy Markdown
Contributor Author

@scott-epperly scott-epperly left a comment

Choose a reason for hiding this comment

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

here's what it should look like

Comment thread docs/t-sql/statements/alter-database-transact-sql.md Outdated
Comment thread docs/t-sql/statements/alter-database-transact-sql.md Outdated
Updated link and corrected name
@prmerger-automator
Copy link
Copy Markdown
Contributor

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

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

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

@opbld33
Copy link
Copy Markdown

opbld33 commented Aug 26, 2022

Docs Build status updates of commit 459858b:

✅ Validation status: passed

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

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@opbld30
Copy link
Copy Markdown

opbld30 commented Aug 26, 2022

Docs Build status updates of commit fc6b956:

✅ Validation status: passed

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

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@WilliamDAssafMSFT
Copy link
Copy Markdown
Contributor

#sign-off

@prmerger-automator
Copy link
Copy Markdown
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @markingmyname

@WilliamDAssafMSFT WilliamDAssafMSFT merged commit 9810a23 into MicrosoftDocs:live Aug 26, 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.

6 participants