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

Lenient schema validation #145

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

skestle
Copy link
Contributor

@skestle skestle commented Jun 10, 2021

(This single commit PR requires the deep DF validation changes)
When validating data against a schema, it verifies that all field elements are the same.

However, a data frame column that is specified as non-null should be considered valid against a schema column that allows nulls.

This PR implements that change.

@skestle skestle force-pushed the lenient-schema-validation branch from e857258 to a90b4bb Compare June 10, 2021 02:24
@skestle
Copy link
Contributor Author

skestle commented Jun 10, 2021

This is draft simply because #144 has not been merged.

requiredSchema
)

c.validateSchema()
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you please add a test assertion?

skestle added 2 commits June 21, 2021 10:21
…different order (mrpowers-io#144)

* Highlighted issues with schema validation's order sensitivity of nested structures.

* Implemented recursive schema checker

Enables nested structures to be compared regardless of order.
Previously, a dataframe would be considered invalid if it had not-null data,
 but the schema allowed the data to be nullable
@skestle skestle force-pushed the lenient-schema-validation branch from a90b4bb to c84ded8 Compare June 20, 2021 22:29
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