sql-docs icon indicating copy to clipboard operation
sql-docs copied to clipboard

Update use-the-system-health-session.md

Open sqlworldwide opened this issue 1 year ago • 6 comments

The system_health Extended Events session keeps ten files with a maximum size of 100 MB. You can confirm this by looking at GUI properties or scripting the definition.

ADD TARGET package0.event_file(SET filename=N'system_health.xel',max_file_size=(100),max_rollover_files=(10)),

sqlworldwide avatar Sep 05 '24 15:09 sqlworldwide

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

prmerger-automator[bot] avatar Sep 05 '24 15:09 prmerger-automator[bot]

Learn Build status updates of commit 8a2e7cc:

:white_check_mark: Validation status: passed

File Status Preview URL Details
docs/relational-databases/extended-events/use-the-system-health-session.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

@WilliamDAssafMSFT

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

Court72 avatar Sep 05 '24 15:09 Court72

Learn Build status updates of commit 689da68:

:warning: Validation status: warnings

File Status Preview URL Details
docs/connect/ado-net/sql/azure-active-directory-authentication.md :warning:Warning Details
docs/relational-databases/extended-events/use-the-system-health-session.md :white_check_mark:Succeeded

docs/connect/ado-net/sql/azure-active-directory-authentication.md

  • Line 330, Column 1: [Warning: codesnippet-not-found] Invalid code snippet link: '~/../sqlclient/doc/samples/AADAuthenticationCustomDeviceFlowCallback.cs'.
  • Line 338, Column 1: [Warning: codesnippet-not-found] Invalid code snippet link: '~/../sqlclient/doc/samples/ApplicationClientIdAzureAuthenticationProvider.cs'.
  • Line 368, Column 1: [Warning: codesnippet-not-found] Invalid code snippet link: '~/../sqlclient/doc/samples/CustomDeviceCodeFlowAzureAuthenticationProvider.cs'.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@sqlworldwide Does it make sense to add this line, or just replace the currently noted limits? I'm not familiar with this subject matter.

@WilliamDAssafMSFT

rwestMSFT avatar Sep 05 '24 16:09 rwestMSFT

@sqlworldwide @rwestMSFT after reviewing the source code, this is a little more complicated. It is edition dependent. I updated the PR accordingly, then I'll wait for some engineering confirmation.

WilliamDAssafMSFT avatar Sep 05 '24 16:09 WilliamDAssafMSFT

Learn Build status updates of commit 3efb866:

:white_check_mark: Validation status: passed

File Status Preview URL Details
docs/relational-databases/extended-events/use-the-system-health-session.md :white_check_mark:Succeeded

For more details, please refer to the build report.

#sign-off

WilliamDAssafMSFT avatar Jul 17 '25 01:07 WilliamDAssafMSFT

Learn Build status updates of commit de0e02f:

:white_check_mark: Validation status: passed

File Status Preview URL Details
docs/relational-databases/extended-events/use-the-system-health-session.md :white_check_mark:Succeeded

For more details, please refer to the build report.

Learn Build status updates of commit 6db2238:

:white_check_mark: Validation status: passed

File Status Preview URL Details
docs/relational-databases/extended-events/use-the-system-health-session.md :white_check_mark:Succeeded

For more details, please refer to the build report.

#sign-off

WilliamDAssafMSFT avatar Jul 17 '25 01:07 WilliamDAssafMSFT

PR 9874 has been merged from sqlworldwide:patch-9 to MicrosoftDocs:live by rwestMSFT.

@sqlworldwide, @WilliamDAssafMSFT, @rwest

prmerger-automator[bot] avatar Jul 17 '25 03:07 prmerger-automator[bot]