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

Include predefined check functions by default when applying custom checks by metadata #203

Merged
merged 7 commits into from
Mar 3, 2025

Conversation

mwojtyczka
Copy link
Contributor

Changes

  • Include predefined check functions by default when applying checks with metadata (yaml/json). Previously, when defining custom check functions, one had to explicitly import pre-defined functions as well which was very cumbersome and not intuitive.

Linked issues

Resolves #48

Tests

  • manually tested
  • added unit tests
  • added integration tests

@mwojtyczka mwojtyczka requested a review from a team as a code owner March 3, 2025 12:05
@mwojtyczka mwojtyczka requested review from tombonfert and removed request for a team March 3, 2025 12:05
@mwojtyczka mwojtyczka changed the title Include predefined check functions by default when applying checks with metadata (yaml/json) Include predefined check functions by default when applying custom checks by metadata Mar 3, 2025
Copy link

github-actions bot commented Mar 3, 2025

✅ 127/127 passed, 1 skipped, 16m54s total

Running from acceptance #533

@mwojtyczka mwojtyczka merged commit 5835138 into main Mar 3, 2025
9 checks passed
@mwojtyczka mwojtyczka deleted the func_registry branch March 3, 2025 14:18
mwojtyczka added a commit that referenced this pull request Mar 10, 2025
* Added uniqueness check([#200](#200)). A uniqueness check has been added, which reports an issue for each row containing a duplicate value in a specified column. This resolves issue [154](#154).
* Added column expression support for limits in not less and not greater than checks, and updated docs ([#200](#200)). This commit introduces several changes to simplify and enhance data quality checking in PySpark workloads for both streaming and batch data. The naming conventions of rule functions have been unified, and the `is_not_less_than` and `is_not_greater_than` functions now accept column names or expressions as limits. The input parameters for range checks have been unified, and the logic of `is_not_in_range` has been updated to be inclusive of the boundaries. The project's documentation has been improved, with the addition of comprehensive examples, and the contribution guidelines have been clarified. This change includes a breaking change for some of the checks. Users are advised to review and test the changes before implementation to ensure compatibility and avoid any disruptions. Reslves issues: [131](#131), [197](#200), [175](#175), [205](#205)
* Include predefined check functions by default when applying custom checks by metadata ([#203](#203)). The data quality engine has been updated to include predefined check functions by default when applying custom checks using metadata in the form of YAML or JSON. This change simplifies the process of defining custom checks, as users no longer need to manually import predefined functions, which were previously required and could be cumbersome. The default behavior now is to import all predefined checks. The `validate_checks` method has been updated to accept a dictionary of custom check functions instead of global variables. This improvement resolves issue [#48](#48).
@mwojtyczka mwojtyczka mentioned this pull request Mar 10, 2025
mwojtyczka added a commit that referenced this pull request Mar 10, 2025
* Added uniqueness
check([#200](#200)). A
uniqueness check has been added, which reports an issue for each row
containing a duplicate value in a specified column. This resolves issue
[154](#154).
* Added column expression support for limits in not less and not greater
than checks, and updated docs
([#200](#200)). This commit
introduces several changes to simplify and enhance data quality checking
in PySpark workloads for both streaming and batch data. The naming
conventions of rule functions have been unified, and the
`is_not_less_than` and `is_not_greater_than` functions now accept column
names or expressions as limits. The input parameters for range checks
have been unified, and the logic of `is_not_in_range` has been updated
to be inclusive of the boundaries. The project's documentation has been
improved, with the addition of comprehensive examples, and the
contribution guidelines have been clarified. This change includes a
breaking change for some of the checks. Users are advised to review and
test the changes before implementation to ensure compatibility and avoid
any disruptions. Reslves issues:
[131](#131),
[197](#200),
[175](#175),
[205](#205)
* Include predefined check functions by default when applying custom
checks by metadata
([#203](#203)). The data
quality engine has been updated to include predefined check functions by
default when applying custom checks using metadata in the form of YAML
or JSON. This change simplifies the process of defining custom checks,
as users no longer need to manually import predefined functions, which
were previously required and could be cumbersome. The default behavior
now is to import all predefined checks. The `validate_checks` method has
been updated to accept a dictionary of custom check functions instead of
global variables. This improvement resolves issue
[#48](#48).
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.

[FEATURE]: Provide registry for check functions instead of using globals
2 participants