Skip to content

Update sys-dm-db-resource-stats-azure-sql-database.md - #10385

Open
Ves (sirsql) wants to merge 1 commit into
MicrosoftDocs:livefrom
sirsql:patch-2
Open

Update sys-dm-db-resource-stats-azure-sql-database.md#10385
Ves (sirsql) wants to merge 1 commit into
MicrosoftDocs:livefrom
sirsql:patch-2

Conversation

@sirsql

Copy link
Copy Markdown
Contributor

Per https://learn.microsoft.com/en-us/azure/azure-sql/managed-instance/monitoring-with-dmvs?view=azuresql#sysdm_db_resource_stats the use of sys.dm_db_resource_stats is supported in Managed Instances making this note incorrect.

@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@prmerger-automator

Copy link
Copy Markdown
Contributor

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

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 61de99f:

✅ Validation status: passed

File Status Preview URL Details
docs/relational-databases/system-dynamic-management-objects/sys-dm-db-resource-stats-azure-sql-database.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards

Copy link
Copy Markdown
Contributor

Randolph West MSFT (@rwestMSFT)

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way 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 Sep 2, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The note removal makes the page’s applies-to/include and intro text inconsistent with its azuresqldb-mi-current moniker and existing examples that mention Managed Instance.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Removes an incorrect note stating sys.dm_db_resource_stats isn’t supported on Azure SQL Managed Instance, aligning the page with current platform support.

Changes:

  • Removed the Managed Instance “not supported” note from the sys.dm_db_resource_stats reference page.
File summaries
File Description
docs/relational-databases/system-dynamic-management-objects/sys-dm-db-resource-stats-azure-sql-database.md Removes outdated guidance about Managed Instance support for sys.dm_db_resource_stats.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@@ -28,8 +28,6 @@ monikerRange: "=azuresqldb-current || =azuresqldb-mi-current || =fabric-sqldb"

Returns CPU, I/O, and memory consumption for a database in [!INCLUDE [ssazure-sqldb](../../includes/ssazure-sqldb.md)]. One row exists for every 15 seconds, even if there's no activity. Historical data is maintained for approximately one hour.
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