-
Notifications
You must be signed in to change notification settings - Fork 549
Shared Tree: In-memory format and API changes for persisted schema metadata #24759
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
TommyBrosman
merged 67 commits into
microsoft:main
from
TommyBrosman:metadata-in-memory
Jun 13, 2025
Merged
Changes from 64 commits
Commits
Show all changes
67 commits
Select commit
Hold shift + click to select a range
bd8406a
Initial change.
TommyBrosman 3165fe3
persistedMetadata is now backed by an object instead of a string.
TommyBrosman 9709ede
Updated codec.spec.ts to cover FormatV2.
TommyBrosman ba058e1
Removed extraneous whitespace change.
TommyBrosman b819742
- Changed TreeNodeStoredSchema implementations to take the write vers…
TommyBrosman 0b5c842
- Changed the stored schema implementation to handle multiple schema …
TommyBrosman ecdca7a
Merge branch 'main' into metadata-schema-3
TommyBrosman 1117c2c
Apply suggestions from code review
TommyBrosman 94a5c39
- Switched to using the SchemaCodecVersion in public APIs. Eventually…
TommyBrosman 302cb21
Merge branch 'metadata-schema-3' of https://github.com/TommyBrosman/F…
TommyBrosman 0ba34d1
- Removed the schema version constant and replaced it with SchemaCode…
TommyBrosman 3275ee0
Minor: reverted changes to FluidClientVersion utils.
TommyBrosman 7c48b9d
Switched back to using min client version for importCompressed.
TommyBrosman a2f4195
Minor: reverted accidental change.
TommyBrosman 4786dcc
Apply suggestions from code review
TommyBrosman 6fa8836
Made persisted metadata schema field naming consistent.
TommyBrosman e4064c7
- Created a separate storedSchemaDecodeDispatcher for v2 schemas.
TommyBrosman ab1b882
- Refactored the node kind dispatch.
TommyBrosman 1027daa
Updated snapshots.
TommyBrosman e527eba
- Removed minimum client version changes. Equivalent changes will be …
TommyBrosman 3eae1f6
Removed more minimumClientVersion glue.
TommyBrosman 98e1b76
Updated API files.
TommyBrosman ecc77c9
Rename: SchemaCodecVersion -> SchemaVersion.
TommyBrosman 37e900f
Merge branch 'main' into metadata-schema-3
TommyBrosman 737633b
Removed top-level persisted metadata.
TommyBrosman 4bb136c
Fixed comments.
TommyBrosman 0bcb90f
Exposed persistedMetadata as a JsonCompatibleReadOnlyObject.
TommyBrosman a675338
Changeset.
TommyBrosman 1d1ef5f
Wired up toStoredSchema. Still needs tests.
TommyBrosman b9898e8
- Updated an old snapshot that didn't include the metadata field.
TommyBrosman 6ca189a
Refactor: persistedMetadata -> metadata on persisted types.
TommyBrosman 85240dc
Apply suggestions from code review
TommyBrosman 8883d45
- Updated the changeset description
TommyBrosman 9c2c13f
Reverted unnecessary change.
TommyBrosman c62f7ec
Wired up node schema metadata persistence. Currently errors on tests …
TommyBrosman 2f6b519
Reverted unneeded change.
TommyBrosman ae1779b
Removed tests for the alpha API I removed in a previous revision.
TommyBrosman 7eff8ab
This revision adds simple-tree persistence for node and field schema …
TommyBrosman f3f572c
- Fixed missing persistedMetadata on fields. toStoredSchema.ts and sh…
TommyBrosman faaf2f7
Merge branch 'main' into metadata-schema-3
TommyBrosman f80d027
Removed changes for persisted format, codecs. A few tests are failing.
TommyBrosman a7e72f0
Minor cleanup.
TommyBrosman 3dcfb10
Updated changeset description.
TommyBrosman dcf4e0e
Merge branch 'main' into metadata-in-memory
TommyBrosman bca4c8c
Regenerated API Extractor output.
TommyBrosman b66c928
Renamed the in-memory version of the persistedMetadata field to persi…
TommyBrosman 98c853e
Merge branch 'main' into metadata-in-memory
TommyBrosman 155b320
- Added a simple default comparison for persistedMetadata.
TommyBrosman 4ff48d8
Minor: clarified use of persistedMetadata in tests.
TommyBrosman 6400590
- Removed redundant alpha APIs. For now they are exposed only as static
TommyBrosman b329851
Reverted changes that belong in a different PR.
TommyBrosman 6b425b2
Removed more changes that shouldn't be in this PR.
TommyBrosman c0eb6dc
Apply suggestions from code review
TommyBrosman c9a1346
Reverted another unrelated change.
TommyBrosman 03d3bc4
Merge branch 'main' into metadata-in-memory
TommyBrosman 002a80c
Updated API Extractor output.
TommyBrosman e3443bd
Changed SchemaFactoryAlpha calls to use instance methods.
TommyBrosman 72e32ab
- Cleaned up the type for in-memory persistedMetadata.
TommyBrosman 9d86b72
Merge branch 'main' into metadata-in-memory
TommyBrosman f4b258a
Regenerated API Extractor output.
TommyBrosman a743d39
Update packages/dds/tree/src/simple-tree/api/schemaFactory.ts
TommyBrosman 01d5eae
Docs.
TommyBrosman 58bbeb7
Merge branch 'metadata-in-memory' of https://github.com/TommyBrosman/…
TommyBrosman c4e29c8
Merge branch 'main' into metadata-in-memory
TommyBrosman e2f350c
- Cleaned up comparators and added tests.
TommyBrosman ea00cc9
Merge branch 'main' into metadata-in-memory
TommyBrosman a6348b4
Merge branch 'main' into metadata-in-memory
TommyBrosman File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.