-
Notifications
You must be signed in to change notification settings - Fork 549
Shared Tree: Schema FormatV2 and codecs #24812
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
Conversation
- Added in-memory schema format v2. - Implemented the schema format v2 codec. - Updated test utils that previously assumed there was only one format. - Added snapshots for many of the v2 tests.
…ion as a parameter. - Fixed naming of schema index formats. - Changed schemaChangeFormat to be schema format-agnostic.
…formats. - Moved SchemaCodecVersion.
Co-authored-by: Craig Macomber (Microsoft) <42876482+CraigMacomber@users.noreply.github.com>
… we want something more intuitive (e.g., minimum client version). - Split the encode method for TreeNodeStoredSchema.
…luidFramework into metadata-schema-3
…cVersion. - Changed schema-stored/FormatV2 to only define the new elements. - Updated all instantiations of TreeFieldStoredSchema so that persistedMetadata is specified. - HACK: importCompressed needs to handle both schema formats. A few of the roundtrip JSON tests are failing.
Co-authored-by: Alex Villarreal <716334+alexvy86@users.noreply.github.com>
- Made importCompressed always use a v1 schema when encoding.
- Added better test data to one of the codec tests for the v2 schema format.
…added in another PR. - Updated a comment.
- Added tests calling schemaFactoryAlpha.
Co-authored-by: Abram Sanderson <Abram.sanderson@gmail.com>
/azp run Build - client packages |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving for docs. Left a few comments on the changeset but can be addressed in a follow-up PR.
Co-authored-by: Alex Villarreal <716334+alexvy86@users.noreply.github.com>
🔗 No broken links found! ✅ Your attention to detail is admirable. linkcheck output
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed the delta since previous approval; merge from main and a doc update.
Adds support for persisting metadata along with the schema.
Changes
Notes for reviewers
Todo