Skip to content
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

Minor improvements to Ion Schema Model #288

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

popematt
Copy link
Contributor

Issue #, if available:

None

Description of changes:

  • Updates the AnnotationsV2 model to have special support for the simplified syntax, since it was discovered that there are significant performance improvements by doing so. (See Improves performance of annotations constraint #287) Also updates the schema reader and SchemaSymbolUtil accordingly.
  • Fixes a typo in ContinuousRange.toString()
  • Adds factory functions that will allow users to create a Fields constraint directly from TypeArguments instead of requiring VariableOccurringTypeArguments.

Related PRs in ion-schema, ion-schema-tests, ion-schema-schemas:

None

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@codecov
Copy link

codecov bot commented Sep 22, 2023

@popematt popematt merged commit 9165691 into amazon-ion:master Sep 25, 2023
3 of 5 checks passed
@popematt popematt deleted the ann_refactor branch September 25, 2023 20:42
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.

2 participants