Skip to content

Clarify file extension conventions for backups - #10380

Open
Deepesh Dhake (deepeshd87) wants to merge 2 commits into
MicrosoftDocs:livefrom
deepeshd87:patch-1
Open

Clarify file extension conventions for backups#10380
Deepesh Dhake (deepeshd87) wants to merge 2 commits into
MicrosoftDocs:livefrom
deepeshd87:patch-1

Conversation

@deepeshd87

Copy link
Copy Markdown

The "Best practice recommendations" section listed .BAK/.TRN under "To ensure security," but these extensions are a naming conventions. SQL Server doesn't enforce or read them, and they provide no security property. This rewords the guidance to reflect that, while keeping a concrete operational reason to follow the convention (antivirus exclusion configuration, per Configure antivirus software to work with SQL Server). Also adds a missing terminal period to the .BAK list item for consistency.

Updated best practice recommendations for backup file extensions.
@prmerger-automator

Copy link
Copy Markdown
Contributor

Deepesh Dhake (@deepeshd87) : 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 8dc0097:

✅ Validation status: passed

File Status Preview URL Details
docs/relational-databases/backup-restore/back-up-and-restore-of-sql-server-databases.md ✅Succeeded

For more details, please refer to the build report.

@prmerger-automator

Copy link
Copy Markdown
Contributor

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

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.

Pull request overview

Updates SQL Server backup documentation to clarify that .BAK/.TRN are naming conventions (not security features), while still explaining an operational reason to follow them.

Changes:

  • Reworded guidance to remove the implication that file extensions provide security properties.
  • Added operational rationale for consistent extensions (backup identification/management and antivirus exclusion configuration).
  • Added a missing terminal period for consistency in the .BAK bullet.

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

@Court72

Copy link
Copy Markdown
Contributor

Masha Thomas (MSFT) (@MashaMSFT)

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 Aug 24, 2026
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 5c313d8:

✅ Validation status: passed

File Status Preview URL Details
docs/relational-databases/backup-restore/back-up-and-restore-of-sql-server-databases.md ✅Succeeded

For more details, please refer to the build report.

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