Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 6, 2026

  • Review the current relational-storage.md file to understand all references to UseJsonFormat
  • Update the configuration example to use GrainStorageSerializer instead of UseJsonFormat
  • Update the AdoNetGrainStorageOptions class documentation to reflect current API (remove UseJsonFormat, UseXmlFormat, etc.)
  • Add explanation about configuring serialization for grain storage
  • Add reference to serialization configuration documentation
  • Update DynamoDB storage documentation to remove UseJson reference
  • Update general persistence documentation recommendations
  • Verify no other Orleans documentation files need similar updates
  • Test building the documentation (markdownlint passed)
  • Add zone pivots to scope Orleans 7.0+ guidance and preserve Orleans 3.x guidance
  • Add zone pivots to relational-storage.md and dynamodb-storage.md
  • Improve custom serializer example with explicit DI registration
  • Clarify StorageSerializationPicker vs GrainStorageSerializer usage
Original prompt

This section details on the original issue you should resolve

<issue_title>Ado Persistence Documentation Out of Date </issue_title>
<issue_description>

Type of issue

Other (describe below)

Description

The documentation has not been updated to include the new way to set storage serialization.
UseJsonFormat no longer exists. dotnet/orleans#8383 (comment)

Page URL

https://learn.microsoft.com/en-us/dotnet/orleans/grains/grain-persistence/relational-storage

Content source URL

https://github.com/dotnet/docs/blob/live/docs/orleans/grains/grain-persistence/relational-storage.md

Document Version Independent Id

3a311967-0163-2594-02a1-7d6863571e14

Article author

mosagie

Metadata

  • ID: eb892ec8-fbbc-9d3b-b84a-ed1d870ed3f9
  • Service: dotnet-orleans

Associated WorkItem - 551266</issue_description>

Comments on the Issue (you are @copilot in this section)

@gewarren @meaghanlewis FYI

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.


Internal previews

📄 File 🔗 Preview link
docs/orleans/grains/grain-persistence/dynamodb-storage.md docs/orleans/grains/grain-persistence/dynamodb-storage
docs/orleans/grains/grain-persistence/index.md docs/orleans/grains/grain-persistence/index
docs/orleans/grains/grain-persistence/relational-storage.md docs/orleans/grains/grain-persistence/relational-storage

…Json

Co-authored-by: meaghanlewis <10103121+meaghanlewis@users.noreply.github.com>
Copilot AI changed the title [WIP] Update Ado persistence documentation for storage serialization Update Orleans persistence docs for Orleans 7.0+ serialization API Feb 6, 2026
Copilot AI requested a review from meaghanlewis February 6, 2026 19:59
Co-authored-by: meaghanlewis <10103121+meaghanlewis@users.noreply.github.com>
Copilot AI requested a review from meaghanlewis February 6, 2026 20:41
@meaghanlewis meaghanlewis marked this pull request as ready for review February 6, 2026 21:41
Copilot AI review requested due to automatic review settings February 6, 2026 21:41
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates Orleans grain persistence documentation to reflect the Orleans 7.0+ storage serialization configuration model (using IGrainStorageSerializer and the GrainStorageSerializer options property) and removes outdated UseJsonFormat/UseJson references.

Changes:

  • Replace legacy UseJsonFormat/UseJson mentions with Orleans 7.0+ GrainStorageSerializer guidance.
  • Add new “Configure serialization” sections and cross-link to the main serialization documentation.
  • Add version-scoped guidance in the grain persistence overview via zone pivots.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.

File Description
docs/orleans/grains/grain-persistence/relational-storage.md Removes UseJsonFormat, adds Orleans 7.0+ serializer configuration guidance, and updates referenced options docs.
docs/orleans/grains/grain-persistence/index.md Adds zone-pivoted recommendation text for Orleans 7.0+ storage serializer configuration.
docs/orleans/grains/grain-persistence/dynamodb-storage.md Removes UseJson from the sample and adds an Orleans 7.0+ serializer configuration section.

meaghanlewis and others added 2 commits February 6, 2026 13:53
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: meaghanlewis <10103121+meaghanlewis@users.noreply.github.com>
@meaghanlewis meaghanlewis merged commit dcc1717 into main Feb 10, 2026
12 checks passed
@meaghanlewis meaghanlewis deleted the copilot/update-persistence-documentation branch February 10, 2026 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ado Persistence Documentation Out of Date

3 participants